Hi Sebastien,

Apart from what Rodrigo said I only two tiny issues with the patch.
First it would be nice if you could split it up between the part that
handles field access, the part that handles wrappers and the part that
handles the image stuff.  And then there's this:

+       for (i = 0; i < 11; i++) {
+               if (strstr (image->name, platform_code_assemblies[i]))
+                       return TRUE;
+       }

Please don't hard-code the number of elements in the array :-)

Mark
_______________________________________________
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list

Reply via email to