gradle-native icon indicating copy to clipboard operation
gradle-native copied to clipboard

Consider .hpp and .hxx files to be header files too (C++)

Open BrainStone opened this issue 5 years ago • 3 comments

Expected Behavior

Consider all 3 common C++ header endings as valid header file endings: .h, .hpp and .hxx

Current Behavior

Currently the plugin is hardcoded to only use .h files: https://github.com/gradle/gradle/blob/ea650c049886424271b3583cc57ba260458f68c0/subprojects/language-native/src/main/java/org/gradle/language/cpp/internal/DefaultCppComponent.java#L112

Context

Other plugins trying to find the header files (like the VisualStudio plugin) will only get a list of files with the .h ending when they try to get a list of all header files.

BrainStone avatar Nov 01 '20 22:11 BrainStone

Thanks for opening this issue. Nokee also consider h++ extension as being C++ headers. However, we also allow this default value to be configurable.

lacasseio avatar Nov 02 '20 00:11 lacasseio

Yeah you are right. The .h++ extension should be supported as well.

BrainStone avatar Nov 02 '20 21:11 BrainStone

Hello,

What's about concidering *.ipp files as well?

abelavusau avatar Jul 17 '23 20:07 abelavusau