[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mozilla-patches
Subject:    warning fix : ns/lib/layout/layscrip.c
From:       Rick Osborne <rick () spacey ! net>
Date:       1998-04-14 4:08:58
[Download RAW message or body]

(In the continuing quest to eliminate even compiler warnings ... )

Bugzilla #202.

Just a couple of #ifdef statements mucking with variable scope and label
usage.

(538) : warning C4102: 'out' : unreferenced label
(538) : warning C4101: 'principals' : unreferenced local variable
(538) : warning C4101: 'name' : unreferenced local variable

--- layscrip.old.c      Wed Apr 08 17:52:27 1998
+++ layscrip.c  Tue Apr 14 00:03:13 1998
@@ -452,4 +452,6 @@ lo_script_archive_exit_fn(URL_Struct *ur
     ScriptData *data = NULL;
+#ifdef JAVA  /* Vars only used in JAVA context */
     char *name;
     JSPrincipals *principals;
+#endif
     ETEvalStuff * stuff;
@@ -534,3 +536,5 @@ lo_script_archive_exit_fn(URL_Struct *ur

+#ifdef JAVA  /* Label only used in this context */
 out:
+#endif
     /* Always free (or drop a ref on) the url_struct before returning. */

<!-- Rick Osborne -- http://www.rixsoft.com/resume.pdf -->
The US is naked, absolutely nude, to attack by the Soviets. -Senator Dan
Quayle 8/20/88

[prev in list] [next in list] [prev in thread] [next in thread]