Skip to content

Conversation

isidorostsa
Copy link
Contributor

@isidorostsa isidorostsa commented May 30, 2024

Maybe .hpp2 files should be added instead, or both? Came across this need when I added headers to a project. Currently the C++2 docs do not specify anything about headers, but it is a nice to have.

Thank you for making this amazing extension by the way, it is a life-saver!

Maybe .hpp2 files should be added instead, or both?
@elazarcoh elazarcoh merged commit 9313b5a into elazarcoh:master May 30, 2024
@elazarcoh
Copy link
Owner

@isidorostsa Thanks!

@isidorostsa
Copy link
Contributor Author

Thanks for merging, I think .hpp2 files should also be supported. What do you think?

@elazarcoh
Copy link
Owner

I'm not actively developing with cpp2 so I wouldn't know.
I went through the cppfront GitHub issues and found that people use h2. If you think hpp2 is being used as well, then sure.

@isidorostsa isidorostsa deleted the patch-1 branch May 30, 2024 18:11
@isidorostsa
Copy link
Contributor Author

Given this discussion: hsutter/cppfront#1091 on the C++2 wiki, we should stick to .h2 for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants