Closed
Bug 85250
Opened 24 years ago
Closed 23 years ago
Incorrect usage of PR_CWait
Categories
(MailNews Core :: Composition, defect)
MailNews Core
Composition
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.6
People
(Reporter: bryner, Assigned: Bienvenu)
Details
(Whiteboard: have fix)
Attachments
(1 file, 1 obsolete file)
|
646 bytes,
patch
|
bugzilla
:
review+
mscott
:
superreview+
|
Details | Diff | Splinter Review |
In nsMsgCopy::DoCopy, we do:
PR_CWait(aCopyListener, 1000UL);
We should be using a PRIntervalTime here, and the only portable way to get a
PRIntervalTime is to use PR_SecondsToInterval().
|
Reporter | |
Comment 1•24 years ago
|
||
|
||
Comment 2•24 years ago
|
||
R=ducarroz
|
|
||
Updated•23 years ago
|
|
|
||
Updated•23 years ago
|
Attachment #37929 -
Flags: review+
|
Assignee | |
Comment 3•23 years ago
|
||
I don't think we want to wait a whole second...I have a different fix in my tree
|
|
Assignee | |
Comment 4•23 years ago
|
||
|
|
||
Updated•23 years ago
|
Attachment #37929 -
Attachment is obsolete: true
|
|
||
Comment 5•23 years ago
|
||
Comment on attachment 52949 [details] [diff] [review]
proposed fix
R=ducarroz
Attachment #52949 -
Flags: review+
|
||
Comment 6•23 years ago
|
||
Comment on attachment 52949 [details] [diff] [review]
proposed fix
sr=mscott
Attachment #52949 -
Flags: superreview+
|
|
Assignee | |
Comment 8•23 years ago
|
||
fix checked in.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
|
||
Updated•23 years ago
|
QA Contact: sheelar → stephend
Verified FIXED using LXR.
Status: RESOLVED → VERIFIED
|
||
Updated•20 years ago
|
Product: MailNews → Core
|
||
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•