New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for clang format. #21
Comments
If you can give me the workflow of how the program provisioning for the eclipse formatter works I could work on a PR for this feature. |
I guess Eclipse formatter is a Java program, so the provisioning is easier there. |
The eclipse CDT formatter isn't up to date with the newer versions of c++ in its parsing (It actually isn't even fully for older versions) so updating the eclipse format would basically involve fixing the whole parsing of the CDT formatter. I'll have to look whether it is possible to to provisioning for clang-format. Otherwise the user would have to provide the installation. While not ideal it would still be a good addition. |
Ah, ok. I see there's npm-packaged clang-format: https://www.npmjs.com/package/clang-format which seems to be implemented for multiple platforms. I do not see existing So the options could be: |
However, publishing to Central is preferred as it would work better when building with limited internet connectivity (e.g. when corporate proxies are used) |
I've created https://github.com/autostyle/clang-format that will be published like https://oss.sonatype.org/content/repositories/comgithubautostyle-1024/com/github/autostyle/autostyle-clang-format/1.0/ |
The eclipse CDT formatted hasn’t been up to the standard of newer c++ versions for some time. e.g it has a hard time to parse angles brackets around template arguments, mistaking them as comparison operators.
It would be great to be able to use the clang formatted instead, as it is a lot better at parsing newer c++ code.
The text was updated successfully, but these errors were encountered: