nsEscape.cpp: disable the formatting of netCharType & EscapeChars

RESOLVED FIXED in Firefox 58

Status

()

Core
XPCOM
RESOLVED FIXED
20 days ago
18 days ago

People

(Reporter: sylvestre, Assigned: sylvestre)

Tracking

(Blocks: 1 bug)

Trunk
mozilla58
Points:
---

Firefox Tracking Flags

(firefox58 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

20 days ago
The formatting is pretty bad (the tool has no way to do otherwise)
(Assignee)

Updated

20 days ago
Summary: nsEscape.cpp: disable the formatting of netCharType → nsEscape.cpp: disable the formatting of netCharType & EscapeChars
Comment hidden (mozreview-request)
(Assignee)

Updated

20 days ago
Assignee: nobody → sledru
Comment on attachment 8923710 [details]
Bug 1413103 - nsEscape.cpp: disable the formatting of netCharType & EscapeChars

https://reviewboard.mozilla.org/r/194854/#review200110

Thanks! I can imagine we'll need this for a few other tables. Possibly [Base64.cpp](http://searchfox.org/mozilla-central/rev/1c4901d060e3953e41088c038b62a0c026c1e1fb/xpcom/io/Base64.cpp).
Attachment #8923710 - Flags: review?(erahm) → review+

Comment 3

19 days ago
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ea7edb8d90e0
nsEscape.cpp: disable the formatting of netCharType & EscapeChars r=erahm
https://hg.mozilla.org/mozilla-central/rev/ea7edb8d90e0
Status: NEW → RESOLVED
Last Resolved: 18 days ago
status-firefox58: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.