search

tkruse / cpplint

Fork of google cpplint, deprecated
3 stars 4 forks source link

Fix extensions #22

Closed tkruse closed 8 years ago

tkruse commented 8 years ago

@itamaro, @mattyclarkson, @massenz Can someone review these changes?

Basically all this does is to allow different file extensions, similar to #17, but with more tests

tkruse commented 8 years ago

Excellent review, I made the changes you suggested as additional commit, please look at it when you find the time

tkruse commented 8 years ago

now also added more text to the usage section as suggested in #17

tkruse commented 8 years ago

sorry, this does not work yet, I do need those tests

tkruse commented 8 years ago

@itamaro I wont have time to write tests this week. I fixed the problem though, and you may review the solution for the variables and the usage string.

tkruse commented 8 years ago

Superceded by https://github.com/theandrewdavis/cpplint/pull/14