PatchSet 4364 
Date: 2004/01/23 17:17:24
Author: dalibor
Branch: HEAD
Tag: (none) 
Log:
Fixed a few more gcc 3.3.2 -W warnings

2004-01-23  Dalibor Topic <[EMAIL PROTECTED]>

        * kaffe/kaffevm/jit3/labels.c:
        (linkLabels) Removed unused code. Fixed gcc 3.3.2
        warning.

        * kaffe/kaffevm/jit3/machine.c
        (codeblock_size): Made unsigned to fix gcc warning.
        Changed scope to static since it's not used outside
        of machine.c.
        (checkCaughtExceptions): Changed type of i to
        unsigned int to fix compiler warning. Changed type
        of parameter pc to unit32 to match declaration of
        pc to fix compiler warnings.

Members: 
        ChangeLog:1.1949->1.1950 
        kaffe/kaffevm/jit3/labels.c:1.12->1.13 
        kaffe/kaffevm/jit3/machine.c:1.46->1.47 

Index: kaffe/ChangeLog
diff -u kaffe/ChangeLog:1.1949 kaffe/ChangeLog:1.1950
--- kaffe/ChangeLog:1.1949      Thu Jan 22 22:35:14 2004
+++ kaffe/ChangeLog     Fri Jan 23 17:17:24 2004
@@ -1,3 +1,18 @@
+2004-01-23  Dalibor Topic <[EMAIL PROTECTED]>
+
+       * kaffe/kaffevm/jit3/labels.c:
+       (linkLabels) Removed unused code. Fixed gcc 3.3.2 
+       warning.
+
+       * kaffe/kaffevm/jit3/machine.c
+       (codeblock_size): Made unsigned to fix gcc warning.
+       Changed scope to static since it's not used outside
+       of machine.c.
+       (checkCaughtExceptions): Changed type of i to 
+       unsigned int to fix compiler warning. Changed type
+       of parameter pc to unit32 to match declaration of
+       pc to fix compiler warnings.
+       
 2004-01-22  Christophe Dubach <[EMAIL PROTECTED]>
 
        * kaffe/kaffevm/jni.c
Index: kaffe/kaffe/kaffevm/jit3/labels.c
diff -u kaffe/kaffe/kaffevm/jit3/labels.c:1.12 kaffe/kaffe/kaffevm/jit3/labels.c:1.13
--- kaffe/kaffe/kaffevm/jit3/labels.c:1.12      Sun Sep 21 18:18:19 2003
+++ kaffe/kaffe/kaffevm/jit3/labels.c   Fri Jan 23 17:17:27 2004
@@ -4,6 +4,9 @@
  * Copyright (c) 1996, 1997
  *     Transvirtual Technologies, Inc.  All rights reserved.
  *
+ * Copyright (c) 2004
+ *     Kaffe.org contributors. See ChangeLog for details.  All rights reserved.
+ *
  * See the file "license.terms" for information on usage and redistribution 
  * of this file. 
  */
@@ -161,7 +164,7 @@
                        dest = 0;
                        break;
                default:
-                       goto unhandled;
+                       return;
                }
 
                /*
@@ -178,7 +181,7 @@
                        dest -= codebase;
                        break;
                default:
-                       goto unhandled;
+                       return;
                }
 
                /* Get the insertion point. */
@@ -209,28 +212,7 @@
                /* Machine specific labels go in this magic macro */
                        EXTRA_LABELS(place, dest, l);
 
-               unhandled:
-#if 0
-               default:
-#if defined(KAFFE_VMDEBUG)
-                       kprintf("Label type 0x%x not supported (%p).\n", l->type & 
Ltypemask, l);
-#endif
-                       ABORT();
-#endif
                }
-#if 0
-               /*
-                * If we were saving relocation information we must save all
-                * labels which are 'Labsolute', that is they hold an absolute
-                * address for something.  Note that this doesn't catch
-                * everything, specifically it doesn't catch string objects
-                * or references to classes.
-                */
-               if ((l->type & Labsolute) != 0) {
-                       l->snext = savedLabel;
-                       savedLabel = l;
-               }
-#endif
        }
 }
 
Index: kaffe/kaffe/kaffevm/jit3/machine.c
diff -u kaffe/kaffe/kaffevm/jit3/machine.c:1.46 kaffe/kaffe/kaffevm/jit3/machine.c:1.47
--- kaffe/kaffe/kaffevm/jit3/machine.c:1.46     Thu Jan 15 02:29:33 2004
+++ kaffe/kaffe/kaffevm/jit3/machine.c  Fri Jan 23 17:17:27 2004
@@ -94,7 +94,7 @@
 /* Codeblock redzone - allows for safe overrun when generating instructions */
 #define        CODEBLOCKREDZONE        256
 
-int codeblock_size;
+static uint codeblock_size;
 static int code_generated;
 static int bytecode_processed;
 static int codeperbytecode;
@@ -113,7 +113,7 @@
  * @param meth The method that may contain an exception handler.
  * @param pc The location within the method to look for a handler.
  */
-static void checkCaughtExceptions(Method* meth, int pc);
+static void checkCaughtExceptions(Method* meth, uint32 pc);
 
 static void initFakeCalls(void);
 static void makeFakeCalls(void);
@@ -820,9 +820,9 @@
  */
 static
 void 
-checkCaughtExceptions(Method* meth, int pc)
+checkCaughtExceptions(Method* meth, uint32 pc)
 {
-       int i;
+       unsigned int i;
 
        willcatch.ANY = false;
        willcatch.BADARRAYINDEX = false;

_______________________________________________
kaffe mailing list
[EMAIL PROTECTED]
http://kaffe.org/cgi-bin/mailman/listinfo/kaffe

Reply via email to