[Mono-dev] XmlTextReader.cs patch (IBM bug workaround)

[Atsushi Eno]

I don't think we should accept such patches. Such a JIT bug will
happen to *every* classes in every libraries, and EVEN IF we
change the every code like this, issues will still happen on
any third-party libraries. Do you think you can ask everyone
to change his or her managed code just for this reason?

Atsushi Eno


Sasha Kogan wrote:
> 
> 
>  
> 
>  
> 
> ** /Sasha **
> 
> * From: * Sasha Kogan
> *Sent:* Monday, April 16, 2007 4:14 PM
> *To:* 'mono-devel-list at lists.ximian.com'
> *Subject:* XmlTextReader.cs patch (IBM bug workaround)
> 
>  
> 
> Please review the attached patch for XmlTextReader.cs. The patch is 
> actually a workaround for an IBM bug (seems to be a JIT compiler bug) 
> that causes java.lang.AbstractMethodError to be thrown upon access of 
> some virtual methods/properties within the class.
> 
> The workaround is to access such methods/properties indirectly, via the 
> introduced ‘This’ property that just returns ‘this’ reference.
> 
>  
> 
> Alternativelly, we could wrap the patch in “#if TARGET_JVM”, but since 
> there’re numerous places in the code where it should be applied, this 
> approach would dirty the code too much.
> 
>  
> 
> ** /Sasha **
> 
>  
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list