Bug 748711 - DRL parser expects a newline at the end of the file
Summary: DRL parser expects a newline at the end of the file
Keywords:
Status: VERIFIED
Alias: None
Product: JBoss Enterprise BRMS Platform 5
Classification: JBoss
Component: BRE (Expert, Fusion)
Version: BRMS 5.3.0.GA
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: Nobody
QA Contact: Lukáš Petrovický
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-10-25 05:53 UTC by Tomas Schlosser
Modified: 2022-07-18 08:29 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Including a comment on the last line of a DRL file, caused a no viable alternative error to occur and the parser to return a null package. This has been resolved and the no viable alternative error no longer occurs.
Clone Of:
Environment:
Last Closed:
Type: Bug


Attachments (Terms of Use)
Simple reproducer (1.08 KB, text/plain)
2011-10-25 05:53 UTC, Tomas Schlosser
no flags Details

Description Tomas Schlosser 2011-10-25 05:53:12 UTC
Created attachment 530001 [details]
Simple reproducer

Description of problem:
If you put a comment on the last line of the DRL file you end up with no viable alternative error and no packages.

Version-Release number of selected component (if applicable):
BRMS-5.2.0.GA

How reproducible:
Every time.

Steps to Reproduce:
1. Run the included class
  
Actual results:
[7,19]: [ERR 101] Line 7:19 no viable alternative at input ''
[0,0]: Parser returned a null Package

Expected results:
Hello world!
Finished...

Additional info:

Comment 3 Iva Zakova 2012-07-23 14:18:19 UTC
A pull request for this issue was created: https://github.com/droolsjbpm/drools/pull/130

Comment 4 Edson Tirelli 2012-08-16 22:28:17 UTC
Porcelli,

Can you please take a look at this. Lets talk on IRC and I can tell you why this is happening.

Thanks

Comment 5 Alexandre Porcelli 2012-08-20 17:42:01 UTC
Fix already available on master, 5.4.x and 5.3.x branches of Drools project.

Comment 6 Iva Zakova 2012-09-17 07:33:44 UTC
Verified in 5.3.1.ER1.


Note You need to log in before you can comment on or make changes to this bug.