Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug in CTM lexer and decimal occurrence of 1.0 #356

Closed
GoogleCodeExporter opened this issue Mar 16, 2015 · 4 comments
Closed

Bug in CTM lexer and decimal occurrence of 1.0 #356

GoogleCodeExporter opened this issue Mar 16, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Try to load the attached CTM topic map into Omnigator
2. You will get the following error report:

Could not reload topic map: ctm_1_0_bug.ctm
java.io.IOException: Lexical error at file:/C:/Program 
Files/Ontopia/ontopia-5.1.2/apache-tomcat/webapps/omnigator/WEB-INF/topicmaps/ct
m_1_0_bug.ctm:6:0: expecting STOP, found 'another_decimal_occurrence'The store 
registry has been updated in APPLICATION_SCOPE.

What is the expected output? What do you see instead?

Expected to successfully load the topic map and have two occurrences for "some 
topic", one with decimal value 1.2 and one with decimal value 1.0 . 

Please use labels and text to provide additional information.

I was looking through the code and noticed a rule in ctm.flex:

YYINITIAL {
"1.0" { return(newToken.CTMParser.ONEOH); }
...
}

This rule is active in the YYINITIAL state so it prevents the {decimal} rule 
from finding a valid decimal value of 1.0.


Original issue reported on code.google.com by dan.sp...@gmail.com on 1 Feb 2011 at 2:14

Attachments:

@GoogleCodeExporter
Copy link
Author

Oh, bloody hell! Now there's a stupid bug. :-/

Anyway, outstanding error report. I'll fix this asap.

Original comment by lar...@gmail.com on 1 Feb 2011 at 7:12

  • Changed state: A
  • Added labels: Component-Engine, Syntax-CTM

@GoogleCodeExporter
Copy link
Author

Original comment by lar...@gmail.com on 1 Feb 2011 at 7:13

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author

Fixed in revision 1634. A test case has been created, but will need to be added 
to the cxtm-tests project separately.

Original comment by lar...@gmail.com on 7 Feb 2011 at 5:16

@GoogleCodeExporter
Copy link
Author

Original comment by lar...@gmail.com on 7 Feb 2011 at 5:16

  • Changed state: Verified

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant