Open Bug 1551246 Opened 6 years ago Updated 3 years ago

[Automated review] clang-format wants to make a line too long

Categories

(Developer Infrastructure :: Lint and Formatting, defect, P3)

Product:
Developer Infrastructure
Component:
Lint and Formatting
Type:
defect

Tracking

(Not tracked)

People

(Reporter: jfkthame, Unassigned)

Details

I thought the style guide called for a max length of 80 characters, aside from a few exceptional circumstances.[1]

In https://phabricator.services.mozilla.com/D30912#908883, the re-wrapping proposed by the clang-format patch makes the affected line 81 characters long, for no good reason that I can see.

[1] https://google.github.io/styleguide/cppguide.html#Line_Length

Priority: -- → P3
Product: Firefox Build System → Developer Infrastructure
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.