[Mono-dev] CoreCLR patches

Mark Probst mark.probst at gmail.com
Tue Mar 17 05:58:36 EDT 2009


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


More information about the Mono-devel-list mailing list