Closed
Bug 665023
Opened 13 years ago
Closed 12 years ago
XPITriggerEvent::Run (nsXPITriggerInfo.cpp) should use NS_OK, not 0
Categories
(Core Graveyard :: Installer: XPInstall Engine, defect)
Core Graveyard
Installer: XPInstall Engine
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mounir, Assigned: mounir)
References
Details
(Whiteboard: [fixed by bug 652376])
Attachments
(1 file)
2.78 KB,
patch
|
Details | Diff | Splinter Review |
Basically, the patch is doing s/return 0;/return NS_OK;/. Looks like the method returns 0 even when failing but I'm not going to fix that in this patch (if it has to be fixed...).
Attachment #540073 -
Flags: review?
Assignee | ||
Updated•13 years ago
|
Attachment #540073 -
Flags: review? → review?(dveditz)
Comment 1•13 years ago
|
||
Comment on attachment 540073 [details] [diff] [review] Trivial patch This code isn't built and is due to be removed, no point wasting time on it
Attachment #540073 -
Flags: review?(dveditz) → review-
Assignee | ||
Comment 2•13 years ago
|
||
Is there a bug tracking the removal of that code?
Whiteboard: [needs review]
Assignee | ||
Updated•13 years ago
|
Attachment #540073 -
Flags: review-
Comment 3•13 years ago
|
||
(In reply to comment #2) > Is there a bug tracking the removal of that code? Bug 652376
Comment 4•12 years ago
|
||
The dependent bug has now been fixed and the underlying code is gone. We should close this bug.
Assignee | ||
Comment 5•12 years ago
|
||
Indeed. Thanks for the reminder :)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed by bug 652376]
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•