Bug #3381
closed![Author: Michael Kay](https://www.gravatar.com/avatar/db6526d63053f09b62e52c2da8b2230a?rating=PG&size=50&default=https%3A%2F%2Fassets.plan.io%2Fimages%2Fdefault_avatar.png)
![Assignee: Michael Kay](https://www.gravatar.com/avatar/db6526d63053f09b62e52c2da8b2230a?rating=PG&size=22&default=https%3A%2F%2Fassets.plan.io%2Fimages%2Fdefault_avatar.png)
Spurious "unmatched ')'" error in regular expression
100%
Description
Saxon-JS reports "unmatched ')'" in a regular expression, caused by failing to ignore ')' characters that appear within a character class (that is, within square brackets).
Shows up in test case analyze-string-097.
When we fix this the test shows a further error, caused by incorrect treatment of escaped hyphens within square brackets: the backslash is being incorrectly removed.
Updated by Michael Kay almost 7 years ago
- Priority changed from Low to Normal
- Applies to JS Branch 0.9, 1.0, Trunk added
- Fix Committed on JS Branch Trunk added
The same test also showed up one further errors: "$" outside square brackets was (for some unfathomable reason) being replaced by "$".
Changes made on the 2.0 branch and the test case is now working.
Updated by Debbie Lockett over 6 years ago
- Status changed from New to Resolved
- Fix Committed on JS Branch 1.0 added
Relevant fixes in regex.js (belatedly) also made on 1.0 branch.
Updated by Debbie Lockett about 6 years ago
- Status changed from Resolved to Closed
- % Done changed from 0 to 100
- Fixed in JS Release set to Saxon-JS 1.1.0
Bug fix applied in the Saxon-JS 1.1.0 maintenance release.
Please register to edit this issue
Also available in: Atom PDF Tracking page