[clang-tools-extra] r254635 - Fixed header determination logic. Was missing extensionless headers in coverage check.

[David Blaikie via cfe-commits]

test case?

On Thu, Dec 3, 2015 at 11:20 AM, John Thompson via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: jtsoftware
> Date: Thu Dec  3 13:20:43 2015
> New Revision: 254635
>
> URL: http://llvm.org/viewvc/llvm-project?rev=254635&view=rev
> Log:
> Fixed header determination logic. Was missing extensionless headers in
> coverage check.
>
> Modified:
>     clang-tools-extra/trunk/modularize/ModularizeUtilities.cpp
>
> Modified: clang-tools-extra/trunk/modularize/ModularizeUtilities.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/modularize/ModularizeUtilities.cpp?rev=254635&r1=254634&r2=254635&view=diff
>
> ==============================================================================
> --- clang-tools-extra/trunk/modularize/ModularizeUtilities.cpp (original)
> +++ clang-tools-extra/trunk/modularize/ModularizeUtilities.cpp Thu Dec  3
> 13:20:43 2015
> @@ -468,7 +468,7 @@ std::string ModularizeUtilities::getCano
>  bool ModularizeUtilities::isHeader(StringRef FileName) {
>    StringRef Extension = llvm::sys::path::extension(FileName);
>    if (Extension.size() == 0)
> -    return false;
> +    return true;
>    if (Extension.equals_lower(".h"))
>      return true;
>    if (Extension.equals_lower(".inc"))
>
>
> _______________________________________________
> 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/20151203/5554b20e/attachment.html>