Closed
Bug 1155955
Opened 10 years ago
Closed 10 years ago
remove a self assignment in nsImapUtils.cpp
Categories
(MailNews Core :: Import, defect)
MailNews Core
Import
Tracking
(firefox40 affected)
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| firefox40 | --- | affected |
People
(Reporter: Sylvestre, Assigned: Sylvestre)
Details
(Keywords: coverity)
Attachments
(2 files)
|
947 bytes,
patch
|
jcranmer
:
review+
|
Details | Diff | Splinter Review |
|
6.11 KB,
patch
|
jcranmer
:
review+
|
Details | Diff | Splinter Review |
Found by coverity (CID 1137445)
The code is useless
Attachment #8594310 -
Flags: review?(Pidgeot18)
|
Assignee | |
Comment 1•10 years ago
|
||
This patch removes the trailing spaces. Let me know if you want a separate bug for this.
Attachment #8594311 -
Flags: review?(Pidgeot18)
|
||
Updated•10 years ago
|
Attachment #8594310 -
Flags: review?(Pidgeot18) → review+
|
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8594311 [details] [diff] [review]
remove-trailing-spaces.diff
Review of attachment 8594311 [details] [diff] [review]:
-----------------------------------------------------------------
(Sigh, we probably need a fix-whitespace mass-change somewhere along the line. Too many coding standards!)
Attachment #8594311 -
Flags: review?(Pidgeot18) → review+
|
|
Assignee | |
Comment 3•10 years ago
|
||
https://hg.mozilla.org/comm-central/rev/3eb933502ced
https://hg.mozilla.org/comm-central/rev/dbef5e81cd11
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.
Description
•