jiridanek edited a comment on pull request #169: URL: https://github.com/apache/qpid-proton/pull/169#issuecomment-632371846
+1, I'd like to have a reasonable `.clang-format` in the project. It is nowadays _the_ way to specify C/C++ code formatting across editing tools (together with `.editorconfig`, but that is simpler tooling that cannot take language syntax into account). It would be very useful when writing a new file, so that I know what is the projects desired code formatting. I probably wouldn't start by mass reformatting of the entire codebase. That can come later. (That is, for C/C++. I probably would want to reformat all Python right away.) Good thing about autoformatters is that after codebase reformat, they can be also applied to any work-in-progress code in branches, and that will hopefully significantly decrease the effort to adopt the style there, when merging. It would be good if other Qpid projects (i.e. Dispatch) could adopt the same style. I do not have particular preference on what the style should be. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
