(mono_runtime_delegate_invoke): impl.
[mono.git] / mono / metadata / class.c
index f5ceebf1035f618c168f36d759f4a4c21d231cbe..7de930864c2e3fb36b661b3409a7e5922c9df550 100644 (file)
@@ -38,49 +38,8 @@ gboolean mono_print_vtable = FALSE;
 
 static MonoClass * mono_class_create_from_typedef (MonoImage *image, guint32 type_token);
 
-static gpointer
-default_trampoline (MonoMethod *method)
-{
-       return method;
-}
-
-static gpointer
-default_remoting_trampoline (MonoMethod *method)
-{
-       g_error ("remoting not installed");
-       return NULL;
-}
-
-static void
-default_runtime_class_init (MonoClass *klass)
-{
-       return;
-}
-
-static MonoTrampoline arch_create_jit_trampoline = default_trampoline;
-static MonoTrampoline arch_create_remoting_trampoline = default_remoting_trampoline;
-static MonoRuntimeClassInit mono_runtime_class_init = default_runtime_class_init;
-
-void
-mono_install_trampoline (MonoTrampoline func) 
-{
-       arch_create_jit_trampoline = func? func: default_trampoline;
-}
-
-void
-mono_install_remoting_trampoline (MonoTrampoline func) 
-{
-       arch_create_remoting_trampoline = func? func: default_remoting_trampoline;
-}
-
-void
-mono_install_runtime_class_init (MonoRuntimeClassInit func)
-{
-       mono_runtime_class_init = func? func: default_runtime_class_init;
-}
-
-static MonoClass *
-mono_class_create_from_typeref (MonoImage *image, guint32 type_token)
+MonoClass *
+mono_class_from_typeref (MonoImage *image, guint32 type_token)
 {
        guint32 cols [MONO_TYPEREF_SIZE];
        MonoTableInfo  *t = &image->tables [MONO_TABLE_TYPEREF];
@@ -89,8 +48,24 @@ mono_class_create_from_typeref (MonoImage *image, guint32 type_token)
        MonoClass *res;
 
        mono_metadata_decode_row (t, (type_token&0xffffff)-1, cols, MONO_TYPEREF_SIZE);
-       g_assert ((cols [MONO_TYPEREF_SCOPE] & 0x3) == 2);
-       idx = cols [MONO_TYPEREF_SCOPE] >> 2;
+
+       name = mono_metadata_string_heap (image, cols [MONO_TYPEREF_NAME]);
+       nspace = mono_metadata_string_heap (image, cols [MONO_TYPEREF_NAMESPACE]);
+       
+       idx = cols [MONO_TYPEREF_SCOPE] >> RESOLTION_SCOPE_BITS;
+       switch (cols [MONO_TYPEREF_SCOPE] & RESOLTION_SCOPE_MASK) {
+       case RESOLTION_SCOPE_MODULE:
+               if (!idx)
+                       g_error ("null ResolutionScope not yet handled");
+               /* a typedef in disguise */
+               return mono_class_from_name (image, nspace, name);
+       case RESOLTION_SCOPE_MODULEREF:
+                       g_error ("ModuleRef ResolutionScope not yet handled");
+       case RESOLTION_SCOPE_TYPEREF:
+                       g_error ("TypeRef ResolutionScope not yet handled");
+       case RESOLTION_SCOPE_ASSEMBLYREF:
+               break;
+       }
 
        if (!image->references ||  !image->references [idx-1]) {
                /* 
@@ -107,9 +82,6 @@ mono_class_create_from_typeref (MonoImage *image, guint32 type_token)
                return res;
        }       
 
-       name = mono_metadata_string_heap (image, cols [MONO_TYPEREF_NAME]);
-       nspace = mono_metadata_string_heap (image, cols [MONO_TYPEREF_NAMESPACE]);
-       
        /* load referenced assembly */
        image = image->references [idx-1]->image;
 
@@ -134,6 +106,7 @@ class_compute_field_layout (MonoClass *class)
        guint32 layout = class->flags & TYPE_ATTRIBUTE_LAYOUT_MASK;
        MonoTableInfo *t = &m->tables [MONO_TABLE_FIELD];
        int i;
+       guint32 rva;
 
        /*
         * Fetch all the field information.
@@ -153,9 +126,10 @@ class_compute_field_layout (MonoClass *class)
                class->fields [i].type = mono_metadata_parse_field_type (
                        m, cols [MONO_FIELD_FLAGS], sig + 1, &sig);
                if (cols [MONO_FIELD_FLAGS] & FIELD_ATTRIBUTE_HAS_FIELD_RVA) {
-                       mono_metadata_field_info (m, idx, NULL, &class->fields [i].data, NULL);
-                       if (!class->fields [i].data)
+                       mono_metadata_field_info (m, idx, NULL, &rva, NULL);
+                       if (!rva)
                                g_warning ("field %s in %s should have RVA data, but hasn't", class->fields [i].name, class->name);
+                       class->fields [i].data = mono_cli_rva_map (class->image->image_info, rva);
                }
                if (class->enumtype && !(cols [MONO_FIELD_FLAGS] & FIELD_ATTRIBUTE_STATIC)) {
                        class->enum_basetype = class->fields [i].type;
@@ -380,10 +354,13 @@ void
 mono_class_init (MonoClass *class)
 {
        MonoClass *k, *ic;
-       MonoMethod **vtable = class->vtable;
-       int i, max_iid, cur_slot = 0;
+       MonoMethod **vtable;
+       int i, max_vtsize = 0, max_iid, cur_slot = 0;
        static MonoMethod *default_ghc = NULL;
+       static MonoMethod *default_finalize = NULL;
+       static int finalize_slot = -1;
        static int ghc_slot = -1;
+       guint32 packing_size = 0;
 
        g_assert (class);
 
@@ -397,6 +374,8 @@ mono_class_init (MonoClass *class)
 
        class->init_pending = 1;
 
+//     mono_stats.initialized_class_count++;
+
        if (class->parent) {
                if (!class->parent->inited)
                        mono_class_init (class->parent);
@@ -407,6 +386,10 @@ mono_class_init (MonoClass *class)
        } else
                class->min_align = 1;
 
+       if (mono_metadata_packing_from_typedef (class->image, class->type_token, &packing_size, &class->instance_size))
+               class->instance_size += sizeof (MonoObject);
+       /* use packing_size in field layout */
+
        /*
         * Computes the size used by the fields, and their locations
         */
@@ -415,12 +398,40 @@ mono_class_init (MonoClass *class)
                class_compute_field_layout (class);
        }
 
+       if (!(class->flags & TYPE_ATTRIBUTE_INTERFACE)) {
+               for (i = 0; i < class->interface_count; i++) 
+                       max_vtsize += class->interfaces [i]->method.count;
+       
+               if (class->parent)
+                       max_vtsize += class->parent->vtable_size;
+
+               max_vtsize += class->method.count;
+       }
+
+       vtable = alloca (sizeof (gpointer) * max_vtsize);
+       memset (vtable, 0, sizeof (gpointer) * max_vtsize);
+
        /* initialize method pointers */
        class->methods = g_new (MonoMethod*, class->method.count);
        for (i = 0; i < class->method.count; ++i)
                class->methods [i] = mono_get_method (class->image,
                        MONO_TOKEN_METHOD_DEF | (i + class->method.first + 1), class);
 
+       init_properties (class);
+       init_events (class);
+
+       i = mono_metadata_nesting_typedef (class->image, class->type_token);
+       while (i) {
+               MonoClass* nclass;
+               guint32 cols [MONO_NESTED_CLASS_SIZE];
+               mono_metadata_decode_row (&class->image->tables [MONO_TABLE_NESTEDCLASS], i - 1, cols, MONO_NESTED_CLASS_SIZE);
+               if (cols [MONO_NESTED_CLASS_ENCLOSING] != mono_metadata_token_index (class->type_token))
+                       break;
+               nclass = mono_class_create_from_typedef (class->image, MONO_TOKEN_TYPE_DEF | cols [MONO_NESTED_CLASS_NESTED]);
+               class->nested_classes = g_list_prepend (class->nested_classes, nclass);
+               ++i;
+       }
+
        if (class->flags & TYPE_ATTRIBUTE_INTERFACE) {
                for (i = 0; i < class->method.count; ++i)
                        class->methods [i]->slot = i;
@@ -465,7 +476,7 @@ mono_class_init (MonoClass *class)
                                int io = k->interface_offsets [ic->interface_id];
 
                                g_assert (io >= 0);
-                               g_assert (io <= class->vtable_size);
+                               g_assert (io <= max_vtsize);
 
                                class->interface_offsets [ic->interface_id] = io;
                        }
@@ -473,7 +484,7 @@ mono_class_init (MonoClass *class)
        }
 
        if (class->parent && class->parent->vtable_size)
-               memcpy (class->vtable, class->parent->vtable,  sizeof (gpointer) * class->parent->vtable_size);
+               memcpy (vtable, class->parent->vtable,  sizeof (gpointer) * class->parent->vtable_size);
  
        for (k = class; k ; k = k->parent) {
                for (i = 0; i < k->interface_count; i++) {
@@ -483,7 +494,7 @@ mono_class_init (MonoClass *class)
                        io = k->interface_offsets [ic->interface_id];
                        
                        g_assert (io >= 0);
-                       g_assert (io <= class->vtable_size);
+                       g_assert (io <= max_vtsize);
 
                        if (k == class) {
                                for (l = 0; l < ic->method.count; l++) {
@@ -496,7 +507,7 @@ mono_class_init (MonoClass *class)
                                                        continue;
                                                if (!strcmp(cm->name, im->name) && 
                                                    mono_metadata_signature_equal (cm->signature, im->signature)) {
-                                                       g_assert (io + l <= class->vtable_size);
+                                                       g_assert (io + l <= max_vtsize);
                                                        vtable [io + l] = cm;
                                                }
                                        }
@@ -511,7 +522,7 @@ mono_class_init (MonoClass *class)
                                MonoMethod *im = ic->methods [l];                                               
                                MonoClass *k1;
 
-                               g_assert (io + l <= class->vtable_size);
+                               g_assert (io + l <= max_vtsize);
 
                                if (vtable [io + l])
                                        continue;
@@ -526,13 +537,13 @@ mono_class_init (MonoClass *class)
                                                
                                                if (!strcmp(cm->name, im->name) && 
                                                    mono_metadata_signature_equal (cm->signature, im->signature)) {
-                                                       g_assert (io + l <= class->vtable_size);
+                                                       g_assert (io + l <= max_vtsize);
                                                        vtable [io + l] = cm;
                                                        break;
                                                }
                                                
                                        }
-                                       g_assert (io + l <= class->vtable_size);
+                                       g_assert (io + l <= max_vtsize);
                                        if (vtable [io + l])
                                                break;
                                }
@@ -556,7 +567,7 @@ mono_class_init (MonoClass *class)
                                        
                                        if (((fqname && !strcmp (cm->name, fqname)) || !strcmp (cm->name, qname)) &&
                                            mono_metadata_signature_equal (cm->signature, im->signature)) {
-                                               g_assert (io + l <= class->vtable_size);
+                                               g_assert (io + l <= max_vtsize);
                                                vtable [io + l] = cm;
                                                break;
                                        }
@@ -569,7 +580,7 @@ mono_class_init (MonoClass *class)
                        if (!(class->flags & TYPE_ATTRIBUTE_ABSTRACT)) {
                                for (l = 0; l < ic->method.count; l++) {
                                        MonoMethod *im = ic->methods [l];                                               
-                                       g_assert (io + l <= class->vtable_size);
+                                       g_assert (io + l <= max_vtsize);
                                        if (!(vtable [io + l])) {
                                                printf ("no implementation for interface method %s.%s::%s in class %s.%s\n",
                                                        ic->name_space, ic->name, im->name, class->name_space, class->name);
@@ -588,7 +599,7 @@ mono_class_init (MonoClass *class)
                                MonoMethod *im = vtable [io + l];
 
                                if (im) {
-                                       g_assert (io + l <= class->vtable_size);
+                                       g_assert (io + l <= max_vtsize);
                                        if (im->slot < 0) {
                                                // fixme: why do we need this ?
                                                im->slot = io + l;
@@ -604,11 +615,17 @@ mono_class_init (MonoClass *class)
               
                cm = class->methods [i];
 
+               
+#if 0
                if (!(cm->flags & METHOD_ATTRIBUTE_VIRTUAL) ||
                    (cm->slot >= 0))
                        continue;
-               
-               if (!(cm->flags & METHOD_ATTRIBUTE_NEW_SLOT)) {
+#else  /* EXT_VTABLE_HACK */
+               if (cm->slot >= 0)
+                       continue;
+#endif
+
+               if (!(cm->flags & METHOD_ATTRIBUTE_NEW_SLOT) && (cm->flags & METHOD_ATTRIBUTE_VIRTUAL)) {
                        for (k = class->parent; k ; k = k->parent) {
                                int j;
                                for (j = 0; j < k->method.count; ++j) {
@@ -618,7 +635,7 @@ mono_class_init (MonoClass *class)
                                        if (!strcmp(cm->name, m1->name) && 
                                            mono_metadata_signature_equal (cm->signature, m1->signature)) {
                                                cm->slot = k->methods [j]->slot;
-                                               g_assert (cm->slot < class->vtable_size);
+                                               g_assert (cm->slot < max_vtsize);
                                                break;
                                        }
                                }
@@ -634,20 +651,13 @@ mono_class_init (MonoClass *class)
                        vtable [cm->slot] = cm;
        }
 
-       init_properties (class);
-       init_events (class);
 
-       i = mono_metadata_nesting_typedef (class->image, class->type_token);
-       while (i) {
-               MonoClass* nclass;
-               guint32 cols [MONO_NESTED_CLASS_SIZE];
-               mono_metadata_decode_row (&class->image->tables [MONO_TABLE_NESTEDCLASS], i - 1, cols, MONO_NESTED_CLASS_SIZE);
-               if (cols [MONO_NESTED_CLASS_ENCLOSING] != mono_metadata_token_index (class->type_token))
-                       break;
-               nclass = mono_class_create_from_typedef (class->image, MONO_TOKEN_TYPE_DEF | cols [MONO_NESTED_CLASS_NESTED]);
-               class->nested_classes = g_list_prepend (class->nested_classes, nclass);
-               ++i;
-       }
+       class->vtable_size = cur_slot;
+       class->vtable = g_malloc0 (sizeof (gpointer) * class->vtable_size);
+       memcpy (class->vtable, vtable,  sizeof (gpointer) * class->vtable_size);
+
+       class->inited = 1;
+       class->init_pending = 0;
 
        if (mono_print_vtable) {
                int icount = 0;
@@ -693,9 +703,6 @@ mono_class_init (MonoClass *class)
                }
        }
 
-       class->inited = 1;
-       class->init_pending = 0;
-
        if (!default_ghc) {
                if (class == mono_defaults.object_class) { 
                       
@@ -720,205 +727,32 @@ mono_class_init (MonoClass *class)
                if (vtable [ghc_slot] == default_ghc) {
                        class->ghcimpl = 0;
                }
-
        }
-}
-
-#if HAVE_BOEHM_GC
-static void
-vtable_finalizer (void *obj, void *data) {
-       g_print ("%s finalized (%p)\n", (char*)data, obj);
-}
-#endif
 
-/**
- * mono_class_vtable:
- * @domain: the application domain
- * @class: the class to initialize
- *
- * VTables are domain specific because we create domain specific code, and 
- * they contain the domain specific static class data.
- */
-MonoVTable *
-mono_class_vtable (MonoDomain *domain, MonoClass *class)
-{
-       MonoClass *k;
-       MonoVTable *vt;
-       MonoClassField *field;
-       guint32 cindex;
-       guint32 cols [MONO_CONSTANT_SIZE];
-       const char *p;
-       char *t;
-       int i, len;
-
-       g_assert (class);
-
-       /* can interfaces have static fields? */
-       if (class->flags & TYPE_ATTRIBUTE_INTERFACE)
-               g_assert_not_reached ();
-
-       if ((vt = mono_g_hash_table_lookup (domain->class_vtable_hash, class)))
-               return vt;
-       
-       if (!class->inited)
-               mono_class_init (class);
-               
-#if HAVE_BOEHM_GC
-       vt = GC_debug_malloc (sizeof (MonoVTable) + class->vtable_size * sizeof (gpointer), class->name, 1);
-       GC_register_finalizer (vt, vtable_finalizer, "vtable", NULL, NULL);
-#else
-       vt = g_malloc0 (sizeof (MonoVTable) + class->vtable_size * sizeof (gpointer));
-#endif
-       vt->klass = class;
-       vt->domain = domain;
-
-       if (class->class_size) {
-#if HAVE_BOEHM_GC
-               vt->data = GC_debug_malloc (class->class_size + 8, class->name, 2);
-               GC_register_finalizer (vt->data, vtable_finalizer, class->name, NULL, NULL);
-#else
-               vt->data = g_malloc0 (class->class_size + 8);
-#endif
-               /* align: fixme not 64 bit clean */
-               if (((guint32)vt->data) & 0x7)
-                       vt->data = (char*)vt->data + 8 - (((guint32)vt->data) & 0x7);
-       }
-
-       for (i = class->field.first; i < class->field.last; ++i) {
-               field = &class->fields [i - class->field.first];
-               if (!(field->type->attrs & FIELD_ATTRIBUTE_STATIC))
-                       continue;
-               if (!(field->type->attrs & FIELD_ATTRIBUTE_HAS_DEFAULT))
-                       continue;
-               cindex = mono_metadata_get_constant_index (class->image, MONO_TOKEN_FIELD_DEF | (i + 1));
-               if (!cindex) {
-                       g_warning ("constant for field %s not found", field->name);
-                       continue;
-               }
-               mono_metadata_decode_row (&class->image->tables [MONO_TABLE_CONSTANT], cindex - 1, cols, MONO_CONSTANT_SIZE);
-               p = mono_metadata_blob_heap (class->image, cols [MONO_CONSTANT_VALUE]);
-               len = mono_metadata_decode_blob_size (p, &p);
-               t = (char*)vt->data + field->offset;
-               /* should we check that the type matches? */
-               switch (cols [MONO_CONSTANT_TYPE]) {
-               case MONO_TYPE_BOOLEAN:
-               case MONO_TYPE_U1:
-               case MONO_TYPE_I1:
-                       *t = *p;
-                       break;
-               case MONO_TYPE_CHAR:
-               case MONO_TYPE_U2:
-               case MONO_TYPE_I2: {
-                       guint16 *val = (guint16*)t;
-                       *val = read16 (p);
-                       break;
-               }
-               case MONO_TYPE_U4:
-               case MONO_TYPE_I4: {
-                       guint32 *val = (guint32*)t;
-                       *val = read32 (p);
-                       break;
-               }
-               case MONO_TYPE_U8:
-               case MONO_TYPE_I8: {
-                       guint64 *val = (guint64*)t;
-                       *val = read64 (p);
-                       break;
-               }
-               case MONO_TYPE_R4: {
-                       float *val = (float*)t;
-                       readr4 (p, val);
-                       break;
-               }
-               case MONO_TYPE_R8: {
-                       double *val = (double*)t;
-                       readr8 (p, val);
-                       break;
-               }
-               case MONO_TYPE_STRING: {
-                       gpointer *val = (gpointer*)t;
-                       *val = mono_string_new_utf16 (domain, (const guint16*)p, len/2);
-                       break;
-               }
-               case MONO_TYPE_CLASS:
-                       /* nothing to do, we malloc0 the data and the value can be 0 only */
-                       break;
-               default:
-                       g_warning ("type 0x%02x should not be in constant table", cols [MONO_CONSTANT_TYPE]);
-               }
-       }
+       if (!default_finalize) {
+               if (class == mono_defaults.object_class) { 
+                      
+                       for (i = 0; i < class->vtable_size; ++i) {
+                               MonoMethod *cm = vtable [i];
+              
+                               if (!strcmp (cm->name, "Finalize")) {
+                                       finalize_slot = i;
+                                       break;
+                               }
+                       }
 
-       vt->interface_offsets = g_malloc0 (sizeof (gpointer) * (class->max_interface_id + 1));
+                       g_assert (finalize_slot > 0);
 
-       /* initialize interface offsets */
-       for (k = class; k ; k = k->parent) {
-               for (i = 0; i < k->interface_count; i++) {
-                       int slot;
-                       MonoClass *ic = k->interfaces [i];
-                       slot = class->interface_offsets [ic->interface_id];
-                       vt->interface_offsets [ic->interface_id] = &vt->vtable [slot];
+                       default_finalize = vtable [finalize_slot];
                }
        }
 
-       /* initialize vtable */
-       for (i = 0; i < class->vtable_size; ++i) {
-               MonoMethod *cm;
-              
-               if ((cm = class->vtable [i]))
-                       vt->vtable [i] = arch_create_jit_trampoline (cm);
-       }
-
-       mono_g_hash_table_insert (domain->class_vtable_hash, class, vt);
-
-       mono_runtime_class_init (class);
-
-       return vt;
-}
-
-/**
- * mono_class_proxy_vtable:
- * @domain: the application domain
- * @class: the class to proxy
- *
- * Creates a vtable for transparent proxies. It is basically
- * a copy of the real vtable of @class, but all function pointers invoke
- * the remoting functions, and vtable->klass points to the 
- * transparent proxy class, and not to @class.
- */
-MonoVTable *
-mono_class_proxy_vtable (MonoDomain *domain, MonoClass *class)
-{
-       MonoVTable *vt, *pvt;
-       int i, vtsize;
-
-       if ((pvt = mono_g_hash_table_lookup (domain->proxy_vtable_hash, class)))
-               return pvt;
-
-       vt = mono_class_vtable (domain, class);
-       vtsize = sizeof (MonoVTable) + class->vtable_size * sizeof (gpointer);
-
-#if HAVE_BOEHM_GC
-       pvt = GC_malloc (vtsize);
-       GC_register_finalizer (vt, vtable_finalizer, "vtable", NULL, NULL);
-#else
-       pvt = g_malloc0 (vtsize);
-#endif
-       
-       memcpy (pvt, vt, vtsize);
-
-       pvt->klass = mono_defaults.transparent_proxy_class;
-
-       /* initialize vtable */
-       for (i = 0; i < class->vtable_size; ++i) {
-               MonoMethod *cm;
-              
-               if ((cm = class->vtable [i]))
-                       pvt->vtable [i] = arch_create_remoting_trampoline (cm);
+       /* Object::Finalize should have empty implemenatation */
+       class->has_finalize = 0;
+       if (class != mono_defaults.object_class) { 
+               if (vtable [finalize_slot] != default_finalize)
+                       class->has_finalize = 1;
        }
-
-       mono_g_hash_table_insert (domain->proxy_vtable_hash, class, pvt);
-
-       return pvt;
 }
 
 
@@ -1049,11 +883,19 @@ mono_class_setup_parent (MonoClass *class, MonoClass *parent)
                class->instance_size = sizeof (MonoObject);
                return;
        }
+       if (!strcmp (class->name, "<Module>")) {
+               class->parent = NULL;
+               class->instance_size = 0;
+               return;
+       }
 
        if (!(class->flags & TYPE_ATTRIBUTE_INTERFACE)) {
                int rnum = 0;
                class->parent = parent;
 
+               if (!parent)
+                       g_assert_not_reached (); /* FIXME */
+
                class->marshalbyref = parent->marshalbyref;
                class->contextbound  = parent->contextbound;
                class->delegate  = parent->delegate;
@@ -1062,7 +904,7 @@ mono_class_setup_parent (MonoClass *class, MonoClass *parent)
                        if (*class->name == 'M' && !strcmp (class->name, "MarshalByRefObject"))
                                class->marshalbyref = 1;
 
-                       if (*class->name == 'M' && !strcmp (class->name, "ContextBoundObject")) 
+                       if (*class->name == 'C' && !strcmp (class->name, "ContextBoundObject")) 
                                class->contextbound  = 1;
 
                        if (*class->name == 'D' && !strcmp (class->name, "Delegate")) 
@@ -1097,49 +939,29 @@ mono_class_create_from_typedef (MonoImage *image, guint32 type_token)
        guint32 cols_next [MONO_TYPEDEF_SIZE];
        guint tidx = mono_metadata_token_index (type_token);
        const char *name, *nspace;
-       guint vtsize = 0, icount = 0; 
+       guint icount = 0; 
        MonoClass **interfaces;
-       int i;
 
        if ((class = g_hash_table_lookup (image->class_cache, GUINT_TO_POINTER (type_token))))
                return class;
 
        g_assert (mono_metadata_token_table (type_token) == MONO_TABLE_TYPEDEF);
        
-       mono_metadata_decode_row (tt, tidx - 1, cols, CSIZE (cols));
+       mono_metadata_decode_row (tt, tidx - 1, cols, MONO_TYPEDEF_SIZE);
        
-       if (tt->rows > tidx) {          
-               mono_metadata_decode_row (tt, tidx, cols_next, CSIZE (cols_next));
-               vtsize += cols_next [MONO_TYPEDEF_METHOD_LIST] - cols [MONO_TYPEDEF_METHOD_LIST];
-       } else {
-               vtsize += image->tables [MONO_TABLE_METHOD].rows - cols [MONO_TYPEDEF_METHOD_LIST] + 1;
-       }
+       name = mono_metadata_string_heap (image, cols [MONO_TYPEDEF_NAME]);
+       nspace = mono_metadata_string_heap (image, cols [MONO_TYPEDEF_NAMESPACE]);
 
-       name = mono_metadata_string_heap (image, cols[1]);
-       nspace = mono_metadata_string_heap (image, cols[2]);
-
-       if (!(!strcmp (nspace, "System") && !strcmp (name, "Object")) &&
-           !(cols [0] & TYPE_ATTRIBUTE_INTERFACE)) {
-               parent = mono_class_get (image, mono_metadata_token_from_dor (cols [3]));
-       }
+       if (cols [MONO_TYPEDEF_EXTENDS])
+               parent = mono_class_get (image, mono_metadata_token_from_dor (cols [MONO_TYPEDEF_EXTENDS]));
        interfaces = mono_metadata_interfaces_from_typedef (image, type_token, &icount);
 
-       for (i = 0; i < icount; i++) 
-               vtsize += interfaces [i]->method.count;
-       
-       if (parent)
-               vtsize += parent->vtable_size;
-
-       if (cols [0] & TYPE_ATTRIBUTE_INTERFACE)
-               vtsize = 0;
-
-       class = g_malloc0 (sizeof (MonoClass) + vtsize * sizeof (gpointer));
-       
+       class = g_malloc0 (sizeof (MonoClass));
+                          
        g_hash_table_insert (image->class_cache, GUINT_TO_POINTER (type_token), class);
 
        class->interfaces = interfaces;
        class->interface_count = icount;
-       class->vtable_size = vtsize;
 
        class->name = name;
        class->name_space = nspace;
@@ -1317,8 +1139,9 @@ mono_class_create_from_typespec (MonoImage *image, guint32 type_spec)
                class = mono_class_from_mono_type (type->data.type);
                break;
        default:
-               g_warning ("implement me: %08x", type->type);
-               g_assert_not_reached ();                
+               /* it seems any type can be stored in TypeSpec as well */
+               class = mono_class_from_mono_type (type);
+               break;
        }
 
        mono_metadata_free_type (type);
@@ -1340,27 +1163,27 @@ mono_array_class_get (MonoType *element_type, guint32 rank)
        MonoClass *eclass;
        MonoImage *image;
        MonoClass *class;
-       static MonoClass *parent = NULL;
-       guint32 key;
+       MonoClass *parent = NULL;
+       GSList *list;
        int rnum = 0;
 
        eclass = mono_class_from_mono_type (element_type);
        g_assert (rank <= 255);
 
-       if (!parent)
-               parent = mono_defaults.array_class;
+       parent = mono_defaults.array_class;
 
        if (!parent->inited)
                mono_class_init (parent);
 
        image = eclass->image;
 
-       g_assert (!eclass->type_token ||
-                 mono_metadata_token_table (eclass->type_token) == MONO_TABLE_TYPEDEF);
-
-       key = ((rank & 0xff) << 24) | (eclass->type_token & 0xffffff);
-       if ((class = g_hash_table_lookup (image->array_cache, GUINT_TO_POINTER (key))))
-               return class;
+       if ((list = g_hash_table_lookup (image->array_cache, element_type))) {
+               for (; list; list = list->next) {
+                       class = list->data;
+                       if (class->rank == rank)
+                               return class;
+               }
+       }
        
        class = g_malloc0 (sizeof (MonoClass) + parent->vtable_size * sizeof (gpointer));
 
@@ -1386,13 +1209,15 @@ mono_array_class_get (MonoType *element_type, guint32 rank)
                at->rank = rank;
                /* FIXME: complete.... */
        } else {
+               /* FIXME: this is not correct. the lbound could be >0 */
                class->byval_arg.type = MONO_TYPE_SZARRAY;
                class->byval_arg.data.type = &eclass->byval_arg;
        }
        class->this_arg = class->byval_arg;
        class->this_arg.byref = 1;
-       
-       g_hash_table_insert (image->array_cache, GUINT_TO_POINTER (key), class);
+
+       list = g_slist_append (list, class);
+       g_hash_table_insert (image->array_cache, &class->element_class->byval_arg, list);
        return class;
 }
 
@@ -1524,7 +1349,8 @@ mono_class_get (MonoImage *image, guint32 type_token)
                class = mono_class_create_from_typedef (image, type_token);
                break;          
        case MONO_TOKEN_TYPE_REF:
-               return mono_class_create_from_typeref (image, type_token);
+               class = mono_class_from_typeref (image, type_token);
+               break;
        case MONO_TOKEN_TYPE_SPEC:
                class = mono_class_create_from_typespec (image, type_token);
                break;
@@ -1533,6 +1359,8 @@ mono_class_get (MonoImage *image, guint32 type_token)
                g_assert_not_reached ();
        }
 
+       if (!class)
+               g_warning ("Could not load class from token 0x%08x in %s", type_token, image->name);
        return class;
 }