[HTML5] Address code style comments from timeless

RESOLVED FIXED

Status

()

defect
--
trivial
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: hsivonen, Assigned: hsivonen)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

NS_ASSERTION((!loadAsHtml5) has pointless parentheses still. Get rid of them.
Morphing the bug to be more general, because dealing with each comment individually would lead to a ton of one-line patches to review and check in.

This is now a catch-all for parentheses, indents and comments.
Summary: [HTML5] Remove pointless parentheses in assert → [HTML5] Address code style comments from timeless
Not included:
 * Stuff that involves changing the translator.
 * Stuff that has dedicated bugs.
 * Stuff that would make it more difficult to transform the Java code to use Mozilla-style write params for string returns.
 * Use of contract id in place of class id.
Attachment #386054 - Flags: superreview?(mrbkap)
Attachment #386054 - Flags: review?(jonas)
Attachment #386054 - Flags: superreview?(mrbkap) → superreview+
Comment on attachment 386054 [details] [diff] [review]
Patch addressing comments from timeless

mrbkap, requesting r, since it seems sr isn't what's needed for this, but I think it would be inappropriate for me just treat your sr as an r.
Attachment #386054 - Flags: review?(jonas) → review?(mrbkap)
Attachment #386054 - Flags: review?(mrbkap) → review+
Pushed. http://hg.mozilla.org/mozilla-central/rev/712073dde3c0
Thanks!
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.