Re: [Mono-dev] [Patch] fix for #82022

2007-07-10 Thread Rodrigo Kumpera
Hi Paolo, Shouldn't the other case that returns NULL in mono_class_create_from_typedef use mono_class_set_failure as well? From now on I'll send patches using .diff extension. Thanks for the review! Rodrigo On 7/9/07, Paolo Molaro [EMAIL PROTECTED] wrote: On 07/06/07 Rodrigo Kumpera wrote:

Re: [Mono-dev] [Patch] fix for #82022

2007-07-09 Thread Paolo Molaro
On 07/06/07 Rodrigo Kumpera wrote: The attached patch fixes #82022. The fix is quite simple, it just checks if the enum field is null and abort the loading process. But there are two issues with this patch, first it could return a more informative message about the problem and I'm not sure if

[Mono-dev] [Patch] fix for #82022

2007-07-06 Thread Rodrigo Kumpera
Hey guys, The attached patch fixes #82022. The fix is quite simple, it just checks if the enum field is null and abort the loading process. But there are two issues with this patch, first it could return a more informative message about the problem and I'm not sure if it won't leak memory or