Closed Bug 152 Opened 26 years ago Closed

warning fix : ns/nsprpub/pr/src/threads/prcthr.c

Categories

(NSPR :: NSPR, defect, P3)

Product:
NSPR
Component:
NSPR
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: rick, Assigned: wtc)

Details

Created by Rick Osborne (rick@rixsoft.com) on Saturday, April 11, 1998 10:52:00 PM PDT
Additional Details :
(In the continuing quest to eliminate even compiler warnings
... )

Ever more compiler directives mucking with variable scopes.

--- prcthr.c    Sun Apr 12 01:54:07 1998
+++ prcthr.old.c        Wed Apr 08 17:55:09 1998
@@ -94,5 +94,3 @@ PR_IMPLEMENT(PRStatus) PR_Sleep(PRInterv
         */
-#ifndef _PR_GLOBAL_THREADS_ONLY  /* vars not used in this
context */
         PRIntn is;
-#endif
         PRThread *me = PR_GetCurrentThread();
@@ -242,5 +240,3 @@ PR_IMPLEMENT(void) PR_ClearInterrupt()
 {
-#ifndef _PR_GLOBAL_THREADS_ONLY  /* vars not used in this
context */
     PRIntn is;
-#endif
     PRThread *me = _PR_MD_CURRENT_THREAD();
Updated by Wan-Teh Chang (wtc@netscape.com) on Friday, April 24, 1998 12:35:57 PM PDT
Additional Details :
Assigned bug to myself.
per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
Wan-Teh, these are some pretty old bugs. i can't help but assume they have
been fixed by now. If you don't have any objections, i will close out and
verify these bugs in a couple of days: 146, 148, 149, 150, 151, 152, 154
please take corrective action if this is not doing the Right Thing, but,
as it has been a while, i'm marking these fixed and will immediately verify:
bugs 146, 148, 149, 150, 151, 152, 154
Status: RESOLVED → VERIFIED
verified.
NSPR now has its own Bugzilla product.  Moving this bug to the NSPR product.
You need to log in before you can comment on or make changes to this bug.