[rules-dev] Another MVEL hiccup

[Wolfgang Laun]

OK, thanks.

Not having looked at the code, I can only say that it is strange that
some errors are handled correctly while this one has slipped through.

-W


On 12/03/2012, Mario Fusco <mario.fusco at gmail.com> wrote:
> Hi Wolfgang,
>
> This is not a mvel issue, that in this is case is correctly reporting the
> syntax error, but only a small bug in drools failing to catch and correctly
> menage the excpetion thrown by mvel in that particular case.
>
> I just pushed the fix, thank you for reporting this.
>
> Mario
>
> On Sun, Mar 11, 2012 at 7:26 PM, Wolfgang Laun
> <wolfgang.laun at gmail.com>wrote:
>
>> This is not correct:
>>     Foo( id == "\a" )
>> and it is rightly flagged as a syntax error, but the ensuing stack dump is
>> uncalled for:
>>
>> Exception in thread "main" [Error: illegal escape sequence: a]
>> [Near : {... "\a" ....}]
>>                ^
>> [Line: 1, Column: 3]
>> at org.mvel2.util.ParseTools.handleEscapeSequence(ParseTools.java:889)
>>  at org.mvel2.util.ParseTools.handleStringEscapes(ParseTools.java:1533)
>> at org.mvel2.compiler.AbstractParser.nextToken(AbstractParser.java:1105)
>>  at
>> org.mvel2.compiler.ExpressionCompiler._compile(ExpressionCompiler.java:128)
>> at
>> org.mvel2.compiler.ExpressionCompiler.compile(ExpressionCompiler.java:62)
>>  at org.mvel2.MVEL.analysisCompile(MVEL.java:670)
>> at org.mvel2.MVEL.analysisCompile(MVEL.java:674)
>>  at
>> org.drools.rule.builder.PatternBuilder.setInputs(PatternBuilder.java:942)
>> at
>> org.drools.rule.builder.PatternBuilder.buildRelationalExpression(PatternBuilder.java:638)
>>  at org.drools.rule.builder.PatternBuilder.build(PatternBuilder.java:588)
>> at
>> org.drools.rule.builder.PatternBuilder.processConstraintsAndBinds(PatternBuilder.java:449)
>>  at org.drools.rule.builder.PatternBuilder.build(PatternBuilder.java:296)
>> at org.drools.rule.builder.PatternBuilder.build(PatternBuilder.java:118)
>>  at
>> org.drools.rule.builder.GroupElementBuilder.build(GroupElementBuilder.java:65)
>> at org.drools.rule.builder.RuleBuilder.build(RuleBuilder.java:80)
>>  at org.drools.compiler.PackageBuilder.addRule(PackageBuilder.java:2503)
>> at org.drools.compiler.PackageBuilder.addPackage(PackageBuilder.java:847)
>>  at
>> org.drools.compiler.PackageBuilder.addPackageFromDrl(PackageBuilder.java:443)
>> at
>> org.drools.compiler.PackageBuilder.addKnowledgeResource(PackageBuilder.java:626)
>>  at
>> org.drools.builder.impl.KnowledgeBuilderImpl.add(KnowledgeBuilderImpl.java:41)
>> at foobar.Main.setup(Main.java:68)
>>  at foobar.Main.main(Main.java:124)
>>
>> -W
>>
>>
>> _______________________________________________
>> rules-dev mailing list
>> rules-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/rules-dev
>>
>>
>