Closed
Bug 460005
Opened 16 years ago
Closed 16 years ago
QuerryInterface in nsMailDefaultHandler.js isn't correct
Categories
(Thunderbird :: General, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 459784
People
(Reporter: eagle.lu, Unassigned)
Details
Attachments
(1 file)
|
795 bytes,
patch
|
Details | Diff | Splinter Review |
There are following codes in nsMailDefaultHandler.js:
394 var Module = {
395 /* nsISupports */
396
397 QueryInterface : function QI(iid) {
398 if (iid.equals(Components.interfaces.nsIModule) &&
399 iid.equals(Components.interfaces.nsISupports))
400 return this;
401
402 throw Components.results.NS_ERROR_NO_INTERFACE;
403 },
We should use "||" instead of "&&" here.
Attachment #343194 -
Flags: review?(bugzilla)
|
||
Comment 2•16 years ago
|
||
Comment on attachment 343194 [details] [diff] [review]
patch
Whilst this does fix the issue, I prefer my patch in bug 459784 which does a bit more tidy up as well.
Attachment #343194 -
Flags: review?(bugzilla)
|
|
||
Updated•16 years ago
|
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•