Closed Bug 72261 Opened 19 years ago Closed 19 years ago

Missing NSPR_API in primpl.h

Categories

(NSPR :: NSPR, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mkaply, Assigned: larryh)

References

Details

Attachments

(2 files)

A number of the functions in primpl.h are missing NSPR_API.

I'll attach a diff.
Sorry for the spam, attaching to the meta bug.
Blocks: 72050
I got this totally wrong.

The problem is that OS/2 put PR_IMPLEMENTs in its files when it shouldn't have.

These are not supposed to be exported functions.

I've attached a diff for this.

There are other platforms that have this problem as well.

The only PR_IMPLEMENT that should be in md/platform directories is PR_Now since 
it is implemented in the actual platform directory.

I'm looking to get this OS/2 only change checked in. Thanks.
Checked in parts on NSPRPUB_CLIENT_BRANCH

cvs commit -m"Bugzilla: 72261. fix linking problem" pr/src/md/os2/os2cv.c
Checking in pr/src/md/os2/os2cv.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2cv.c,v  <--  os2cv.c
new revision: 3.3.72.1; previous revision: 3.3
done
cvs commit -m"Bugzilla: 72261. fix linking problem" pr/src/md/os2/os2inrval.c
Checking in pr/src/md/os2/os2inrval.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2inrval.c,v  <--  os2inrval.c
new revision: 3.3.72.1; previous revision: 3.3
done
cvs commit -m"Bugzilla: 72261. fix linking problem" pr/src/md/os2/os2misc.c
Checking in pr/src/md/os2/os2misc.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2misc.c,v  <--  os2misc.c
new revision: 3.6.18.1; previous revision: 3.6
done
cvs commit -m"Bugzilla: 72261. fix linking problem" pr/src/md/os2/os2sem.c
Checking in pr/src/md/os2/os2sem.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2sem.c,v  <--  os2sem.c
new revision: 3.2.72.1; previous revision: 3.2
done
cvs commit -m"Bugzilla: 72261. fix linking problem" pr/src/md/os2/os2thred.c
Checking in pr/src/md/os2/os2thred.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2thred.c,v  <--  os2thred.c
new revision: 3.6.18.1; previous revision: 3.6
done
Status: NEW → ASSIGNED
Refit patch and applied to the tip of tree.

cvs commit -m"bugzilla: 72261. fix linking problems" pr/src/md/os2/os2cv.c
Checking in pr/src/md/os2/os2cv.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2cv.c,v  <--  os2cv.c
new revision: 3.6; previous revision: 3.5
done
cvs commit -m"bugzilla: 72261. fix linking problems" pr/src/md/os2/os2inrval.c
Checking in pr/src/md/os2/os2inrval.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2inrval.c,v  <--  os2inrval.c
new revision: 3.6; previous revision: 3.5
done
cvs commit -m"bugzilla: 72261. fix linking problems" pr/src/md/os2/os2misc.c
Checking in pr/src/md/os2/os2misc.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2misc.c,v  <--  os2misc.c
new revision: 3.10; previous revision: 3.9
done
cvs commit -m"bugzilla: 72261. fix linking problems" pr/src/md/os2/os2sem.c
Checking in pr/src/md/os2/os2sem.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2sem.c,v  <--  os2sem.c
new revision: 3.5; previous revision: 3.4
done
cvs commit -m"bugzilla: 72261. fix linking problems" pr/src/md/os2/os2thred.c
Checking in pr/src/md/os2/os2thred.c;
/cvsroot/mozilla/nsprpub/pr/src/md/os2/os2thred.c,v  <--  os2thred.c
new revision: 3.9; previous revision: 3.8
done
Mike,
Please checkout or merge the changes and build on both NSPRPUB_CLIENT_BRANCH and 
the tip. Make sure I got the patches applied OK.
marking fixed, absent objections.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Code change - this is fixed
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.