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
Duplicate of this bug: 501013
Duplicate of this bug: 501011
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) → superreview+
Duplicate of this bug: 503068
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.