Grammatica - Bugs: bug #12767, Tokenizer does not do '/* **/'...
You are not allowed to post comments on this tracker with your current authentication level.
bug #12767: Tokenizer does not do '/* **/' comment correctly
Submitter: | None | ||
Submitted: | Thu 21 Apr 2005 01:53:45 PM UTC | ||
Severity: | 5 - Major | Item Group: | Software |
Status: | Fixed | Assigned to: | cederberg |
Open/Closed: | Closed |
Discussion locked!
Sat 14 Mar 2009 09:34:13 AM UTC, comment #4: |
Per Cederberg <cederberg>![]() ![]() |
Sat 27 Aug 2005 10:53:23 AM UTC, comment #3: Oops. That is correct. I think the correct fix here is to simplify the whole regexp by using a reluctant quantifier '*?' instead of the current greedy '*' quantifier. That change should fix the whole issue while also simplifying the regexp. Here is the new version:
|
Per Cederberg <cederberg>![]() ![]() |
Wed 24 Aug 2005 10:53:16 PM UTC, comment #2: The regexp in 1.5.alpha1 is imho still wrong. A small test shows that something like this fragment
|
Anonymous |
Thu 21 Apr 2005 06:53:13 PM UTC, comment #1: Ah, yes. This is indeed a bug. I'd go for the following solution, though:
|
Per Cederberg <cederberg>![]() ![]() |
Thu 21 Apr 2005 01:53:45 PM UTC, original submission:
Considering the way multiline comments are defined in the Grammatica grammar (that is: MULTI_LINE_COMMENT = <</\*([^*]|\*[^/])*\*/>> %ignore%) it can tokenize the following comment (which is legal):
|
Anonymous |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Version 1.5 was released on 2009-03-07. Closing this issue.