languages: Recognize module interface file as C++ file#45968
Conversation
This comment was marked as off-topic.
This comment was marked as off-topic.
|
Thanks! |
| name = "C++" | ||
| grammar = "cpp" | ||
| path_suffixes = ["cc", "hh", "cpp", "h", "hpp", "cxx", "hxx", "c++", "h++", "ipp", "inl", "ino", "ixx", "cu", "cuh", "C", "H"] | ||
| path_suffixes = ["cc", "hh", "cpp", "h", "hpp", "cxx", "hxx", "c++", "h++", "ipp", "inl", "ino", "ixx", "cu", "cuh", "C", "H", "cppm", "ccm", "cxxm"] |
There was a problem hiding this comment.
@hokein does it make sense to add .tpp files too?
Can we also add .tpp suffix for C++ ? Thank you,
There was a problem hiding this comment.
On second thought: what about this list mentioned here?
There was a problem hiding this comment.
I think it makes sense to add them. (I would rather do it in a separate PR, as this one only focuses on the module files.)
There was a problem hiding this comment.
I mean I’m not against updating the PR title and just squeezing in the rest -- the end goal is greater file_types recognition for C++ projects in Zed. But maybe I’m still misunderstanding something.
|
Is there anything else I should address on my end to get this ready for merge? |
…s#45968) https://clang.llvm.org/docs/StandardCPlusPlusModules.html#file-name-requirements, `.cppm`, '.ccm', ".cxxm" files are C++20 module interface files. Release Notes: - Recognize module interface file as C++ file.
…s#45968) https://clang.llvm.org/docs/StandardCPlusPlusModules.html#file-name-requirements, `.cppm`, '.ccm', ".cxxm" files are C++20 module interface files. Release Notes: - Recognize module interface file as C++ file.
…s#45968) https://clang.llvm.org/docs/StandardCPlusPlusModules.html#file-name-requirements, `.cppm`, '.ccm', ".cxxm" files are C++20 module interface files. Release Notes: - Recognize module interface file as C++ file.
https://clang.llvm.org/docs/StandardCPlusPlusModules.html#file-name-requirements,
.cppm, '.ccm', ".cxxm" files are C++20 module interface files.Release Notes: