* builtin_multianewarray: Handle MIPS32 correctly (s4 in an s8 slot)
[cacao.git] / src / vm / builtin.c
index 9e543016a3cf79ac17be7702cd0ae92c74cf41d9..f0376467d0734df6a2626d73760de3f557b65038 100644 (file)
@@ -1,4 +1,4 @@
-/* vm/builtin.c - functions for unsupported operations
+/* src/vm/builtin.c - functions for unsupported operations
 
    Copyright (C) 1996-2005 R. Grafl, A. Krall, C. Kruegel, C. Oates,
    R. Obermaisser, M. Platter, M. Probst, S. Ring, E. Steiner,
             Andreas Krall
             Mark Probst
 
+   Changes: Christian Thalinger
+
    Contains C functions for JavaVM Instructions that cannot be
    translated to machine language directly. Consequently, the
    generated machine code for these instructions contains function
    calls instead of machine instructions, using the C calling
    convention.
 
-   $Id: builtin.c 1774 2004-12-20 20:16:57Z jowenn $
+   $Id: builtin.c 3134 2005-08-23 14:45:29Z cacao $
 
 */
 
 
 #include <assert.h>
+#include <stdlib.h>
 #include <string.h>
-#include <math.h>
+
+#include "config.h"
+#include "arch.h"
+#include "md-abi.h"
+#include "types.h"
+
+#include "classpath/native/fdlibm/fdlibm.h"
 
 #include "mm/boehm.h"
 #include "mm/memory.h"
 #include "native/native.h"
 #include "native/include/java_lang_Cloneable.h"
+#include "native/include/java_lang_Object.h"          /* required by VMObject */
 #include "native/include/java_lang_VMObject.h"
 
 #if defined(USE_THREADS)
 #endif
 
 #include "toolbox/logging.h"
+#include "toolbox/util.h"
 #include "vm/builtin.h"
 #include "vm/exceptions.h"
 #include "vm/global.h"
+#include "vm/initialize.h"
 #include "vm/loader.h"
 #include "vm/options.h"
+#include "vm/stringlocal.h"
 #include "vm/tables.h"
 #include "vm/jit/asmpart.h"
+#include "vm/jit/patcher.h"
 
 
 #undef DEBUG /*define DEBUG 1*/
 
 THREADSPECIFIC methodinfo* _threadrootmethod = NULL;
-THREADSPECIFIC void *_thread_nativestackframeinfo=NULL;
+THREADSPECIFIC void *_thread_nativestackframeinfo = NULL;
+
+
+/* include builtin tables *****************************************************/
+
+#include "vm/builtintable.inc"
+
+
+/* builtintable_init ***********************************************************
+
+   Parse the descriptors of builtin functions and create the parsed
+   descriptors.
+
+*******************************************************************************/
+
+static bool builtintable_init(void)
+{
+       descriptor_pool *descpool;
+       s4               dumpsize;
+       utf             *descriptor;
+       s4               entries_internal;
+       s4               entries_automatic;
+       s4               i;
+
+       /* mark start of dump memory area */
+
+       dumpsize = dump_size();
+
+       /* create a new descriptor pool */
+
+       descpool = descriptor_pool_new(class_java_lang_Object);
+
+       /* add some entries we need */
+
+       if (!descriptor_pool_add_class(descpool, utf_java_lang_Object))
+               return false;
+
+       if (!descriptor_pool_add_class(descpool, utf_java_lang_Class))
+               return false;
+
+       /* calculate table entries statically */
+
+       entries_internal =
+               sizeof(builtintable_internal) / sizeof(builtintable_entry);
+
+       entries_automatic =
+               sizeof(builtintable_automatic) / sizeof(builtintable_entry)
+               - 1; /* last filler entry (comment see builtintable.inc) */
+
+       /* first add all descriptors to the pool */
+
+       for (i = 0; i < entries_internal; i++) {
+               /* create a utf8 string from descriptor */
+
+               descriptor = utf_new_char(builtintable_internal[i].descriptor);
+
+               if (!descriptor_pool_add(descpool, descriptor, NULL)) {
+                       /* release dump area */
+
+                       dump_release(dumpsize);
+
+                       return false;
+               }
+       }
+
+       for (i = 0; i < entries_automatic; i++) {
+               /* create a utf8 string from descriptor */
+
+               descriptor = utf_new_char(builtintable_automatic[i].descriptor);
+
+               if (!descriptor_pool_add(descpool, descriptor, NULL)) {
+                       /* release dump area */
+
+                       dump_release(dumpsize);
+
+                       return false;
+               }
+       }
+
+       /* create the class reference table */
+
+       (void) descriptor_pool_create_classrefs(descpool, NULL);
+
+       /* allocate space for the parsed descriptors */
+
+       descriptor_pool_alloc_parsed_descriptors(descpool);
+
+       /* now parse all descriptors */
+
+       for (i = 0; i < entries_internal; i++) {
+               /* create a utf8 string from descriptor */
+
+               descriptor = utf_new_char(builtintable_internal[i].descriptor);
+
+               /* parse the descriptor, builtin is always static (no `this' pointer) */
+
+               builtintable_internal[i].md =
+                       descriptor_pool_parse_method_descriptor(descpool, descriptor,
+                                                                                                       ACC_STATIC, NULL);
+       }
+
+       for (i = 0; i < entries_automatic; i++) {
+               /* create a utf8 string from descriptor */
+
+               descriptor = utf_new_char(builtintable_automatic[i].descriptor);
+
+               /* parse the descriptor, builtin is always static (no `this' pointer) */
+
+               builtintable_automatic[i].md =
+                       descriptor_pool_parse_method_descriptor(descpool, descriptor,
+                                                                                                       ACC_STATIC, NULL);
+       }
+
+       /* release dump area */
+
+       dump_release(dumpsize);
+
+       return true;
+}
+
+
+/* builtintable_comparator *****************************************************
+
+   qsort comparator for the automatic builtin table.
+
+*******************************************************************************/
+
+static int builtintable_comparator(const void *a, const void *b)
+{
+       builtintable_entry *bte1;
+       builtintable_entry *bte2;
+
+       bte1 = (builtintable_entry *) a;
+       bte2 = (builtintable_entry *) b;
+
+       return (bte1->opcode < bte2->opcode) ? -1 : (bte1->opcode > bte2->opcode);
+}
+
+
+/* builtintable_sort_automatic *************************************************
+
+   Sorts the automatic builtin table.
+
+*******************************************************************************/
+
+static void builtintable_sort_automatic(void)
+{
+       s4 entries;
+
+       /* calculate table size statically (`- 1' comment see builtintable.inc) */
+
+       entries = sizeof(builtintable_automatic) / sizeof(builtintable_entry) - 1;
+
+       qsort(builtintable_automatic, entries, sizeof(builtintable_entry),
+                 builtintable_comparator);
+}
+
+
+/* builtin_init ****************************************************************
+
+   XXX
+
+*******************************************************************************/
+
+bool builtin_init(void)
+{
+       /* initialize the builtin tables */
+
+       if (!builtintable_init())
+               return false;
+
+       /* sort builtin tables */
+
+       builtintable_sort_automatic();
+
+       return true;
+}
+
+
+/* builtintable_get_internal ***************************************************
+
+   Finds an entry in the builtintable for internal functions and
+   returns the a pointer to the structure.
+
+*******************************************************************************/
+
+builtintable_entry *builtintable_get_internal(functionptr fp)
+{
+       s4 i;
+
+       for (i = 0; builtintable_internal[i].fp != NULL; i++) {
+               if (builtintable_internal[i].fp == fp)
+                       return &builtintable_internal[i];
+       }
+
+       return NULL;
+}
+
+
+/* builtintable_get_automatic **************************************************
+
+   Finds an entry in the builtintable for functions which are replaced
+   automatically and returns the a pointer to the structure.
+
+*******************************************************************************/
+
+builtintable_entry *builtintable_get_automatic(s4 opcode)
+{
+       builtintable_entry *first;
+       builtintable_entry *last;
+       builtintable_entry *middle;
+       s4                  half;
+       s4                  entries;
+
+       /* calculate table size statically (`- 1' comment see builtintable.inc) */
+
+       entries = sizeof(builtintable_automatic) / sizeof(builtintable_entry) - 1;
+
+       first = builtintable_automatic;
+       last = builtintable_automatic + entries;
+
+       while (entries > 0) {
+               half = entries / 2;
+               middle = first + half;
+
+               if (middle->opcode < opcode) {
+                       first = middle + 1;
+                       entries -= half + 1;
+               } else
+                       entries = half;
+       }
+
+       return (first != last ? first : NULL);
+}
+
 
 /*****************************************************************************
                                                                TYPE CHECKS
@@ -88,29 +336,35 @@ THREADSPECIFIC void *_thread_nativestackframeinfo=NULL;
                                        
 *****************************************************************************/                                 
 s4 builtin_isanysubclass(classinfo *sub, classinfo *super)
-{ 
+{
        s4 res;
        castinfo classvalues;
 
-       /*classinfo *tmp;*/
+       if (sub == super)
+               return 1;
+
        if (super->flags & ACC_INTERFACE)
                return (sub->vftbl->interfacetablelength > super->index) &&
                        (sub->vftbl->interfacetable[-super->index] != NULL);
 
        asm_getclassvalues_atomic(super->vftbl, sub->vftbl, &classvalues);
 
-       res = (unsigned) (classvalues.sub_baseval - classvalues.super_baseval) <=
-               (unsigned) classvalues.super_diffval;
+       res = (u4) (classvalues.sub_baseval - classvalues.super_baseval) <=
+               (u4) classvalues.super_diffval;
 
        return res;
 }
 
-s4 builtin_isanysubclass_vftbl(vftbl_t *sub,vftbl_t *super)
+
+s4 builtin_isanysubclass_vftbl(vftbl_t *sub, vftbl_t *super)
 {
        s4 res;
-       int base;
+       s4 base;
        castinfo classvalues;
-       
+
+       if (sub == super)
+               return 1;
+
        asm_getclassvalues_atomic(super, sub, &classvalues);
 
        if ((base = classvalues.super_baseval) <= 0)
@@ -118,8 +372,8 @@ s4 builtin_isanysubclass_vftbl(vftbl_t *sub,vftbl_t *super)
                res = (sub->interfacetablelength > -base) &&
                        (sub->interfacetable[base] != NULL);
        else
-           res = (unsigned) (classvalues.sub_baseval - classvalues.super_baseval)
-                       <= (unsigned) classvalues.super_diffval;
+           res = (u4) (classvalues.sub_baseval - classvalues.super_baseval)
+                       <= (u4) classvalues.super_diffval;
 
        return res;
 }
@@ -141,8 +395,10 @@ s4 builtin_instanceof(java_objectheader *obj, classinfo *class)
 #ifdef DEBUG
        log_text ("builtin_instanceof called");
 #endif 
-       if (!obj) return 0;
-       return builtin_isanysubclass (obj->vftbl->class, class);
+       if (!obj)
+               return 0;
+
+       return builtin_isanysubclass(obj->vftbl->class, class);
 }
 
 
@@ -178,21 +434,28 @@ s4 builtin_checkcast(java_objectheader *obj, classinfo *class)
 }
 
 
-/*********** internal function: builtin_descriptorscompatible ******************
+/* builtin_descriptorscompatible ***********************************************
 
-       Checks if two array type descriptors are assignment compatible
-       Return value:  1 ... target = desc is possible
-                                  0 ... otherwise
+   Checks if two array type descriptors are assignment compatible
+
+   Return value: 1 ... target = desc is possible
+                 0 ... otherwise
                        
-******************************************************************************/
+*******************************************************************************/
 
-static s4 builtin_descriptorscompatible(arraydescriptor *desc,arraydescriptor *target)
+static s4 builtin_descriptorscompatible(arraydescriptor *desc, arraydescriptor *target)
 {
-       if (desc==target) return 1;
-       if (desc->arraytype != target->arraytype) return 0;
-       if (desc->arraytype != ARRAYTYPE_OBJECT) return 1;
+       if (desc == target)
+               return 1;
+
+       if (desc->arraytype != target->arraytype)
+               return 0;
+
+       if (desc->arraytype != ARRAYTYPE_OBJECT)
+               return 1;
        
        /* {both arrays are arrays of references} */
+
        if (desc->dimension == target->dimension) {
                /* an array which contains elements of interface types is allowed to be casted to Object (JOWENN)*/
                if ( (desc->elementvftbl->baseval<0) && (target->elementvftbl->baseval==1) ) return 1;
@@ -201,42 +464,43 @@ static s4 builtin_descriptorscompatible(arraydescriptor *desc,arraydescriptor *t
        if (desc->dimension < target->dimension) return 0;
 
        /* {desc has higher dimension than target} */
-       return builtin_isanysubclass_vftbl(pseudo_class_Arraystub_vftbl,target->elementvftbl);
+       return builtin_isanysubclass_vftbl(pseudo_class_Arraystub->vftbl, target->elementvftbl);
 }
 
 
-/******************** function: builtin_checkarraycast ***********************
+/* builtin_arraycheckcast ******************************************************
 
-       Checks if an object is really a subtype of the requested array type.
-       The object has to be an array to begin with. For simple arrays (int, short,
-       double, etc.) the types have to match exactly.
-       For arrays of objects, the type of elements in the array has to be a
-       subtype (or the same type) of the requested element type. For arrays of
-       arrays (which in turn can again be arrays of arrays), the types at the
-       lowest level have to satisfy the corresponding sub class relation.
+   Checks if an object is really a subtype of the requested array
+   type.  The object has to be an array to begin with. For simple
+   arrays (int, short, double, etc.) the types have to match exactly.
+   For arrays of objects, the type of elements in the array has to be
+   a subtype (or the same type) of the requested element type. For
+   arrays of arrays (which in turn can again be arrays of arrays), the
+   types at the lowest level have to satisfy the corresponding sub
+   class relation.
        
-       Return value:  1 ... cast is possible
-                                  0 ... otherwise
-       
-       ATTENTION: a cast with a NULL pointer is always possible.
-                       
-*****************************************************************************/
+*******************************************************************************/
 
-s4 builtin_checkarraycast(java_objectheader *o, vftbl_t *target)
+s4 builtin_arraycheckcast(java_objectheader *o, vftbl_t *target)
 {
        arraydescriptor *desc;
-       
-       if (!o) return 1;
-       if ((desc = o->vftbl->arraydesc) == NULL) return 0;
 
+       if (!o)
+               return 1;
+
+       if ((desc = o->vftbl->arraydesc) == NULL)
+               return 0;
        return builtin_descriptorscompatible(desc, target->arraydesc);
 }
 
 
 s4 builtin_arrayinstanceof(java_objectheader *obj, vftbl_t *target)
 {
-       if (!obj) return 1;
-       return builtin_checkarraycast(obj, target);
+       if (!obj)
+               return 1;
+
+       return builtin_arraycheckcast(obj, target);
 }
 
 
@@ -246,24 +510,57 @@ s4 builtin_arrayinstanceof(java_objectheader *obj, vftbl_t *target)
 
 java_objectheader *builtin_throw_exception(java_objectheader *xptr)
 {
+    java_lang_Throwable *t;
+       char                *logtext;
+       s4                   logtextlen;
+       s4                   dumpsize;
+
        if (opt_verbose) {
-               char logtext[MAXLOGTEXT];
-               sprintf(logtext, "Builtin exception thrown: ");
-               if (xptr) {
-                       java_lang_Throwable *t = (java_lang_Throwable *) xptr;
+               t = (java_lang_Throwable *) xptr;
+
+               /* calculate message length */
+
+               logtextlen = strlen("Builtin exception thrown: ") + strlen("0");
+
+               if (t) {
+                       logtextlen +=
+                               utf_strlen(xptr->vftbl->class->name) +
+                               strlen(": ") +
+                               javastring_strlen((java_objectheader *) t->detailMessage);
+
+               } else
+                       logtextlen += strlen("(nil)");
+
+               /* allocate memory */
+
+               dumpsize = dump_size();
 
+               logtext = DMNEW(char, logtextlen);
+
+               strcpy(logtext, "Builtin exception thrown: ");
+
+               if (t) {
                        utf_sprint_classname(logtext + strlen(logtext),
                                                                 xptr->vftbl->class->name);
 
                        if (t->detailMessage) {
-                               sprintf(logtext + strlen(logtext), ": %s",
-                                               javastring_tochar((java_objectheader *) t->detailMessage));
+                               char *buf;
+
+                               buf = javastring_tochar((java_objectheader *) t->detailMessage);
+                               strcat(logtext, ": ");
+                               strcat(logtext, buf);
+                               MFREE(buf, char, strlen(buf));
                        }
 
                } else {
-                       sprintf(logtext + strlen(logtext), "Error: <Nullpointer instead of exception>");
+                       strcat(logtext, "(nil)");
                }
+
                log_text(logtext);
+
+               /* release memory */
+
+               dump_release(dumpsize);
        }
 
        *exceptionptr = xptr;
@@ -273,25 +570,26 @@ java_objectheader *builtin_throw_exception(java_objectheader *xptr)
 
 
 
-/******************* function: builtin_canstore *******************************
+/* builtin_canstore ************************************************************
 
-       Checks, if an object can be stored in an array.
-       Return value:  1 ... possible
-                                  0 ... otherwise
+   Checks, if an object can be stored in an array.
 
-******************************************************************************/
+   Return value: 1 ... possible
+                 0 ... otherwise
 
-s4 builtin_canstore (java_objectarray *a, java_objectheader *o)
+*******************************************************************************/
+
+s4 builtin_canstore(java_objectarray *a, java_objectheader *o)
 {
        arraydescriptor *desc;
        arraydescriptor *valuedesc;
        vftbl_t *componentvftbl;
        vftbl_t *valuevftbl;
-    int dim_m1;
        int base;
        castinfo classvalues;
        
-       if (!o) return 1;
+       if (!o)
+               return 1;
 
        /* The following is guaranteed (by verifier checks):
         *
@@ -301,10 +599,10 @@ s4 builtin_canstore (java_objectarray *a, java_objectheader *o)
         */
        
        desc = a->header.objheader.vftbl->arraydesc;
-    componentvftbl = desc->componentvftbl;
+       componentvftbl = desc->componentvftbl;
        valuevftbl = o->vftbl;
 
-    if ((dim_m1 = desc->dimension - 1) == 0) {
+       if ((desc->dimension - 1) == 0) {
                s4 res;
 
                /* {a is a one-dimensional array} */
@@ -324,8 +622,9 @@ s4 builtin_canstore (java_objectarray *a, java_objectheader *o)
                        <= (unsigned) classvalues.super_diffval;
 
                return res;
-    }
-    /* {a has dimension > 1} */
+       }
+
+       /* {a has dimension > 1} */
        /* {componentvftbl->arraydesc != NULL} */
 
        /* check if o is an array */
@@ -417,52 +716,45 @@ s4 builtin_canstore_onedim_class(java_objectarray *a, java_objectheader *o)
 }
 
 
-/******************** Function: builtin_new **********************************
+/* builtin_new *****************************************************************
 
-       Creates a new instance of class c on the heap.
-       Return value:  pointer to the object or NULL if no memory is
-                                  available
-                       
-*****************************************************************************/
+   Creates a new instance of class c on the heap.
 
-#define ALIGNMENT 3
-#define align_size(size)       ((size + ((1 << ALIGNMENT) - 1)) & ~((1 << ALIGNMENT) - 1))
+   Return value: pointer to the object or NULL if no memory is
+   available
+                       
+*******************************************************************************/
 
 java_objectheader *builtin_new(classinfo *c)
 {
        java_objectheader *o;
 
        /* is the class loaded */
-       if (!c->loaded)
-               if (!class_load(c))
-                       return NULL;
+
+       assert(c->loaded);
 
        /* is the class linked */
        if (!c->linked)
-               if (!class_link(c))
+               if (!link_class(c))
                        return NULL;
 
        if (!c->initialized) {
                if (initverbose)
                        log_message_class("Initialize class (from builtin_new): ", c);
 
-               if (!class_init(c))
+               if (!initialize_class(c))
                        return NULL;
        }
 
-#ifdef SIZE_FROM_CLASSINFO
-       c->alignedsize = align_size(c->instancesize);
-       o = heap_allocate(c->alignedsize, true, c->finalizer);
-#else
        o = heap_allocate(c->instancesize, true, c->finalizer);
-#endif
 
        if (!o)
                return NULL;
 
-       memset(o, 0, c->instancesize);
+       MSET(o, 0, u1, c->instancesize);
 
        o->vftbl = c->vftbl;
+
 #if defined(USE_THREADS) && defined(NATIVE_THREADS)
        initObjectLock(o);
 #endif
@@ -471,16 +763,16 @@ java_objectheader *builtin_new(classinfo *c)
 }
 
 
-/********************** Function: builtin_newarray **************************
+/* builtin_newarray ************************************************************
 
-       Creates an array with the given vftbl on the heap.
+   Creates an array with the given vftbl on the heap.
 
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is available
 
-    CAUTION: The given vftbl must be the vftbl of the *array* class,
-    not of the element class.
+   CAUTION: The given vftbl must be the vftbl of the *array* class,
+   not of the element class.
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_arrayheader *builtin_newarray(s4 size, vftbl_t *arrayvftbl)
 {
@@ -495,236 +787,248 @@ java_arrayheader *builtin_newarray(s4 size, vftbl_t *arrayvftbl)
        componentsize = desc->componentsize;
 
        if (size < 0) {
-               *exceptionptr =
-                       new_exception(string_java_lang_NegativeArraySizeException);
+               *exceptionptr = new_negativearraysizeexception();
                return NULL;
        }
 
-#ifdef SIZE_FROM_CLASSINFO
-       actualsize = align_size(dataoffset + size * componentsize);
        actualsize = dataoffset + size * componentsize;
-#else
-       actualsize = 0;
-#endif
 
        if (((u4) actualsize) < ((u4) size)) { /* overflow */
                *exceptionptr = new_exception(string_java_lang_OutOfMemoryError);
                return NULL;
        }
 
-       a = heap_allocate(actualsize,
-                                         (desc->arraytype == ARRAYTYPE_OBJECT),
-                                         NULL);
+       a = heap_allocate(actualsize, (desc->arraytype == ARRAYTYPE_OBJECT), NULL);
 
        if (!a)
                return NULL;
 
-       memset(a, 0, actualsize);
+       MSET(a, 0, u1, actualsize);
 
        a->objheader.vftbl = arrayvftbl;
+
 #if defined(USE_THREADS) && defined(NATIVE_THREADS)
        initObjectLock(&a->objheader);
 #endif
+
        a->size = size;
-#ifdef SIZE_FROM_CLASSINFO
-       a->alignedsize = actualsize;
-#endif
 
        return a;
 }
 
 
-/********************** Function: builtin_anewarray *************************
-
-       Creates an array of references to the given class type on the heap.
+/* builtin_anewarray ***********************************************************
 
-       Return value: pointer to the array or NULL if no memory is available
+   Creates an array of references to the given class type on the heap.
 
-    XXX This function does not do The Right Thing, because it uses a
-    classinfo pointer at runtime. builtin_newarray should be used
-    instead.
+   Return value: pointer to the array or NULL if no memory is
+   available
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_objectarray *builtin_anewarray(s4 size, classinfo *component)
 {
+       classinfo *c;
+       
        /* is class loaded */
-       if (!component->loaded)
-               if (!class_load(component))
-                       return NULL;
+       assert(component->loaded);
 
        /* is class linked */
        if (!component->linked)
-               if (!class_link(component))
+               if (!link_class(component))
                        return NULL;
 
-       return (java_objectarray *) builtin_newarray(size, class_array_of(component)->vftbl);
+       c = class_array_of(component, true);
+
+       if (!c)
+               return NULL;
+
+       return (java_objectarray *) builtin_newarray(size, c->vftbl);
 }
 
 
-/******************** Function: builtin_newarray_int ***********************
+/* builtin_newarray_int ********************************************************
 
-       Creates an array of 32 bit Integers on the heap.
+   Creates an array of 32 bit Integers on the heap.
 
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is
+   available
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_intarray *builtin_newarray_int(s4 size)
 {
-       return (java_intarray*) builtin_newarray(size, primitivetype_table[ARRAYTYPE_INT].arrayvftbl);
+       return (java_intarray *)
+               builtin_newarray(size, primitivetype_table[ARRAYTYPE_INT].arrayvftbl);
 }
 
 
-/******************** Function: builtin_newarray_long ***********************
+/* builtin_newarray_long *******************************************************
 
-       Creates an array of 64 bit Integers on the heap.
+   Creates an array of 64 bit Integers on the heap.
 
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is
+   available
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_longarray *builtin_newarray_long(s4 size)
 {
-       return (java_longarray*) builtin_newarray(size, primitivetype_table[ARRAYTYPE_LONG].arrayvftbl);
+       return (java_longarray *)
+               builtin_newarray(size, primitivetype_table[ARRAYTYPE_LONG].arrayvftbl);
 }
 
 
-/******************** function: builtin_newarray_float ***********************
+/* builtin_newarray_float ******************************************************
 
-       Creates an array of 32 bit IEEE floats on the heap.
+   Creates an array of 32 bit IEEE floats on the heap.
 
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is
+   available
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_floatarray *builtin_newarray_float(s4 size)
 {
-       return (java_floatarray*) builtin_newarray(size, primitivetype_table[ARRAYTYPE_FLOAT].arrayvftbl);
+       return (java_floatarray *)
+               builtin_newarray(size, primitivetype_table[ARRAYTYPE_FLOAT].arrayvftbl);
 }
 
 
-/******************** function: builtin_newarray_double ***********************
+/* builtin_newarray_double *****************************************************
 
-       Creates an array of 64 bit IEEE floats on the heap.
+   Creates an array of 64 bit IEEE floats on the heap.
 
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is
+   available
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_doublearray *builtin_newarray_double(s4 size)
 {
-       return (java_doublearray*) builtin_newarray(size, primitivetype_table[ARRAYTYPE_DOUBLE].arrayvftbl);
+       return (java_doublearray *)
+               builtin_newarray(size,
+                                                primitivetype_table[ARRAYTYPE_DOUBLE].arrayvftbl);
 }
 
 
-/******************** function: builtin_newarray_byte ***********************
+/* builtin_newarray_byte *******************************************************
 
-       Creates an array of 8 bit Integers on the heap.
+   Creates an array of 8 bit Integers on the heap.
 
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is
+   available
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_bytearray *builtin_newarray_byte(s4 size)
 {
-       return (java_bytearray*) builtin_newarray(size, primitivetype_table[ARRAYTYPE_BYTE].arrayvftbl);
+       return (java_bytearray *)
+               builtin_newarray(size, primitivetype_table[ARRAYTYPE_BYTE].arrayvftbl);
 }
 
 
-/******************** function: builtin_newarray_char ************************
+/* builtin_newarray_char *******************************************************
 
-       Creates an array of characters on the heap.
+   Creates an array of characters on the heap.
 
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is
+   available
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_chararray *builtin_newarray_char(s4 size)
 {
-       return (java_chararray*) builtin_newarray(size, primitivetype_table[ARRAYTYPE_CHAR].arrayvftbl);
+       return (java_chararray *)
+               builtin_newarray(size, primitivetype_table[ARRAYTYPE_CHAR].arrayvftbl);
 }
 
 
-/******************** function: builtin_newarray_short ***********************
+/* builtin_newarray_short ******************************************************
 
-       Creates an array of 16 bit Integers on the heap.
+   Creates an array of 16 bit Integers on the heap.
 
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is
+   available
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_shortarray *builtin_newarray_short(s4 size)
 {
-       return (java_shortarray*) builtin_newarray(size, primitivetype_table[ARRAYTYPE_SHORT].arrayvftbl);
+       return (java_shortarray *)
+               builtin_newarray(size, primitivetype_table[ARRAYTYPE_SHORT].arrayvftbl);
 }
 
 
-/******************** function: builtin_newarray_boolean ************************
+/* builtin_newarray_boolean ****************************************************
 
-       Creates an array of bytes on the heap. The array is designated as an array
-       of booleans (important for casts)
+   Creates an array of bytes on the heap. The array is designated as
+   an array of booleans (important for casts)
        
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is
+   available
 
-*****************************************************************************/
+*******************************************************************************/
 
 java_booleanarray *builtin_newarray_boolean(s4 size)
 {
-       return (java_booleanarray*) builtin_newarray(size, primitivetype_table[ARRAYTYPE_BOOLEAN].arrayvftbl);
+       return (java_booleanarray *)
+               builtin_newarray(size,
+                                                primitivetype_table[ARRAYTYPE_BOOLEAN].arrayvftbl);
 }
 
 
-/**************** function: builtin_nmultianewarray ***************************
+/* builtin_multianewarray ******************************************************
 
-       Creates a multi-dimensional array on the heap. The dimensions are passed in
-       an array of longs.
+   Creates a multi-dimensional array on the heap. The dimensions are
+   passed in an array of longs.
 
-    Arguments:
-        n............number of dimensions to create
-        arrayvftbl...vftbl of the array class
-        dims.........array containing the size of each dimension to create
+   Arguments:
+       n............number of dimensions to create
+       arrayvftbl...vftbl of the array class
+       dims.........array containing the size of each dimension to create
 
-       Return value:  pointer to the array or NULL if no memory is available
+   Return value: pointer to the array or NULL if no memory is
+   available
 
 ******************************************************************************/
 
-java_arrayheader *builtin_nmultianewarray(int n, vftbl_t *arrayvftbl, long *dims)
-/*  java_arrayheader *builtin_nmultianewarray(int n, classinfo *arrayclass, long *dims) */
+java_arrayheader *builtin_multianewarray(int n, vftbl_t *arrayvftbl, long *dims)
 {
        s4 size, i;
        java_arrayheader *a;
        vftbl_t *componentvftbl;
 
-/*     utf_display(arrayclass->name); */
-
-/*     class_load(arrayclass); */
-/*     class_link(arrayclass); */
-       
        /* create this dimension */
+
        size = (s4) dims[0];
        a = builtin_newarray(size, arrayvftbl);
-/*     a = builtin_newarray(size, arrayclass->vftbl); */
 
        if (!a)
                return NULL;
 
        /* if this is the last dimension return */
+
        if (!--n)
                return a;
 
        /* get the vftbl of the components to create */
+
        componentvftbl = arrayvftbl->arraydesc->componentvftbl;
-/*     component = arrayclass->vftbl->arraydesc; */
 
-       /* The verifier guarantees this. */
-       /* if (!componentvftbl) */
-       /*      panic ("multianewarray with too many dimensions"); */
+       /* The verifier guarantees that the dimension count is in the range. */
 
        /* create the component arrays */
+
        for (i = 0; i < size; i++) {
-               java_arrayheader *ea = 
-                       builtin_nmultianewarray(n, componentvftbl, dims + 1);
+               java_arrayheader *ea =
+#if defined(__MIPS__) && (SIZEOF_VOID_P == 4)
+                       /* we save an s4 to a s8 slot, 8-byte aligned */
+
+                       builtin_multianewarray(n, componentvftbl, dims + 2);
+#else
+                       builtin_multianewarray(n, componentvftbl, dims + 1);
+#endif
 
                if (!ea)
                        return NULL;
@@ -752,285 +1056,465 @@ java_objectheader *builtin_trace_exception(java_objectheader *xptr,
                                                                                   s4 line,
                                                                                   s4 noindent)
 {
-       if (!noindent) {
-               if (methodindent)
-                       methodindent--;
-               else
-                       log_text("WARNING: unmatched methodindent--");
-       }
+       char *logtext;
+       s4    logtextlen;
+       s4    dumpsize;
+
        if (opt_verbose || runverbose || verboseexception) {
+               /* calculate message length */
+
                if (xptr) {
-                       printf("Exception ");
-                       utf_display_classname(xptr->vftbl->class->name);
+                       logtextlen =
+                               strlen("Exception ") +
+                               utf_strlen(xptr->vftbl->class->name);
+
+               } else
+                       logtextlen = strlen("Some Throwable");
+
+               logtextlen += strlen(" thrown in ");
+
+               if (m) {
+                       logtextlen +=
+                               utf_strlen(m->class->name) +
+                               strlen(".") +
+                               utf_strlen(m->name) +
+                               utf_strlen(m->descriptor) +
+                               strlen("(NOSYNC,NATIVE");
+
+#if SIZEOF_VOID_P == 8
+                       logtextlen +=
+                               strlen(")(0x123456789abcdef0) at position 0x123456789abcdef0 (");
+#else
+                       logtextlen += strlen(")(0x12345678) at position 0x12345678 (");
+#endif
+
+                       if (m->class->sourcefile == NULL)
+                               logtextlen += strlen("<NO CLASSFILE INFORMATION>");
+                       else
+                               logtextlen += utf_strlen(m->class->sourcefile);
+
+                       logtextlen += strlen(":65536)");
+
+               } else
+                       logtextlen += strlen("call_java_method");
+
+               logtextlen += strlen("0");
+
+               /* allocate memory */
+
+               dumpsize = dump_size();
+
+               logtext = DMNEW(char, logtextlen);
+
+               if (xptr) {
+                       strcpy(logtext, "Exception ");
+                       utf_strcat_classname(logtext, xptr->vftbl->class->name);
 
                } else {
-                       printf("Some Throwable");
+                       strcpy(logtext, "Some Throwable");
                }
-               printf(" thrown in ");
+
+               strcat(logtext, " thrown in ");
 
                if (m) {
-                       utf_display_classname(m->class->name);
-                       printf(".");
-                       utf_display(m->name);
-                       if (m->flags & ACC_SYNCHRONIZED) {
-                               printf("(SYNC");
-
-                       } else{
-                               printf("(NOSYNC");
-                       }
+                       utf_strcat_classname(logtext, m->class->name);
+                       strcat(logtext, ".");
+                       utf_strcat(logtext, m->name);
+                       utf_strcat(logtext, m->descriptor);
+
+                       if (m->flags & ACC_SYNCHRONIZED)
+                               strcat(logtext, "(SYNC");
+                       else
+                               strcat(logtext, "(NOSYNC");
 
                        if (m->flags & ACC_NATIVE) {
-                               printf(",NATIVE");
-#if POINTERSIZE == 8
-                               printf(")(0x%016lx) at position %p\n", (s8) m->entrypoint, pos);
+                               strcat(logtext, ",NATIVE");
+
+#if SIZEOF_VOID_P == 8
+                               sprintf(logtext + strlen(logtext),
+                                               ")(0x%016lx) at position 0x%016lx",
+                                               (ptrint) m->entrypoint, (ptrint) pos);
 #else
-                               printf(")(0x%08lx) at position %p\n", (s4) m->entrypoint, pos);
+                               sprintf(logtext + strlen(logtext),
+                                               ")(0x%08x) at position 0x%08x",
+                                               (ptrint) m->entrypoint, (ptrint) pos);
 #endif
 
                        } else {
-#if POINTERSIZE == 8
-                               printf(")(0x%016lx) at position %p (", (s8) m->entrypoint, pos);
+#if SIZEOF_VOID_P == 8
+                               sprintf(logtext + strlen(logtext),
+                                               ")(0x%016lx) at position 0x%016lx (",
+                                               (ptrint) m->entrypoint, (ptrint) pos);
 #else
-                               printf(")(0x%08lx) at position %p (", (s4) m->entrypoint, pos);
+                               sprintf(logtext + strlen(logtext),
+                                               ")(0x%08x) at position 0x%08x (",
+                                               (ptrint) m->entrypoint, (ptrint) pos);
 #endif
-                               if (m->class->sourcefile == NULL) {
-                                       printf("<NO CLASSFILE INFORMATION>");
 
-                               } else {
-                                       utf_display(m->class->sourcefile);
-                               }
-                               printf(":%d)\n", line);
+                               if (m->class->sourcefile == NULL)
+                                       strcat(logtext, "<NO CLASSFILE INFORMATION>");
+                               else
+                                       utf_strcat(logtext, m->class->sourcefile);
+
+                               sprintf(logtext + strlen(logtext), ":%d)", line);
                        }
 
                } else
-                       printf("call_java_method\n");
-               fflush(stdout);
+                       strcat(logtext, "call_java_method");
+
+               log_text(logtext);
+
+               /* release memory */
+
+               dump_release(dumpsize);
        }
 
        return xptr;
 }
 
 
+/* builtin_trace_args **********************************************************
+
+   XXX
+
+*******************************************************************************/
+
 #ifdef TRACE_ARGS_NUM
-void builtin_trace_args(s8 a0, s8 a1, s8 a2, s8 a3, s8 a4, s8 a5,
-#if TRACE_ARGS_NUM > 6
+void builtin_trace_args(s8 a0, s8 a1,
+#if TRACE_ARGS_NUM >= 4
+                                               s8 a2, s8 a3,
+#endif /* TRACE_ARGS_NUM >= 4 */
+#if TRACE_ARGS_NUM >= 6
+                                               s8 a4, s8 a5,
+#endif /* TRACE_ARGS_NUM >= 6 */
+#if TRACE_ARGS_NUM == 8
                                                s8 a6, s8 a7,
-#endif
+#endif /* TRACE_ARGS_NUM == 8 */
                                                methodinfo *m)
 {
-       s4 i;
-       char logtext[MAXLOGTEXT];
-       for (i = 0; i < methodindent; i++)
-               logtext[i] = '\t';
+       methoddesc *md;
+       char       *logtext;
+       s4          logtextlen;
+       s4          dumpsize;
+       s4          i;
 
-       sprintf(logtext + methodindent, "called: ");
-       utf_sprint_classname(logtext + strlen(logtext), m->class->name);
-       sprintf(logtext + strlen(logtext), ".");
-       utf_sprint(logtext + strlen(logtext), m->name);
-       utf_sprint_classname(logtext + strlen(logtext), m->descriptor);
-
-       if (m->flags & ACC_PUBLIC)       sprintf(logtext + strlen(logtext), " PUBLIC");
-       if (m->flags & ACC_PRIVATE)      sprintf(logtext + strlen(logtext), " PRIVATE");
-       if (m->flags & ACC_PROTECTED)    sprintf(logtext + strlen(logtext), " PROTECTED");
-       if (m->flags & ACC_STATIC)       sprintf(logtext + strlen(logtext), " STATIC");
-       if (m->flags & ACC_FINAL)        sprintf(logtext + strlen(logtext), " FINAL");
-       if (m->flags & ACC_SYNCHRONIZED) sprintf(logtext + strlen(logtext), " SYNCHRONIZED");
-       if (m->flags & ACC_VOLATILE)     sprintf(logtext + strlen(logtext), " VOLATILE");
-       if (m->flags & ACC_TRANSIENT)    sprintf(logtext + strlen(logtext), " TRANSIENT");
-       if (m->flags & ACC_NATIVE)       sprintf(logtext + strlen(logtext), " NATIVE");
-       if (m->flags & ACC_INTERFACE)    sprintf(logtext + strlen(logtext), " INTERFACE");
-       if (m->flags & ACC_ABSTRACT)     sprintf(logtext + strlen(logtext), " ABSTRACT");
-       
+       md = m->parseddesc;
+
+       /* calculate message length */
+
+       logtextlen =
+               methodindent + strlen("called: ") +
+               utf_strlen(m->class->name) +
+               strlen(".") +
+               utf_strlen(m->name) +
+               utf_strlen(m->descriptor) +
+               strlen(" SYNCHRONIZED") + strlen("(") + strlen(")");
+
+       /* add maximal argument length */
 
-       sprintf(logtext + strlen(logtext), "(");
+       logtextlen += strlen("0x123456789abcdef0, 0x123456789abcdef0, 0x123456789abcdef0, 0x123456789abcdef0, 0x123456789abcdef0, 0x123456789abcdef0, 0x123456789abcdef0, 0x123456789abcdef0, ...(255)");
 
-       switch (m->paramcount) {
+       /* allocate memory */
+
+       dumpsize = dump_size();
+
+       logtext = DMNEW(char, logtextlen);
+
+       for (i = 0; i < methodindent; i++)
+               logtext[i] = '\t';
+
+       strcpy(logtext + methodindent, "called: ");
+
+       utf_strcat_classname(logtext, m->class->name);
+       strcat(logtext, ".");
+       utf_strcat(logtext, m->name);
+       utf_strcat(logtext, m->descriptor);
+
+       if (m->flags & ACC_PUBLIC)       strcat(logtext, " PUBLIC");
+       if (m->flags & ACC_PRIVATE)      strcat(logtext, " PRIVATE");
+       if (m->flags & ACC_PROTECTED)    strcat(logtext, " PROTECTED");
+       if (m->flags & ACC_STATIC)       strcat(logtext, " STATIC");
+       if (m->flags & ACC_FINAL)        strcat(logtext, " FINAL");
+       if (m->flags & ACC_SYNCHRONIZED) strcat(logtext, " SYNCHRONIZED");
+       if (m->flags & ACC_VOLATILE)     strcat(logtext, " VOLATILE");
+       if (m->flags & ACC_TRANSIENT)    strcat(logtext, " TRANSIENT");
+       if (m->flags & ACC_NATIVE)       strcat(logtext, " NATIVE");
+       if (m->flags & ACC_INTERFACE)    strcat(logtext, " INTERFACE");
+       if (m->flags & ACC_ABSTRACT)     strcat(logtext, " ABSTRACT");
+
+       strcat(logtext, "(");
+
+       /* xxxprintf ?Bug? an PowerPc Linux (rlwinm.inso)                */
+       /* Only Arguments in integer Registers are passed correctly here */
+       /* long longs spilled on Stack have an wrong offset of +4        */
+       /* So preliminary Bugfix: Only pass 3 params at once to sprintf  */
+       /* for SIZEOG_VOID_P == 4 && TRACE_ARGS_NUM == 8                 */
+       switch (md->paramcount) {
        case 0:
                break;
 
-#if defined(__I386__) || defined(__POWERPC__)
+#if SIZEOF_VOID_P == 4
        case 1:
-               sprintf(logtext+strlen(logtext), "%llx", a0);
+               sprintf(logtext + strlen(logtext),
+                               "0x%llx",
+                               a0);
                break;
 
        case 2:
-               sprintf(logtext+strlen(logtext), "%llx, %llx", a0, a1);
+               sprintf(logtext + strlen(logtext),
+                               "0x%llx, 0x%llx",
+                               a0, a1);
                break;
 
+#if TRACE_ARGS_NUM >= 4
        case 3:
-               sprintf(logtext+strlen(logtext), "%llx, %llx, %llx", a0, a1, a2);
+               sprintf(logtext + strlen(logtext),
+                               "0x%llx, 0x%llx, 0x%llx",
+                               a0, a1, a2);
                break;
 
        case 4:
-               sprintf(logtext+strlen(logtext), "%llx, %llx, %llx, %llx",
-                               a0,   a1,   a2,   a3);
+               sprintf(logtext + strlen(logtext), "0x%llx, 0x%llx, 0x%llx"
+                               , a0, a1, a2);
+               sprintf(logtext + strlen(logtext), ", 0x%llx", a3);
+
                break;
+#endif /* TRACE_ARGS_NUM >= 4 */
 
+#if TRACE_ARGS_NUM >= 6
        case 5:
-               sprintf(logtext+strlen(logtext), "%llx, %llx, %llx, %llx, %llx",
-                               a0,   a1,   a2,   a3,   a4);
+               sprintf(logtext + strlen(logtext), "0x%llx, 0x%llx, 0x%llx"
+                               , a0, a1, a2);
+               sprintf(logtext + strlen(logtext), ", 0x%llx, 0x%llx", a3, a4);
                break;
 
+
        case 6:
-               sprintf(logtext+strlen(logtext), "%llx, %llx, %llx, %llx, %llx, %llx",
-                               a0,   a1,   a2,   a3,   a4,   a5);
+               sprintf(logtext + strlen(logtext), "0x%llx, 0x%llx, 0x%llx"
+                               , a0, a1, a2);
+               sprintf(logtext + strlen(logtext), ", 0x%llx, 0x%llx, 0x%llx"
+                               , a3, a4, a5);
                break;
+#endif /* TRACE_ARGS_NUM >= 6 */
 
-#if TRACE_ARGS_NUM > 6
+#if TRACE_ARGS_NUM == 8
        case 7:
-               sprintf(logtext+strlen(logtext), "%llx, %llx, %llx, %llx, %llx, %llx, %llx",
-                               a0,   a1,   a2,   a3,   a4,   a5,   a6);
+               sprintf(logtext + strlen(logtext), "0x%llx, 0x%llx, 0x%llx"
+                               , a0, a1, a2);
+               sprintf(logtext + strlen(logtext), ", 0x%llx, 0x%llx, 0x%llx"
+                               , a3, a4, a5);
+               sprintf(logtext + strlen(logtext), ", 0x%llx", a6);
                break;
 
        case 8:
-               sprintf(logtext+strlen(logtext), "%llx, %llx, %llx, %llx, %llx, %llx, %llx, %llx",
-                               a0,   a1,   a2,   a3,   a4,   a5,   a6,   a7);
+               sprintf(logtext + strlen(logtext), "0x%llx, 0x%llx, 0x%llx"
+                               , a0, a1, a2);
+               sprintf(logtext + strlen(logtext), ", 0x%llx, 0x%llx, 0x%llx"
+                               , a3, a4, a5);
+               sprintf(logtext + strlen(logtext), ", 0x%llx, 0x%llx", a6, a7);
                break;
+#endif /* TRACE_ARGS_NUM == 8 */
 
        default:
-               sprintf(logtext+strlen(logtext), "%llx, %llx, %llx, %llx, %llx, %llx, %llx, %llx, ...(%d)",
-                               a0,   a1,   a2,   a3,   a4,   a5,   a6,   a7,   m->paramcount - 8);
-               break;
-#else
-       default:
-               sprintf(logtext+strlen(logtext), "%llx, %llx, %llx, %llx, %llx, %llx, ...(%d)",
-                               a0,   a1,   a2,   a3,   a4,   a5,   m->paramcount - 6);
-               break;
+#if TRACE_ARGS_NUM == 2
+               sprintf(logtext + strlen(logtext), "0x%llx, 0x%llx, ...(%d)", a0, a1, md->paramcount - 2);
+
+#elif TRACE_ARGS_NUM == 4
+               sprintf(logtext + strlen(logtext),
+                               "0x%llx, 0x%llx, 0x%llx, 0x%llx, ...(%d)",
+                               a0, a1, a2, a3, md->paramcount - 4);
+
+#elif TRACE_ARGS_NUM == 6
+               sprintf(logtext + strlen(logtext),
+                               "0x%llx, 0x%llx, 0x%llx, 0x%llx, 0x%llx, 0x%llx, ...(%d)",
+                               a0, a1, a2, a3, a4, a5, md->paramcount - 6);
+
+#elif TRACE_ARGS_NUM == 8
+               sprintf(logtext + strlen(logtext),"0x%llx, 0x%llx, 0x%llx,"
+                               , a0, a1, a2);
+               sprintf(logtext + strlen(logtext)," 0x%llx, 0x%llx, 0x%llx,"
+                               , a3, a4, a5);
+               sprintf(logtext + strlen(logtext)," 0x%llx, 0x%llx, ...(%d)"
+                               , a6, a7, md->paramcount - 8);
 #endif
-#else
+               break;
+
+#else /* SIZEOF_VOID_P == 4 */
+
        case 1:
-               sprintf(logtext+strlen(logtext), "%lx", a0);
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx",
+                               a0);
                break;
 
        case 2:
-               sprintf(logtext+strlen(logtext), "%lx, %lx", a0, a1);
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx",
+                               a0, a1);
                break;
 
        case 3:
-               sprintf(logtext+strlen(logtext), "%lx, %lx, %lx", a0, a1, a2);
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx, 0x%lx", a0, a1, a2);
                break;
 
        case 4:
-               sprintf(logtext+strlen(logtext), "%lx, %lx, %lx, %lx",
-                               a0,  a1,  a2,  a3);
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx, 0x%lx, 0x%lx",
+                               a0, a1, a2, a3);
                break;
 
+#if TRACE_ARGS_NUM >= 6
        case 5:
-               sprintf(logtext+strlen(logtext), "%lx, %lx, %lx, %lx, %lx",
-                               a0,  a1,  a2,  a3,  a4);
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx",
+                               a0, a1, a2, a3, a4);
                break;
 
        case 6:
-               sprintf(logtext+strlen(logtext), "%lx, %lx, %lx, %lx, %lx, %lx",
-                               a0,  a1,  a2,  a3,  a4,  a5);
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx",
+                               a0, a1, a2, a3, a4, a5);
                break;
+#endif /* TRACE_ARGS_NUM >= 6 */
 
-#if TRACE_ARGS_NUM > 6
+#if TRACE_ARGS_NUM == 8
        case 7:
-               sprintf(logtext+strlen(logtext), "%lx, %lx, %lx, %lx, %lx, %lx, %lx",
-                               a0,  a1,  a2,  a3,  a4,  a5,  a6);
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx",
+                               a0, a1, a2, a3, a4, a5, a6);
                break;
 
        case 8:
-               sprintf(logtext+strlen(logtext), "%lx, %lx, %lx, %lx, %lx, %lx, %lx, %lx",
-                               a0,  a1,  a2,  a3,  a4,  a5,  a6,  a7);
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx",
+                               a0, a1, a2, a3, a4, a5, a6, a7);
                break;
+#endif /* TRACE_ARGS_NUM == 8 */
 
        default:
-               sprintf(logtext+strlen(logtext), "%lx, %lx, %lx, %lx, %lx, %lx, %lx, %lx, ...(%d)",
-                               a0,  a1,  a2,  a3,  a4,  a5,  a6,  a7,  m->paramcount - 8);
-               break;
-#else
-       default:
-               sprintf(logtext+strlen(logtext), "%lx, %lx, %lx, %lx, %lx, %lx, ...(%d)",
-                               a0,  a1,  a2,  a3,  a4,  a5,   m->paramcount - 6);
-               break;
-#endif
+#if TRACE_ARGS_NUM == 4
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx, 0x%lx, 0x%lx, ...(%d)",
+                               a0, a1, a2, a3, md->paramcount - 4);
+
+#elif TRACE_ARGS_NUM == 6
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, ...(%d)",
+                               a0, a1, a2, a3, a4, a5, md->paramcount - 6);
+
+#elif TRACE_ARGS_NUM == 8
+               sprintf(logtext + strlen(logtext),
+                               "0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, 0x%lx, ...(%d)",
+                               a0, a1, a2, a3, a4, a5, a6, a7, md->paramcount - 8);
 #endif
+               break;
+#endif /* SIZEOF_VOID_P == 4 */
        }
 
-       sprintf(logtext + strlen(logtext), ")");
+       strcat(logtext, ")");
+
        log_text(logtext);
 
+       /* release memory */
+
+       dump_release(dumpsize);
+
        methodindent++;
 }
 #endif
 
 
-void builtin_displaymethodstart(methodinfo *m)
-{
-       char logtext[MAXLOGTEXT];
-       sprintf(logtext, "                                                                                              ");
-       sprintf(logtext + methodindent, "called: ");
-       utf_sprint(logtext + strlen(logtext), m->class->name);
-       sprintf(logtext + strlen(logtext), ".");
-       utf_sprint(logtext + strlen(logtext), m->name);
-       utf_sprint(logtext + strlen(logtext), m->descriptor);
-
-       if (m->flags & ACC_PUBLIC)       sprintf(logtext + strlen(logtext), " PUBLIC");
-       if (m->flags & ACC_PRIVATE)      sprintf(logtext + strlen(logtext), " PRIVATE");
-       if (m->flags & ACC_PROTECTED)    sprintf(logtext + strlen(logtext), " PROTECTED");
-       if (m->flags & ACC_STATIC)       sprintf(logtext + strlen(logtext), " STATIC");
-       if (m->flags & ACC_FINAL)        sprintf(logtext + strlen(logtext), " FINAL");
-       if (m->flags & ACC_SYNCHRONIZED) sprintf(logtext + strlen(logtext), " SYNCHRONIZED");
-       if (m->flags & ACC_VOLATILE)     sprintf(logtext + strlen(logtext), " VOLATILE");
-       if (m->flags & ACC_TRANSIENT)    sprintf(logtext + strlen(logtext), " TRANSIENT");
-       if (m->flags & ACC_NATIVE)       sprintf(logtext + strlen(logtext), " NATIVE");
-       if (m->flags & ACC_INTERFACE)    sprintf(logtext + strlen(logtext), " INTERFACE");
-       if (m->flags & ACC_ABSTRACT)     sprintf(logtext + strlen(logtext), " ABSTRACT");
+/* builtin_displaymethodstop ***************************************************
 
-       log_text(logtext);
-       methodindent++;
-}
+   XXX
 
+*******************************************************************************/
 
 void builtin_displaymethodstop(methodinfo *m, s8 l, double d, float f)
 {
-       int i;
-       char logtext[MAXLOGTEXT];
+       methoddesc *md;
+       char       *logtext;
+       s4          logtextlen;
+       s4          dumpsize;
+       s4          i;
+       imm_union   imu;
+
+       md = m->parseddesc;
+
+       /* calculate message length */
+
+       logtextlen =
+               methodindent + strlen("finished: ") +
+               utf_strlen(m->class->name) +
+               strlen(".") +
+               utf_strlen(m->name) +
+               utf_strlen(m->descriptor) +
+               strlen(" SYNCHRONIZED") + strlen("(") + strlen(")");
+
+       /* add maximal argument length */
+
+       logtextlen += strlen("->0.4872328470301428 (0x0123456789abcdef)");
+
+       /* allocate memory */
+
+       dumpsize = dump_size();
+
+       logtext = DMNEW(char, logtextlen);
+
+       /* generate the message */
+
        for (i = 0; i < methodindent; i++)
                logtext[i] = '\t';
+
        if (methodindent)
                methodindent--;
        else
                log_text("WARNING: unmatched methodindent--");
 
-       sprintf(logtext + methodindent, "finished: ");
-       utf_sprint_classname(logtext + strlen(logtext), m->class->name);
-       sprintf(logtext + strlen(logtext), ".");
-       utf_sprint(logtext + strlen(logtext), m->name);
-       utf_sprint_classname(logtext + strlen(logtext), m->descriptor);
+       strcpy(logtext + methodindent, "finished: ");
+       utf_strcat_classname(logtext, m->class->name);
+       strcat(logtext, ".");
+       utf_strcat(logtext, m->name);
+       utf_strcat(logtext, m->descriptor);
 
-       switch (m->returntype) {
+       switch (md->returntype.type) {
        case TYPE_INT:
-               sprintf(logtext + strlen(logtext), "->%d", (s4) l);
+               sprintf(logtext + strlen(logtext), "->%d (0x%08x)", (s4) l, (s4) l);
                break;
 
-       case TYPE_LONG:
-#if defined(__I386__) || defined(__POWERPC__)
-               sprintf(logtext + strlen(logtext), "->%lld", (s8) l);
+       case TYPE_LNG:
+#if SIZEOF_VOID_P == 4
+               sprintf(logtext + strlen(logtext), "->%lld (0x%016llx)", (s8) l, l);
 #else
-               sprintf(logtext + strlen(logtext), "->%ld", (s8) l);
+               sprintf(logtext + strlen(logtext), "->%ld (0x%016lx)", (s8) l, l);
 #endif
                break;
 
-       case TYPE_ADDRESS:
-#if defined(__I386__) || defined(__POWERPC__)
-               sprintf(logtext + strlen(logtext), "->%p", (u1*) ((s4) l));
-#else
-               sprintf(logtext + strlen(logtext), "->%p", (u1*) l);
-#endif
+       case TYPE_ADR:
+               sprintf(logtext + strlen(logtext), "->%p", (void *) (ptrint) l);
                break;
 
-       case TYPE_FLOAT:
-               sprintf(logtext + strlen(logtext), "->%g", f);
+       case TYPE_FLT:
+               imu.f = f;
+               sprintf(logtext + strlen(logtext), "->%.8f (0x%08x)", f, imu.i);
                break;
 
-       case TYPE_DOUBLE:
-               sprintf(logtext + strlen(logtext), "->%g", d);
+       case TYPE_DBL:
+               imu.d = d;
+#if SIZEOF_VOID_P == 4
+               sprintf(logtext + strlen(logtext), "->%.16g (0x%016llx)", d, imu.l);
+#else
+               sprintf(logtext + strlen(logtext), "->%.16g (0x%016lx)", d, imu.l);
+#endif
                break;
        }
+
        log_text(logtext);
+
+       /* release memory */
+
+       dump_release(dumpsize);
 }
 
 
@@ -1038,12 +1522,12 @@ void builtin_displaymethodstop(methodinfo *m, s8 l, double d, float f)
                         SYNCHRONIZATION FUNCTIONS
 *****************************************************************************/
 
+#if defined(USE_THREADS) && !defined(NATIVE_THREADS)
 /*
  * Lock the mutex of an object.
  */
 void internal_lock_mutex_for_object(java_objectheader *object)
 {
-#if defined(USE_THREADS) && !defined(NATIVE_THREADS)
        mutexHashEntry *entry;
        int hashValue;
 
@@ -1084,16 +1568,16 @@ void internal_lock_mutex_for_object(java_objectheader *object)
                entry->object = object;
        
        internal_lock_mutex(&entry->mutex);
-#endif
 }
+#endif
 
 
+#if defined(USE_THREADS) && !defined(NATIVE_THREADS)
 /*
  * Unlocks the mutex of an object.
  */
 void internal_unlock_mutex_for_object (java_objectheader *object)
 {
-#if defined(USE_THREADS) && !defined(NATIVE_THREADS)
        int hashValue;
        mutexHashEntry *entry;
 
@@ -1119,13 +1603,13 @@ void internal_unlock_mutex_for_object (java_objectheader *object)
                        firstFreeOverflowEntry = unlinked;
                }
        }
-#endif
 }
+#endif
 
 
+#if defined(USE_THREADS)
 void builtin_monitorenter(java_objectheader *o)
 {
-#if defined(USE_THREADS)
 #if !defined(NATIVE_THREADS)
        int hashValue;
 
@@ -1142,10 +1626,11 @@ void builtin_monitorenter(java_objectheader *o)
 #else
        monitorEnter((threadobject *) THREADOBJECT, o);
 #endif
+}
 #endif
 
-}
 
+#if defined(USE_THREADS)
 /*
  * Locks the class object - needed for static synchronized methods.
  * The use_class_as_object call is needed in order to circumvent a
@@ -1157,11 +1642,12 @@ void builtin_staticmonitorenter(classinfo *c)
        use_class_as_object(c);
        builtin_monitorenter(&c->header);
 }
+#endif
 
 
-void *builtin_monitorexit(java_objectheader *o)
-{
 #if defined(USE_THREADS)
+void builtin_monitorexit(java_objectheader *o)
+{
 #if !defined(NATIVE_THREADS)
        int hashValue;
 
@@ -1179,13 +1665,11 @@ void *builtin_monitorexit(java_objectheader *o)
                internal_unlock_mutex_for_object(o);
 
        --blockInts;
-       return o;
 #else
        monitorExit((threadobject *) THREADOBJECT, o);
-       return o;
-#endif
 #endif
 }
+#endif
 
 
 /*****************************************************************************
@@ -1212,113 +1696,160 @@ s4 builtin_irem(s4 a, s4 b) { return a % b; }
 
 ******************************************************************************/
 
-
 s8 builtin_ladd(s8 a, s8 b)
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a + b; 
+       c = a + b; 
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_lsub(s8 a, s8 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a - b; 
+       c = a - b; 
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_lmul(s8 a, s8 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a * b; 
+       c = a * b; 
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_ldiv(s8 a, s8 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a / b; 
+       c = a / b; 
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_lrem(s8 a, s8 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a % b; 
+       c = a % b; 
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_lshl(s8 a, s4 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a << (b & 63);
+       c = a << (b & 63);
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_lshr(s8 a, s4 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a >> (b & 63);
+       c = a >> (b & 63);
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_lushr(s8 a, s4 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return ((u8) a) >> (b & 63);
+       c = ((u8) a) >> (b & 63);
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_land(s8 a, s8 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a & b; 
+       c = a & b; 
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_lor(s8 a, s8 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a | b; 
+       c = a | b; 
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_lxor(s8 a, s8 b) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return a ^ b; 
+       c = a ^ b; 
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s8 builtin_lneg(s8 a) 
-{ 
+{
+       s8 c;
+
 #if U8_AVAILABLE
-       return -a;
+       c = -a;
 #else
-       return builtin_i2l(0);
+       c = builtin_i2l(0);
 #endif
+
+       return c;
 }
 
 s4 builtin_lcmp(s8 a, s8 b) 
@@ -1390,19 +1921,64 @@ float builtin_fmul(float a, float b)
 }
 
 
+/* builtin_ddiv ****************************************************************
+
+   Implementation as described in VM Spec.
+
+*******************************************************************************/
+
 float builtin_fdiv(float a, float b)
 {
-       if (finitef(a) && finitef(b)) {
-               if (b != 0)
+       if (finitef(a)) {
+               if (finitef(b)) {
+                       /* If neither value1' nor value2' is NaN, the sign of the result */
+                       /* is positive if both values have the same sign, negative if the */
+                       /* values have different signs. */
+
                        return a / b;
-               else {
-                       if (a > 0)
+
+               } else {
+                       if (isnanf(b)) {
+                               /* If either value1' or value2' is NaN, the result is NaN. */
+
+                               return intBitsToFloat(FLT_NAN);
+
+                       } else {
+                               /* Division of a finite value by an infinity results in a */
+                               /* signed zero, with the sign-producing rule just given. */
+
+                               /* is sign equal? */
+
+                               if (copysignf(1.0, a) == copysignf(1.0, b))
+                                       return 0.0;
+                               else
+                                       return -0.0;
+                       }
+               }
+
+       } else {
+               if (isnanf(a)) {
+                       /* If either value1' or value2' is NaN, the result is NaN. */
+
+                       return intBitsToFloat(FLT_NAN);
+
+               } else if (finitef(b)) {
+                       /* Division of an infinity by a finite value results in a signed */
+                       /* infinity, with the sign-producing rule just given. */
+
+                       /* is sign equal? */
+
+                       if (copysignf(1.0, a) == copysignf(1.0, b))
                                return intBitsToFloat(FLT_POSINF);
-                       else if (a < 0)
+                       else
                                return intBitsToFloat(FLT_NEGINF);
+
+               } else {
+                       /* Division of an infinity by an infinity results in NaN. */
+
+                       return intBitsToFloat(FLT_NAN);
                }
        }
-       return intBitsToFloat(FLT_NAN);
 }
 
 
@@ -1500,43 +2076,64 @@ double builtin_dmul(double a, double b)
 }
 
 
+/* builtin_ddiv ****************************************************************
+
+   Implementation as described in VM Spec.
+
+*******************************************************************************/
+
 double builtin_ddiv(double a, double b)
 {
        if (finite(a)) {
                if (finite(b)) {
+                       /* If neither value1' nor value2' is NaN, the sign of the result */
+                       /* is positive if both values have the same sign, negative if the */
+                       /* values have different signs. */
+
                        return a / b;
 
                } else {
-                       if (isnan(b))
+                       if (isnan(b)) {
+                               /* If either value1' or value2' is NaN, the result is NaN. */
+
                                return longBitsToDouble(DBL_NAN);
-                       else
-                               return copysign(0.0, b);
+
+                       } else {
+                               /* Division of a finite value by an infinity results in a */
+                               /* signed zero, with the sign-producing rule just given. */
+
+                               /* is sign equal? */
+
+                               if (copysign(1.0, a) == copysign(1.0, b))
+                                       return 0.0;
+                               else
+                                       return -0.0;
+                       }
                }
 
        } else {
-               if (finite(b)) {
-                       if (a > 0)
+               if (isnan(a)) {
+                       /* If either value1' or value2' is NaN, the result is NaN. */
+
+                       return longBitsToDouble(DBL_NAN);
+
+               } else if (finite(b)) {
+                       /* Division of an infinity by a finite value results in a signed */
+                       /* infinity, with the sign-producing rule just given. */
+
+                       /* is sign equal? */
+
+                       if (copysign(1.0, a) == copysign(1.0, b))
                                return longBitsToDouble(DBL_POSINF);
-                       else if (a < 0)
+                       else
                                return longBitsToDouble(DBL_NEGINF);
 
-               } else
+               } else {
+                       /* Division of an infinity by an infinity results in NaN. */
+
                        return longBitsToDouble(DBL_NAN);
+               }
        }
-
-/*     if (finite(a) && finite(b)) { */
-/*             if (b != 0) */
-/*                     return a / b; */
-/*             else { */
-/*                     if (a > 0) */
-/*                             return longBitsToDouble(DBL_POSINF); */
-/*                     else if (a < 0) */
-/*                             return longBitsToDouble(DBL_NEGINF); */
-/*             } */
-/*     } */
-
-       /* keep compiler happy */
-       return 0;
 }
 
 
@@ -1545,13 +2142,33 @@ double builtin_drem(double a, double b)
        return fmod(a, b);
 }
 
+/* builtin_dneg ****************************************************************
+
+   Implemented as described in VM Spec.
+
+*******************************************************************************/
 
 double builtin_dneg(double a)
 {
-       if (isnan(a)) return a;
-       else {
-               if (finite(a)) return -a;
-               else return copysign(a, -copysign(1.0, a));
+       if (isnan(a)) {
+               /* If the operand is NaN, the result is NaN (recall that NaN has no */
+               /* sign). */
+
+               return a;
+
+       } else {
+               if (finite(a)) {
+                       /* If the operand is a zero, the result is the zero of opposite */
+                       /* sign. */
+
+                       return -a;
+
+               } else {
+                       /* If the operand is an infinity, the result is the infinity of */
+                       /* opposite sign. */
+
+                       return copysign(a, -copysign(1.0, a));
+               }
        }
 }
 
@@ -1647,8 +2264,11 @@ double builtin_l2d(s8 a)
 
 s4 builtin_f2i(float a) 
 {
+       s4 i;
 
-       return builtin_d2i((double) a);
+       i = builtin_d2i((double) a);
+
+       return i;
 
        /*      float f;
        
@@ -1670,8 +2290,11 @@ s4 builtin_f2i(float a)
 
 s8 builtin_f2l(float a)
 {
+       s8 l;
+
+       l = builtin_d2l((double) a);
 
-       return builtin_d2l((double) a);
+       return l;
 
        /*      float f;
        
@@ -1778,17 +2401,22 @@ inline float longBitsToDouble(s8 l)
 }
 
 
+/* builtin_clone_array *********************************************************
+
+   Wrapper function for cloning arrays.
+
+*******************************************************************************/
+
 java_arrayheader *builtin_clone_array(void *env, java_arrayheader *o)
 {
-       return (java_arrayheader *)
-               Java_java_lang_VMObject_clone(0, 0, (java_lang_Cloneable *) o);
-}
+       java_arrayheader    *ah;
+       java_lang_Cloneable *c;
 
+       c = (java_lang_Object *) o;
 
-s4 builtin_dummy()
-{
-       panic("Internal error: builtin_dummy called (native function is missing)");
-       return 0; /* for the compiler */
+       ah = (java_arrayheader *) Java_java_lang_VMObject_clone(0, 0, c);
+
+       return ah;
 }
 
 
@@ -1799,33 +2427,35 @@ s4 builtin_dummy()
 *******************************************************************************/
 
 #if defined(USE_THREADS) && defined(NATIVE_THREADS)
-java_objectheader **builtin_asm_get_exceptionptrptr()
+java_objectheader **builtin_asm_get_exceptionptrptr(void)
 {
        return builtin_get_exceptionptrptr();
 }
 #endif
 
 
-methodinfo *builtin_asm_get_threadrootmethod()
+methodinfo *builtin_asm_get_threadrootmethod(void)
 {
        return *threadrootmethod;
 }
 
 
-inline void* builtin_asm_get_stackframeinfo()
+void *builtin_asm_get_stackframeinfo(void)
 {
-/*log_text("builtin_asm_get_stackframeinfo()");*/
 #if defined(USE_THREADS) && defined(NATIVE_THREADS)
        return &THREADINFO->_stackframeinfo;
 #else
-#if defined(__GNUC__)
-#warning FIXME FOR OLD THREAD IMPL (jowenn)
-#endif
-               return &_thread_nativestackframeinfo; /* no threading, at least no native*/
+       /* XXX FIXME FOR OLD THREAD IMPL (jowenn) */
+
+       return &_thread_nativestackframeinfo; /* no threading, at least no native*/
 #endif
 }
 
-stacktraceelement *builtin_stacktrace_copy(stacktraceelement **el,stacktraceelement *begin, stacktraceelement *end) {
+
+stacktraceelement *builtin_stacktrace_copy(stacktraceelement **el,
+                                                                                  stacktraceelement *begin,
+                                                                                  stacktraceelement *end)
+{
 /*     stacktraceelement *el;*/
        size_t s;
        s=(end-begin);
@@ -1836,13 +2466,15 @@ stacktraceelement *builtin_stacktrace_copy(stacktraceelement **el,stacktraceelem
 #endif
        memcpy(*el,begin,(end-begin)*sizeof(stacktraceelement));
        (*el)[s].method=0;
-#if defined(__GNUC__)
-#warning change this if line numbers bigger than u2 are allowed, the currently supported class file format does no allow that
-#endif
+
+       /* XXX change this if line numbers bigger than u2 are allowed, the */
+       /* currently supported class file format does no allow that */
+
        (*el)[s].linenumber=-1; /* -1 can never be reched otherwise, since line numbers are only u2, so it is save to use that as flag */
        return *el;
 }
 
+
 /*
  * These are local overrides for various environment variables in Emacs.
  * Please do not remove this and leave it at the end of the file, where