jiridanek commented 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 semantics 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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

Reply via email to