Remove all trailing whitespaces

RESOLVED FIXED in Firefox 56

Status

enhancement
RESOLVED FIXED
2 years ago
7 months ago

People

(Reporter: sylvestre, Assigned: sylvestre)

Tracking

Trunk
mozilla56

Firefox Tracking Flags

(firefox56 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
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

2 years ago
Summary: Remove all trailing whitespace → Remove all trailing whitespaces
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 3

2 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+

Comment 4

2 years ago
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6a629adbb62a
Remove all trailing whitespaces r=Ehsan

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/6a629adbb62a
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
(Assignee)

Updated

2 years ago
Assignee: nobody → sledru

Updated

a year ago
Product: Core → Firefox Build System
(Assignee)

Updated

7 months ago
Blocks: 1489454
You need to log in before you can comment on or make changes to this bug.