<br> Terry,<br><br> A bad commit from my part.<br> It shall work now.<br><br> Regards,<br> Edson<br><br><div><span class="gmail_quote">2007/4/16, Terry Laurenzo <<a href="mailto:tlaurenzo@rcode.net">tlaurenzo@rcode.net
</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">The following issue (737) was marked as closed, but in the current<br>SVN the affected code is commented out -- once again rendering Groovy
<br>fact classes inoperable. Can anyone explain why the patch was backed<br>out? If not, could we reopen JBRULES-737 as a regression or<br>uncomment the check for "<clinit>"?<br><br>Thanks.<br>Terry<br><br>
On Mar 14, 2007, at 3:56 PM, Edson Tirelli wrote:<br><br>><br>> Terry,<br>><br>> Thank you for reporting and providing the patch.<br>> Patch applied:<br>><br>> <a href="http://jira.jboss.com/jira/browse/JBRULES-737">
http://jira.jboss.com/jira/browse/JBRULES-737</a><br>><br>> []s<br>> Edson<br>><br>> Terry Laurenzo wrote:<br>><br>>> The attached diff (against 3.0.5 sources) adds a condition to the<br>>> ClassFieldInspector to not analyze methods named <clinit>. For
<br>>> some reason, the Groovy compiler is outputing classes that were<br>>> causing this check to choke and throw an IllegalArgumentException<br>>> because of a <clinit> method with ACC_PUBLIC access code.
<br>>><br>>> Applying this patch allows Drools to operate correctly with<br>>> compiled Groovy Fact classes.<br>>><br>>> This seems like a pretty innocuous change that someone should<br>>> apply to SVN. Unfortunately, it is rather difficult to create a
<br>>> test case for it.<br>>><br>>> Terry Laurenzo<br>>><br>>> ---------------------------------------------------------------------<br>>> ---<br>>><br>>> _______________________________________________
<br>>> rules-dev mailing list<br>>> <a href="mailto:rules-dev@lists.jboss.org">rules-dev@lists.jboss.org</a><br>>> <a href="https://lists.jboss.org/mailman/listinfo/rules-dev">https://lists.jboss.org/mailman/listinfo/rules-dev
</a><br>>><br>><br>><br>> --<br>> Edson Tirelli<br>> Software Engineer - JBoss Rules Core Developer<br>> Office: +55 11 3124-6000<br>> Mobile: +55 11 9218-4151<br>> JBoss, a division of Red Hat @
<a href="http://www.jboss.com">www.jboss.com</a><br>><br>><br>> _______________________________________________<br>> rules-dev mailing list<br>> <a href="mailto:rules-dev@lists.jboss.org">rules-dev@lists.jboss.org
</a><br>> <a href="https://lists.jboss.org/mailman/listinfo/rules-dev">https://lists.jboss.org/mailman/listinfo/rules-dev</a><br><br>_______________________________________________<br>rules-dev mailing list<br><a href="mailto:rules-dev@lists.jboss.org">
rules-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/rules-dev">https://lists.jboss.org/mailman/listinfo/rules-dev</a><br></blockquote></div><br><br clear="all"><br>-- <br> Edson Tirelli<br>
Software Engineer - JBoss Rules Core Developer<br> Office: +55 11 3124-6000<br> Mobile: +55 11 9218-4151<br> JBoss, a division of Red Hat @ <a href="http://www.jboss.com">www.jboss.com</a>