Re: [rules-users] 5.4.0.Beta2 parse error in consequence /* in string

2012-03-02 Thread Benjamin Bennett
Got permission from my company. Digging around think , correct me if I am wrong. The bug is actually most likely in the mvel project. I checked out the mvel project. In the mvel project there is method call skipWhiteSpace() . This method skips different types of comments. It is a minor

Re: [rules-users] 5.4.0.Beta2 parse error in consequence /* in string

2012-02-28 Thread Benjamin Bennett
Working on the sign contributer agreement. Have to fill out the permission slip at the company I work for to contribute back in my free time. I submitted at merge request https://github.com/droolsjbpm/drools/pull/102 If the commit looks fine but just waiting on the Contributor

Re: [rules-users] 5.4.0.Beta2 parse error in consequence /* in string

2012-02-27 Thread Benjamin Bennett
I had created issue JBRULES-3399 for the issue. I tracked it a bit , it is any comment. // or /* , think the lexar expression for comment isn't correct . I am working towards a fix but not a antlr expert. I have used yacc. On Mon, Feb 27, 2012 at 1:38 AM, Wolfgang Laun

Re: [rules-users] 5.4.0.Beta2 parse error in consequence /* in string

2012-02-27 Thread Wolfgang Laun
This is an MVEL issue, which doesn't have a lexer and doesn't use antlr. MVEL just picks up characters from a stream, trying to compact them into tokens, apparently according to some state, in highly brittle code. (I'm not slandering - the author's comment says so, too.) I cannot reproduce the

Re: [rules-users] 5.4.0.Beta2 parse error in consequence /* in string

2012-02-27 Thread Mark Proctor
On 27/02/2012 15:44, Benjamin Bennett wrote: I had created issue JBRULES-3399 for the issue. I tracked it a bit , it is any comment. // or /* , think the lexar expression for comment isn't correct . I am working towards a fix but not a antlr expert. I have used yacc. just a reminder to read

Re: [rules-users] 5.4.0.Beta2 parse error in consequence /* in string

2012-02-26 Thread Wolfgang Laun
Submitted: JBRULES-3401. -W On 24/02/2012, Benjamin Bennett benbenn...@gmail.com wrote: Just passing on I am using the drools 5.4.0Beta2 The following doesn't compile using the mvel dialect . rule some rule when fact: Fact() then BasicFact fault= new BasicFact( /*); end but rule