Closed Bug 14741 Opened 25 years ago Closed 25 years ago

two returns in UpdateSummaryTotals

Categories

(MailNews Core :: Backend, defect, P3)

Product:
MailNews Core
Component:
Backend
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: colin, Assigned: Bienvenu)

Details

In mailnews/imap/src/nsImapMailFolder.cpp, the end of
nsImapMailFolder::UpdateSummaryTotals has two return
statements:

        return NS_OK;
    return rv;

The second one should be removed (and rv can be removed since its
never used).

Colin.
Assignee: phil → bienvenu
Status: NEW → ASSIGNED
Target Milestone: M15
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
I checked this in with some other unrelated fixes - I just return rv.
QA Contact: lchiang → suresh
Suresh, can you help to verify? Thanks.
Status: RESOLVED → VERIFIED
Verified this with a recent (11/23/99) debug build. It just has one return
statement 'return rv'.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.