/mailnews/local/src/nsLocalMailFolder.*: (get rid or) fix DOWNLOAD_NOTIFY_STYLE code

NEW
Unassigned

Status

MailNews Core
Backend
--
minor
9 years ago
9 years ago

People

(Reporter: sgautherie, Unassigned)

Tracking

({regression})

Trunk
regression
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

9 years ago
http://mxr.mozilla.org/seamonkey/search?string=DOWNLOAD_NOTIFY&case=on&find=%2Fmailnews%2Flocal%2Fsrc%2FnsLocalMailFolder%5C.
DOWNLOAD_NOTIFY_STYLE was added in bug 185184 and remained unchanged since then.

http://hg.mozilla.org/comm-central/rev/a8e7c2927faf#l1.26
"Recently", bug 492665 added code which can't compile because of an extraneous '}'.

This code needs to be fixed, or maybe just drop these DEFINES?

Comment 1

9 years ago
oops, sorry about that. I know nothing of that DEFINE, but it would of course be trivial to just fix the coding error.

Can you do that, Serge?
(Reporter)

Comment 2

9 years ago
(In reply to comment #1)

I could, but the main point is to first figure out what this DEFINE is really worth...
You need to log in before you can comment on or make changes to this bug.