Closed Bug 3537 Opened 26 years ago Closed 25 years ago

NSPR shouldn't call Delay()

Categories

(NSPR :: NSPR, defect, P3)

Product:
NSPR
Component:
NSPR
Platform:
PowerPC
Mac System 8.5
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: gordon, Assigned: sdagley)

Details

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.
Assignee: wtc → gordon
Reassigned the bug to gordon.
Status: NEW → ASSIGNED
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.
Target Milestone: M6
Changed target milestone to M6.
NSPR now has its own Bugzilla product.  Moving this bug to the NSPR product.
Target Milestone: M6 → M9
Assignee: gordon → sdagley
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
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.
Status: RESOLVED → VERIFIED
good enough for me. marking verified (code level fix).
Target Milestone: M9 → ---
You need to log in before you can comment on or make changes to this bug.