ohm icon indicating copy to clipboard operation
ohm copied to clipboard

Improve error messages for invalid escape sequences

Open pdubroy opened this issue 10 years ago • 2 comments

Rather than a generic parse error, we should have a message like "Invalid escape sequence: \p"

pdubroy avatar Jun 03 '15 09:06 pdubroy

Fixed by 56d8ff5.

pdubroy avatar Jun 03 '15 15:06 pdubroy

Fix was reverted (21ca368) because it wasn't right -- grammars should not be accepting invalid inputs. Alex modified the grammar so that the generic parse failure is more useful for this cause, we could still make the errors a lot clearer.

pdubroy avatar Jun 06 '15 16:06 pdubroy