Closed Bug 1155955 Opened 6 years ago Closed 6 years ago

remove a self assignment in nsImapUtils.cpp

Categories

(MailNews Core :: Import, defect)

defect
Not set
normal

Tracking

(firefox40 affected)

RESOLVED FIXED
Tracking Status
firefox40 --- affected

People

(Reporter: Sylvestre, Assigned: Sylvestre)

Details

(Keywords: coverity)

Attachments

(2 files)

Attached patch self-assign.diffSplinter Review
Found by coverity (CID 1137445)
The code is useless
Attachment #8594310 - Flags: review?(Pidgeot18)
This patch removes the trailing spaces. Let me know if you want a separate bug for this.
Attachment #8594311 - Flags: review?(Pidgeot18)
Attachment #8594310 - Flags: review?(Pidgeot18) → review+
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+
https://hg.mozilla.org/comm-central/rev/3eb933502ced
https://hg.mozilla.org/comm-central/rev/dbef5e81cd11
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.