Very innefficient PD_MD_OPEN

VERIFIED FIXED in 4.2

Status

NSPR
NSPR
P1
normal
VERIFIED FIXED
17 years ago
16 years ago

People

(Reporter: mkaply, Assigned: Wan-Teh Chang)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
Currently in PR_MD_OPEN, we increase the number of system file handles by one 
before each DosOpen. This is very expensive and unnecessary.

I have changes the code so that now we only increase the number of file handles 
if we get a return code from DosOpen that says we have too many files open.

Also, we increase it by 20 at a time.
(Reporter)

Comment 1

17 years ago
Created attachment 60699 [details] [diff] [review]
New PR_MD_OPEN

Rewrite of PR_MD_OPEN to be more efficient
Well, get rid of the "/* endif */" comment, and you have r = pedemont
(Assignee)

Comment 3

17 years ago
I checked in the patch (with the /* endif */ comment removed)
into the tip and the client branch of NSPR.

(I changed the patch to indent by four spaces as opposed to
two spaces.)
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Priority: -- → P1
Resolution: --- → FIXED
Target Milestone: --- → 4.2
(Reporter)

Comment 4

16 years ago
I have verified visually that this code is in.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.