NSPR shouldn't call Delay()

VERIFIED FIXED

Status

P3
normal
VERIFIED FIXED
20 years ago
19 years ago

People

(Reporter: gordon, Assigned: sdagley)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

20 years ago
Chris Saari believes that NSPR needs to call WaitNextEvent() instead of Delay().

Additional code may be necessary to prevent certain events from getting lost,
such as AppleEvents.  Also, this might not be desired for all threads.

Updated

20 years ago
Assignee: wtc → gordon

Comment 1

20 years ago
Reassigned the bug to gordon.
(Reporter)

Updated

20 years ago
Status: NEW → ASSIGNED

Comment 2

20 years ago
You won't loose events if you give WaitNextEvent a null event mask. In other
words, you're not really retrieving the next event, you're yeilding time to the
system which Delay() doesn't do.

Delay() is evil. It is a spin loop on a non-preemptive system. Nothing can happen
during a Delay() call.

Updated

20 years ago
Target Milestone: M6

Comment 3

20 years ago
Changed target milestone to M6.

Comment 4

20 years ago
NSPR now has its own Bugzilla product.  Moving this bug to the NSPR product.
(Reporter)

Updated

20 years ago
Target Milestone: M6 → M9
(Assignee)

Updated

20 years ago
Assignee: gordon → sdagley
Status: ASSIGNED → NEW
(Assignee)

Updated

20 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

20 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 20 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

20 years ago
Fix is checked in.

Note to QA - you can't really test this other than looking at the source for /
mozilla/nsprpub/ps/src/md/mac/macio.c.

Updated

20 years ago
Status: RESOLVED → VERIFIED

Comment 6

20 years ago
good enough for me. marking verified (code level fix).

Updated

19 years ago
Target Milestone: M9 → ---
You need to log in before you can comment on or make changes to this bug.