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.
Follow 4 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2009-03-14 | cederberg | Open/Closed | Open | Closed | |
Discussion Lock | None | Locked | |||
2005-05-07 | cederberg | Status | None | Fixed | |
2005-04-21 | cederberg | Assigned to | None | cederberg |
Powered by Savane 3.13-f8d8.
Corresponding source code
Version 1.5 was released on 2009-03-07. Closing this issue.