search

arsenm / sanitizers-cmake

CMake modules to help use sanitizers
Other
377 stars 65 forks source link

handle header without extension #23

Closed gunrot closed 6 years ago

gunrot commented 6 years ago

ignore files without extension. this can be modern c++ header

TheErk commented 6 years ago

Not sure whether if it's relevant but CMake already guess LANGUAGE of sources files from extension and set the LANGUAGE property of the source file.

Shouldn't we use what CMake already guessed? cf: cmake --help-property LANGUAGE

arsenm commented 6 years ago

I agree the whole function seems like something cmake should have determined