[go: up one dir, main page]
Skip to content

clang-format: don't enforce the column limit

Karthik Nayakrequested to merge
405-clang-format-revisit-rules-to-make-the-job-easier-to-follow into master

The clang-format CI job is currently cluttered due to too many errors being reported. See some of the examples here:

So modify the clang-format with the following changes:

  1. Remove the column limit since this is more of a guideline and we always tend to prefer readability. This is the cause of most of the errors reported by the tool and should cleanup the reports so we can actually focus on the real remaining issues.
  2. Don't align expressions after linebreaks to ensure that we instead rely on 'ContinuationIndentWidth'. This fix is rather small and ensures that instead of trying to align wrapped expressions, we follow the indentation width.

Closes #405 (closed)

Merge request reports