]> granicus.if.org Git - clang/commitdiff
Make DiagnosticsEngine() take DiagOpts as DiagnosticsEngine.
authorNico Weber <nicolasweber@gmx.de>
Wed, 17 Jan 2018 02:55:27 +0000 (02:55 +0000)
committerNico Weber <nicolasweber@gmx.de>
Wed, 17 Jan 2018 02:55:27 +0000 (02:55 +0000)
No behavior change, but makes it a bit clearer that DiagnosticsEngine adds a
ref to DiagOpts.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@322611 91177308-0d34-0410-b5e6-96231b3b80d8

include/clang/Basic/Diagnostic.h
lib/Basic/Diagnostic.cpp

index a7458d45618ee8482457550ba8dab80070fab272..d87d25f2543378406a4762ee33e96835dc92d1b0 100644 (file)
@@ -402,7 +402,7 @@ private:
 
 public:
   explicit DiagnosticsEngine(IntrusiveRefCntPtr<DiagnosticIDs> Diags,
-                             DiagnosticOptions *DiagOpts,
+                             IntrusiveRefCntPtr<DiagnosticOptions> DiagOpts,
                              DiagnosticConsumer *client = nullptr,
                              bool ShouldOwnClient = true);
   DiagnosticsEngine(const DiagnosticsEngine &) = delete;
index 26baa838f8c687cf585e763226d226e8cebc57ae..5903d72ea5e01bb1da3f517ec23a3d1f6387d2c3 100644 (file)
@@ -57,11 +57,11 @@ static void DummyArgToStringFn(DiagnosticsEngine::ArgumentKind AK, intptr_t QT,
   Output.append(Str.begin(), Str.end());
 }
 
-DiagnosticsEngine::DiagnosticsEngine(IntrusiveRefCntPtr<DiagnosticIDs> diags,
-                                     DiagnosticOptions *DiagOpts,
-                                     DiagnosticConsumer *client,
-                                     bool ShouldOwnClient)
-    : Diags(std::move(diags)), DiagOpts(DiagOpts), Client(nullptr),
+DiagnosticsEngine::DiagnosticsEngine(
+    IntrusiveRefCntPtr<DiagnosticIDs> diags,
+    IntrusiveRefCntPtr<DiagnosticOptions> DiagOpts, DiagnosticConsumer *client,
+    bool ShouldOwnClient)
+    : Diags(std::move(diags)), DiagOpts(std::move(DiagOpts)), Client(nullptr),
       SourceMgr(nullptr) {
   setClient(client, ShouldOwnClient);
   ArgToStringFn = DummyArgToStringFn;