235481 - Failure to empty trash

Status: VERIFIED FIXED

(MailNews Core :: Database, defect)

Description

[Bob Clary [:bc] (inactive)]

In cvs builds from today, I get

###!!! ASSERTION: no set!  : '0', file
c:/work/mozilla_source/trunk/mozilla/mailnews/db/msgdb/src/nsMsgDatabase.cpp,
line 2315 

when emptying the trash.

Related to
<http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=nsMsgDatabase.cpp&branch=&root=/cvsroot&subdir=mozilla/mailnews/db/msgdb/src&command=DIFF_FRAMESET&rev1=1.329&rev2=1.330>
?

Comment 1

[Bob Clary [:bc] (inactive)]

I use dreamhost for email which uses Courier IMAP I believe. The symptom is
clicking on Empty Trash will cause the messages in trash to disappear however
they reappear whenever you revisit the Trash folder. I can manually delete the
messages from trash however. This did not occur in the last good installable
nightly I have from 2004-02-17 and only began on my CVS builds after building on
2004-02-24 about midday. I still see this behavior with a more recent CVS build
but do not see the ASSERT.

Comment 2

[David :Bienvenu]

Attachment: proposed fix

bad braces - empty trash only worked if you'd deleted folders as well.

Comment 3

[Scott MacGregor]

Comment on attachment 142242 [details] [diff] [review]
proposed fix

heh....

Comment 4

[David :Bienvenu]

fixed

Comment 5

[Bob Clary [:bc] (inactive)]

ok. i built with this and it works ok. I seem to occasionally get the assert no
set and when i try to view trash it shows busy and does nothing, but closing
mailnews and reopening allowed me to work ok. whatever that issue is, this bug
is fixed.

v

Comment 6

[David :Bienvenu]

yes, that assertion had nothing to do with this, not did the bonsai checkin.