Capitalization in hydra dev
henryiii opened this issue · 4 comments
henryiii commented
Some of the files that used to work in Hydra 1 are broken in Hydra dev fork because they are referred to (for example, in hydra/Events.h) with differing capitalization (#include <hydra/detail/config.h>, but the file is Config.h.
Personally recommend identical caps for all files, either TitleCase or lowercase. Thrust uses lowercase.
AAAlvesJr commented
config.h is not compliant with coding rules.
AAAlvesJr commented
Config.h will be kept in the incoming release.
henryiii commented
That's fine, but the include statements need to be changed to ...Config.h>.
AAAlvesJr commented
it is the case in the incoming relase... dont worry,