two returns in UpdateSummaryTotals

VERIFIED FIXED in M15

Status

MailNews Core
Backend
P3
minor
VERIFIED FIXED
19 years ago
10 years ago

People

(Reporter: Colin Blake, Assigned: Bienvenu)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
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.

Updated

19 years ago
Assignee: phil → bienvenu
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M15
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 1

19 years ago
I checked this in with some other unrelated fixes - I just return rv.

Updated

18 years ago
QA Contact: lchiang → suresh

Comment 2

18 years ago
Suresh, can you help to verify? Thanks.

Updated

18 years ago
Status: RESOLVED → VERIFIED

Comment 3

18 years ago
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.