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

Status: NEW

(MailNews Core :: Backend, defect)

Description

[Serge Gautherie (:sgautherie)]

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

[Kent James (:rkent)]

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?

Comment 2

[Serge Gautherie (:sgautherie)]

(In reply to comment #1)

I could, but the main point is to first figure out what this DEFINE is really worth...