Closed Bug 1508298 Opened Last year Closed Last year

Remove an incorrect reformat of a comment in dom/media/DOMMediaStream.h

Categories

(Firefox Build System :: Lint and Formatting, enhancement)

enhancement
Not set

Tracking

(firefox65 fixed)

RESOLVED FIXED
mozilla65
Tracking Status
firefox65 --- fixed

People

(Reporter: Sylvestre, Assigned: Sylvestre)

References

Details

Attachments

(1 file)

Interesting to see that clang-format is not just making whitespace changes, but reordering "*" and "(removed)" etc.

I wonder what is behind its decision to keep some multi-space sequences but not others.

At least it is only doing this (I assume) when the comment was not complying to the 80 column limit.

https://hg.mozilla.org/integration/autoland/diff/0ceae9db9ec0/dom/media/DOMMediaStream.h#l1.90
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f70098015fd4
Disable the reformatting of the comment r=jya
Assignee: nobody → sledru
https://hg.mozilla.org/mozilla-central/rev/f70098015fd4
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
See Also: → 1508472
You need to log in before you can comment on or make changes to this bug.