superreview granted: [Bug 242760] code cleaning: rechecking checked rc's, dropping a temporary comptr, a comment change : [Attachment 147787] store rc's so ifs can check, drop temporary comptr, chang

Johnny Stenback <jst@mozilla.jstenback.com> has granted timeless@myrealbox.com
(working) <timeless@bemail.org>'s request for superreview:
Bug 242760: code cleaning: rechecking checked rc's, dropping a temporary
comptr, a comment change
http://bugzilla.mozilla.org/show_bug.cgi?id=242760

Attachment 147787: store rc's so ifs can check, drop temporary comptr, change
comment, check alloc
http://bugzilla.mozilla.org/attachment.cgi?id=147787&action=edit

------- Additional Comments from Johnny Stenback <jst@mozilla.jstenback.com>
-      element->AppendChild(text, getter_AddRefs(ignore));
+      rc = element->AppendChild(text, getter_AddRefs(ignore));
       if (NS_FAILED(rc)) 
       return rc;

Wanna fix the indentation there while you're in the neighborhood?

Very nice catches!

r+sr=jst
0
bugzilla
5/6/2004 5:50:05 PM
netscape.mozilla.reviewers 29156 articles. 0 followers. Follow

0 Replies
800 Views

Similar Articles

[PageSpeed] 28

Reply:

Similar Artilces:

superreview requested: [Bug 242760] code cleaning: rechecking checked rc's, dropping a temporary comptr, a comment change : [Attachment 147787] store rc's so ifs can check, drop temporary comptr, cha
timeless@myrealbox.com (working) <timeless@bemail.org> has asked Johnny Stenback <jst@mozilla.jstenback.com> for superreview: Bug 242760: code cleaning: rechecking checked rc's, dropping a temporary comptr, a comment change http://bugzilla.mozilla.org/show_bug.cgi?id=242760 Attachment 147787: store rc's so ifs can check, drop temporary comptr, change comment, check alloc http://bugzilla.mozilla.org/attachment.cgi?id=147787&action=edit ...

superreview requested: [Bug 244761] Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected : [Attachment 149378] Change the check for if the dropdown is show
Doron Rosenberg (IBM) <doronr@us.ibm.com> has asked Robert O'Callahan <roc@ocallahan.org> for superreview: Bug 244761: Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected http://bugzilla.mozilla.org/show_bug.cgi?id=244761 Attachment 149378: Change the check for if the dropdown is shown to the correct way http://bugzilla.mozilla.org/attachment.cgi?id=149378&action=edit ...

superreview cancelled: [Bug 244761] Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected : [Attachment 149378] Change the check for if the dropdown is show
Simon Paquet (back in business) <bugzilla@babylonsounds.com> has cancelled Doron Rosenberg (IBM) <doronr@us.ibm.com>'s request for superreview: Bug 244761: Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected http://bugzilla.mozilla.org/show_bug.cgi?id=244761 Attachment 149378: Change the check for if the dropdown is shown to the correct way http://bugzilla.mozilla.org/attachment.cgi?id=149378&action=edit ------- Additional Comments from Simon Paquet (back in business) <bugzilla@babylonsounds.com> ...

superreview granted: [Bug 244761] Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected : [Attachment 149416] Patch for 1.7 per roc
Robert O'Callahan <roc@ocallahan.org> has granted Doron Rosenberg (IBM) <doronr@us.ibm.com>'s request for superreview: Bug 244761: Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected http://bugzilla.mozilla.org/show_bug.cgi?id=244761 Attachment 149416: Patch for 1.7 per roc - adding a if() check http://bugzilla.mozilla.org/attachment.cgi?id=149416&action=edit ------- Additional Comments from Robert O'Callahan <roc@ocallahan.org> + if (mComboBoxFrame != nsnull) { should be + if (...

superreview granted: [Bug 232722] change 'character coding' to 'character encoding' : [Attachment 140854] Addressed jshin's comments
Scott MacGregor <mscott@mozilla.org> has granted neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com>'s request for superreview: Bug 232722: change 'character coding' to 'character encoding' http://bugzilla.mozilla.org/show_bug.cgi?id=232722 Attachment 140854: Addressed jshin's comments http://bugzilla.mozilla.org/attachment.cgi?id=140854&action=edit ------- Additional Comments from Scott MacGregor <mscott@mozilla.org> sorry, I didn't notice this bug had two patches in it needing review. ...

superreview requested: [Bug 244761] Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected : [Attachment 149416] Patch for 1.7 per roc
Doron Rosenberg (IBM) <doronr@us.ibm.com> has asked Robert O'Callahan <roc@ocallahan.org> for superreview: Bug 244761: Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected http://bugzilla.mozilla.org/show_bug.cgi?id=244761 Attachment 149416: Patch for 1.7 per roc - adding a if() check http://bugzilla.mozilla.org/attachment.cgi?id=149416&action=edit ...

superreview requested: [Bug 232722] change 'character coding' to 'character encoding' : [Attachment 140854] Addressed jshin's comments
neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> has asked Scott MacGregor <mscott@mozilla.org> for superreview: Bug 232722: change 'character coding' to 'character encoding' http://bugzilla.mozilla.org/show_bug.cgi?id=232722 Attachment 140854: Addressed jshin's comments http://bugzilla.mozilla.org/attachment.cgi?id=140854&action=edit ...

superreview granted: [Bug 229737] [RFE] Favicons for XUL error pages : [Attachment 193389] address Neil's comments, fix seamonkey's check
neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> has granted Mike Connor <mconnor@steelgryphon.com>'s request for superreview: Bug 229737: [RFE] Favicons for XUL error pages https://bugzilla.mozilla.org/show_bug.cgi?id=229737 Attachment 193389: address Neil's comments, fix seamonkey's check https://bugzilla.mozilla.org/attachment.cgi?id=193389&action=edit ------- Additional Comments from neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> >- var url = getWebNavigation().currentURI.spec >+ var url = document.documentURI.spec ...

superreview cancelled: [Bug 303010] Certificate upgrade can drop S/MIME certificates. : [Attachment 191278] remove the incorrect spacing change to the 'if' in ReadSubjectEntry()
Nelson Bolyard <nelson@bolyard.com> has cancelled Robert Relyea <rrelyea@redhat.com>'s request for superreview: Bug 303010: Certificate upgrade can drop S/MIME certificates. https://bugzilla.mozilla.org/show_bug.cgi?id=303010 Attachment 191278: remove the incorrect spacing change to the 'if' in ReadSubjectEntry() https://bugzilla.mozilla.org/attachment.cgi?id=191278&action=edit ------- Additional Comments from Nelson Bolyard <nelson@bolyard.com> It appears to me that another patch (not this one) was checked in. So I am cancelling the review requ...

superreview requested: [Bug 303010] Certificate upgrade can drop S/MIME certificates. : [Attachment 191278] remove the incorrect spacing change to the 'if' in ReadSubjectEntry()
Robert Relyea <rrelyea@redhat.com> has asked Nelson Bolyard <nelson@bolyard.com> for superreview: Bug 303010: Certificate upgrade can drop S/MIME certificates. https://bugzilla.mozilla.org/show_bug.cgi?id=303010 Attachment 191278: remove the incorrect spacing change to the 'if' in ReadSubjectEntry() https://bugzilla.mozilla.org/attachment.cgi?id=191278&action=edit ...

superreview granted: [Bug 244761] Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected : [Attachment 149404] Cleanup
Robert O'Callahan <roc@ocallahan.org> has granted neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com>'s request for superreview: Bug 244761: Pressing Esc while on <select> that's not dropped down shouldn't change which option is selected http://bugzilla.mozilla.org/show_bug.cgi?id=244761 Attachment 149404: Cleanup http://bugzilla.mozilla.org/attachment.cgi?id=149404&action=edit ...

superreview granted: [Bug 263366] execute() doesn't check if exection succeeded or not, and claims it did even if that's not the case : [Attachment 162166] fix review comments
Daniel Veditz <dveditz@cruzio.com> has granted Daniel Veditz <dveditz@cruzio.com>'s request for superreview: Bug 263366: execute() doesn't check if exection succeeded or not, and claims it did even if that's not the case https://bugzilla.mozilla.org/show_bug.cgi?id=263366 Attachment 162166: fix review comments https://bugzilla.mozilla.org/attachment.cgi?id=162166&action=edit ------- Additional Comments from Daniel Veditz <dveditz@cruzio.com> Carrying over r/sr, seeking branch approval. jst wants this for the plugin finder ...

superreview granted: [Bug 330946] Can drag bookmarks into address book collection of Show All Bookmarks : [Attachment 224328] invalidates drop if it's going to occur on a smart folder's outline view
Mike Pinkerton <mikepinkerton@mac.com> has granted Chris Lawson <bugzilla@chrislawson.net>'s request for superreview: Bug 330946: Can drag bookmarks into address book collection of Show All Bookmarks https://bugzilla.mozilla.org/show_bug.cgi?id=330946 Attachment 224328: invalidates drop if it's going to occur on a smart folder's outline view https://bugzilla.mozilla.org/attachment.cgi?id=224328&action=edit ------- Additional Comments from Mike Pinkerton <mikepinkerton@mac.com> sr=pink ...

superreview cancelled: [Bug 335441] Change tiny drop-down in browser search box into a button/drop-down combo to the right of the search box : [Attachment 220886] Addressing more of Gavin's comments
Pam Greene (Google) <pamg.bugs@gmail.com> has cancelled Pam Greene (Google) <pamg.bugs@gmail.com>'s request for superreview: Bug 335441: Change tiny drop-down in browser search box into a button/drop-down combo to the right of the search box https://bugzilla.mozilla.org/show_bug.cgi?id=335441 Attachment 220886: Addressing more of Gavin's comments https://bugzilla.mozilla.org/attachment.cgi?id=220886&action=edit ------- Additional Comments from Pam Greene (Google) <pamg.bugs@gmail.com> (In reply to comment #33) > + if (textBox.value && !text...

Web resources about - superreview granted: [Bug 242760] code cleaning: rechecking checked rc's, dropping a temporary comptr, a comment change : [Attachment 147787] store rc's so ifs can check, drop temporary comptr, chang - netscape.mozilla.reviewers

Biz & Finance Magazines
Australian magazine subscriptions price comparison.

Ehsan Akhgari
Bugzilla is an essential tool to the working process of Mozilla, and many of us spend a good portion of their day in Bugzilla. The Bugzilla Tweaks ...

Resources last updated: 12/3/2015 2:38:48 PM