[cfe-dev] New warning for mismatched include case

Aaron Ballman aaron at aaronballman.com
Wed Jul 18 16:10:02 PDT 2012


On Wed, Jul 18, 2012 at 5:09 PM, Jason Haslam <jason.haslam at gmail.com> wrote:
> The attached (incomplete proof-of-concept) patch provides a new warning that fires on case-insensitive file systems when the case of the file name in the #include directive doesn't match the case of the file name on disk. The idea is to avoid broken commits that fail to build on case-sensitive file systems. For example:
>
> jason$ ls
> test.cpp test.h
> jason$ cat test.cpp
> #include "Test.h"
> jason$ clang++ -fsyntax-only test.cpp
> test.cpp:1:10: warning: include case does not match the case of the file on disk
> #include "Test.h"
>          ^
> 1 warning generated.
>
> What do you think? Is this desirable? Does the direction look right?

I think this is a great idea (and could definitely use a FixIt).

~Aaron




More information about the cfe-dev mailing list