change mime to use NS_ASSERTIONs instead of PR_ASSERTs

VERIFIED FIXED

Status

MailNews Core
MIME
--
enhancement
VERIFIED FIXED
16 years ago
10 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
I know seth doesn't want to fight this holy war, but I have the patch, i've run 
with it for ages, it doesn't harm any flies and it gives more information in 
addition to allowing people to survive the assert (PR_ASSERTs are fatal, 
NS_ASSERTIONs are optionally fatal).
(Assignee)

Comment 1

16 years ago
Created attachment 89980 [details] [diff] [review]
change mime PR_ASSERT(x) to NS_ASSERTION(x, "rev stamp author")

Updated

16 years ago
QA Contact: gayatri → stephend
Keywords: patch, review
OS: Windows 2000 → All
Hardware: PC → All
Comment on attachment 89980 [details] [diff] [review]
change mime PR_ASSERT(x) to NS_ASSERTION(x, "rev stamp author")

R=ducarroz. We should one day change the default error string to something with
a better meaning.
Attachment #89980 - Flags: review+
Comment on attachment 89980 [details] [diff] [review]
change mime PR_ASSERT(x) to NS_ASSERTION(x, "rev stamp author")

sr=jst
Attachment #89980 - Flags: superreview+
(Assignee)

Comment 4

16 years ago
fixed and filed 155759
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Verified fixed using LXR (as well as updating my source tree).
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.