r331536 - [NFC]Convert Class to use member initialization instead of inline.

David Blaikie via cfe-commits cfe-commits at lists.llvm.org
Mon May 7 12:02:33 PDT 2018


Perhaps this should use non-static data member initializers instead?

On Fri, May 4, 2018 at 9:23 AM Erich Keane via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: erichkeane
> Date: Fri May  4 09:19:53 2018
> New Revision: 331536
>
> URL: http://llvm.org/viewvc/llvm-project?rev=331536&view=rev
> Log:
> [NFC]Convert Class to use member initialization instead of inline.
>
> 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=331536&r1=331535&r2=331536&view=diff
>
> ==============================================================================
> --- cfe/trunk/include/clang/Frontend/DependencyOutputOptions.h (original)
> +++ cfe/trunk/include/clang/Frontend/DependencyOutputOptions.h Fri May  4
> 09:19:53 2018
> @@ -65,15 +65,11 @@ public:
>    std::string ModuleDependencyOutputDir;
>
>  public:
> -  DependencyOutputOptions() {
> -    IncludeSystemHeaders = 0;
> -    ShowHeaderIncludes = 0;
> -    UsePhonyTargets = 0;
> -    AddMissingHeaderDeps = 0;
> -    IncludeModuleFiles = 0;
> -    ShowIncludesDest = ShowIncludesDestination::None;
> -    OutputFormat = DependencyOutputFormat::Make;
> -  }
> +  DependencyOutputOptions()
> +      : IncludeSystemHeaders(0), ShowHeaderIncludes(0),
> UsePhonyTargets(0),
> +        AddMissingHeaderDeps(0), IncludeModuleFiles(0),
> +        ShowIncludesDest(ShowIncludesDestination::None),
> +        OutputFormat(DependencyOutputFormat::Make) {}
>  };
>
>  }  // end namespace clang
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180507/e2913bfe/attachment-0001.html>


More information about the cfe-commits mailing list