[rules-dev] ClassFieldInspector and Groovy Classes

Edson Tirelli tirelli at post.com
Mon Apr 16 15:47:30 EDT 2007


   Terry,

   A bad commit from my part.
   It shall work now.

   Regards,
     Edson

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



-- 
  Edson Tirelli
  Software Engineer - JBoss Rules Core Developer
  Office: +55 11 3124-6000
  Mobile: +55 11 9218-4151
  JBoss, a division of Red Hat @ www.jboss.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/rules-dev/attachments/20070416/886ea82b/attachment.html 


More information about the rules-dev mailing list