Closed
Bug 159
Opened 27 years ago
Closed 24 years ago
warning fix : ns/js/jsj/jsStubs.c
Categories
(Core :: JavaScript Engine, defect, P5)
Core
JavaScript Engine
Tracking
()
VERIFIED
FIXED
People
(Reporter: rick, Unassigned)
Details
Created by Rick Osborne (rick@rixsoft.com) on Sunday, April 12, 1998 6:23:11 PM PDT
Additional Details :
(In the continuing quest to eliminate even compiler warnings
... )
We''re back to compiler directives mucking with var scope.
The fun.
--- jsstubs.old.c Wed Apr 08 17:52:13 1998
+++ jsstubs.c Sun Apr 12 03:16:21 1998
@@ -304,3 +304,5 @@
JSSavedState saved;
+#ifdef JAVA /* only referenced if defined */
const char *cstr;
+#endif
@@ -324,3 +326,5 @@
+#ifdef JAVA /* only referenced if defined */
do_exit:
+#endif
exitJS(env, self, cx, jso, &saved);
@@ -496,4 +500,6 @@
JSSavedState saved;
+#ifdef JAVA /* only referenced if defined */
JSString *jsstr;
char *cstr;
+#endif
struct java_lang_String *ret;
Updated by Mike McCabe (mccabe@netscape.com) on Wednesday, April 29, 1998 8:42:20 AM PDT
Additional Details :
I poked in the changes... they followed the same pattern as many existing
#ifdefs in the file.
Updated•26 years ago
|
Component: JavaScript
Changing component to "Javascript Engine". "Javascript" component is being
retired.
Comment 2•24 years ago
|
||
not sure what the resolution on this was, just verifying...
Status: RESOLVED → VERIFIED
Comment 3•24 years ago
|
||
mid-air collision ? / bugzilla cleanup
Reopening (current State: verified and no resolution)
Status: VERIFIED → REOPENED
Comment 4•24 years ago
|
||
fixed
Status: REOPENED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•