Support c extension for cpplint.py

[groegeorg]

It doesn't seem to be possible to run cpplint.py on *.c files with this plugin. On the command line I can run cpplint.py --extensions=c,h foo.c, but I can't give an argument to cpplint.py in the CppStyle settings dialog.

I even tried to make cpplint.py natively support the c extension, by amending the variable _valid_extensions in the Python code. Then it works on the command line even without the --extensions= argument. But strangely, CppStyle still doesn't work with *.c files.

Are there any plans to support this, or is there any possible workaround?

[wangzw]

As a workaround you can set *.c in project settings.

cpplint.py is a tool for c++, so *.c is not in the scope, correct me if I'm wrong.

[groegeorg]

I didn't know this option, thanks for the hint! However, even with adding *.c to the list or removing all entries from the list, cppstyle.py doesn't output anything for .c files :-(

Also, having to modify this list of file extensions manually for every single checker (there are around 60!) is more than cumbersome...

You are right, cpplint.py is meant for C++, as is the Google C++ Style Guide. We use the Google Style Guide for our C++ projects, but we also have a fair amount of C projects and don't want to introduce a new coding style for those. Since C is (more or less) a subset of C++, the rules of the Google Style Guide can also be applied fairly well to C code. All we need is a style checker tool, because without it any rule is useless. It would be a pity if we can't use cpplint.py, just because of the file extension...

[Krozark]

The same is true for .hpp file. cpplint is not called.
If I understand the pluggin, the trouble is here :

CppStyle/plugin/src/org/wangzw/plugin/cppstyle/CpplintChecker.java

Line 151 in 7c785f6

value[1] = new Path("*.h");

There is some hard codded extension restriction, even if the documentation say that EVERY file save is pass to cpplint. The correct way is to remove any existing restriction, has cpplint olready have a configurations file for this.