[clang-tools-extra] r241724 - Fixed erroneous comment.

Jonathan Roelofs jonathan at codesourcery.com
Wed Jul 8 14:08:22 PDT 2015



On 7/8/15 2:57 PM, John Thompson wrote:
> Author: jtsoftware
> Date: Wed Jul  8 15:57:32 2015
> New Revision: 241724
>
> URL: http://llvm.org/viewvc/llvm-project?rev=241724&view=rev
> Log:
> Fixed erroneous comment.
>
> Modified:
>      clang-tools-extra/trunk/modularize/Modularize.cpp
>
> Modified: clang-tools-extra/trunk/modularize/Modularize.cpp
> URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/modularize/Modularize.cpp?rev=241724&r1=241723&r2=241724&view=diff
> ==============================================================================
> --- clang-tools-extra/trunk/modularize/Modularize.cpp (original)
> +++ clang-tools-extra/trunk/modularize/Modularize.cpp Wed Jul  8 15:57:32 2015
> @@ -298,7 +298,7 @@ static cl::list<std::string>
>   IncludePaths("I", cl::desc("Include path for coverage check."),
>   cl::ZeroOrMore, cl::value_desc("path"));
>
> -// Option for just doing the coverage check.
> +// Option for disabling the coverage check.

Deleting it is probably better... it's not a very useful comment.

>   static cl::opt<bool>
>   NoCoverageCheck("no-coverage-check", cl::init(false),
>   cl::desc("Don't do the coverage check."));
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>

-- 
Jon Roelofs
jonathan at codesourcery.com
CodeSourcery / Mentor Embedded



More information about the cfe-commits mailing list