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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits