Closed
Bug 1378712
Opened 7 years ago
Closed 7 years ago
Remove all trailing whitespaces
Categories
(Developer Infrastructure :: Source Code Analysis, enhancement)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox56 fixed)
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: Sylvestre, Assigned: Sylvestre)
References
Details
Attachments
(1 file)
To avoid follow up patches, I am proposing that we remove them all at once. $ sed --in-place 's/[[:space:]]\+$//' $(find . -iname '*.cpp' -o -iname '*.h' -o -iname '*.c') $ hg revert $(cat tools/rewriting/ThirdPartyPaths.txt )
Assignee | ||
Updated•7 years ago
|
Summary: Remove all trailing whitespace → Remove all trailing whitespaces
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8883896 [details] Bug 1378712 - Remove all trailing whitespaces https://reviewboard.mozilla.org/r/154892/#review159960 Patches like this aren't reviewable in MozReview (please open this in MozReview to see why.) I reviewed the sed script in comemnt 0, r+ on the sed script. Rubberstamp on the patch (but I didn't really look at it.)
Attachment #8883896 -
Flags: review?(ehsan) → review+
Pushed by sledru@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6a629adbb62a Remove all trailing whitespaces r=Ehsan
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6a629adbb62a
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → sledru
Updated•7 years ago
|
status-firefox57:
affected → ---
Updated•6 years ago
|
Product: Core → Firefox Build System
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•