search

eclipse-uprotocol / up-cpp

uProtocol Language Specific Library for C++
Apache License 2.0
15 stars 24 forks source link

clang-format: Extend default line length #142

Open billpittman opened 3 months ago

billpittman commented 3 months ago

Line length of 80 characters is too short in several cases where nested statements are used and long variable names cause the line to be wrapped and makes the code less readable. Additionally, most tools (vscode, etc) allow for showing far more than 80 characters per line.

gregmedd commented 3 months ago

We are not making format changes until a publicly available style guide has been published and approved.

billpittman commented 3 months ago

We are not making format changes until a publicly available style guide has been published and approved.

Please treat this as merely a suggestion/proposal that matches my preferences.

gregmedd commented 3 months ago

Personally, I often end up working in windows that are <100 columns wide, so I tend to prefer an 80 column limit.

When we work on the style guide, I would be open to reviewing specific code examples where 80 columns is making the code harder to read, alongside comparisons of other column width limits for the same code.