r331688 - [NFC] Move 2 variable initialization from Ctor to member initializers.

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Mon May 7 15:01:07 PDT 2018


Author: erichkeane
Date: Mon May  7 15:01:06 2018
New Revision: 331688

URL: http://llvm.org/viewvc/llvm-project?rev=331688&view=rev
Log:
[NFC] Move 2 variable initialization from Ctor to member initializers.

In response to dblaikie's suggestion on r331536, replace the two enum
typed variable initializers in the constructor with member initializers.

Modified:
    cfe/trunk/include/clang/Frontend/DependencyOutputOptions.h

Modified: cfe/trunk/include/clang/Frontend/DependencyOutputOptions.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Frontend/DependencyOutputOptions.h?rev=331688&r1=331687&r2=331688&view=diff
==============================================================================
--- cfe/trunk/include/clang/Frontend/DependencyOutputOptions.h (original)
+++ cfe/trunk/include/clang/Frontend/DependencyOutputOptions.h Mon May  7 15:01:06 2018
@@ -34,10 +34,10 @@ public:
   unsigned IncludeModuleFiles : 1; ///< Include module file dependencies.
 
   /// Destination of cl.exe style /showIncludes info.
-  ShowIncludesDestination ShowIncludesDest;
+  ShowIncludesDestination ShowIncludesDest = ShowIncludesDestination::None;
 
   /// The format for the dependency file.
-  DependencyOutputFormat OutputFormat;
+  DependencyOutputFormat OutputFormat = DependencyOutputFormat::Make;
 
   /// The file to write dependency output to.
   std::string OutputFile;
@@ -67,9 +67,7 @@ public:
 public:
   DependencyOutputOptions()
       : IncludeSystemHeaders(0), ShowHeaderIncludes(0), UsePhonyTargets(0),
-        AddMissingHeaderDeps(0), IncludeModuleFiles(0),
-        ShowIncludesDest(ShowIncludesDestination::None),
-        OutputFormat(DependencyOutputFormat::Make) {}
+        AddMissingHeaderDeps(0), IncludeModuleFiles(0) {}
 };
 
 }  // end namespace clang




More information about the cfe-commits mailing list