PatchSet 4470 
Date: 2004/02/28 14:39:13
Author: dalibor
Branch: HEAD
Tag: (none) 
Log:
Replaces NEXTPC macro by inline function

2004-02-28  Dalibor Topic <[EMAIL PROTECTED]>

        * kaffe/kaffevm/verify.c
        (getNextPC): New static inline function.
        (NEXTPC) Removed.
        Replaced use of NEXTPC by getNextPC.

Members: 
        ChangeLog:1.2050->1.2051 
        kaffe/kaffevm/verify.c:1.62->1.63 

Index: kaffe/ChangeLog
diff -u kaffe/ChangeLog:1.2050 kaffe/ChangeLog:1.2051
--- kaffe/ChangeLog:1.2050      Sat Feb 28 13:15:27 2004
+++ kaffe/ChangeLog     Sat Feb 28 14:39:13 2004
@@ -1,3 +1,10 @@
+2004-02-28  Dalibor Topic <[EMAIL PROTECTED]>
+
+        * kaffe/kaffevm/verify.c
+        (getNextPC): New static inline function.
+        (NEXTPC) Removed.
+        Replaced use of NEXTPC by getNextPC.
+
 2004-02-28    Guilhem Lavaux <[EMAIL PROTECTED]>
 
        * java/text/AttributedStringIterator.java
Index: kaffe/kaffe/kaffevm/verify.c
diff -u kaffe/kaffe/kaffevm/verify.c:1.62 kaffe/kaffe/kaffevm/verify.c:1.63
--- kaffe/kaffe/kaffevm/verify.c:1.62   Fri Feb 27 18:12:53 2004
+++ kaffe/kaffe/kaffevm/verify.c        Sat Feb 28 14:39:16 2004
@@ -1402,6 +1402,16 @@
 }
 
 /*
+ * Helper function for pc access.
+ */
+static inline
+uint32
+getNextPC(const unsigned char * code, const uint32 pc)
+{
+       return (pc + insnLen[code[pc]]);
+}
+
+/*
  * verifyMethod3a()
  *     check static constraints.  section 4.8.1 of JVML Spec 2.
  *
@@ -1458,7 +1468,6 @@
        unsigned char* code = METHOD_BYTECODE_CODE(method);
        
        uint32 pc = 0, newpc = 0, n = 0, idx = 0;
-#define NEXTPC (pc + insnLen[code[pc]])
        int32 branchoffset = 0;
        int32 low, high;
        
@@ -1798,7 +1807,7 @@
                        ENSURE_NON_WIDE;
                        status[pc] |= END_BLOCK;
                        
-                       newpc = NEXTPC;
+                       newpc = getNextPC(code, pc);
                        BRANCH_IN_BOUNDS(newpc, "if<condition> = false");
                        status[newpc] |= START_BLOCK;
                        
@@ -1826,7 +1835,7 @@
                        status[newpc] |= START_BLOCK;
                        
                        /* the next instruction is a target for branching via RET */
-                       pc = NEXTPC;
+                       pc = getNextPC(code, pc);
                        BRANCH_IN_BOUNDS(pc, "jsr/ret");
                        status[pc] |= START_BLOCK;
                        continue;
@@ -1845,7 +1854,7 @@
                                pc += 2;
                        }
                        CHECK_LOCAL_INDEX(idx);
-                       pc = NEXTPC;
+                       pc = getNextPC(code, pc);
                        continue;
                        
                        
@@ -1958,7 +1967,7 @@
                        }
                }
                
-               pc = NEXTPC;
+               pc = getNextPC(code, pc);
        }
        
        
@@ -2066,7 +2075,6 @@
        
        
 #undef CHECK_LOCAL_INDEX       
-#undef NEXTPC
 #undef BRANCH_IN_BOUNDS
 #undef GET_IDX
 #undef GET_WIDX

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

Reply via email to