| Anonymous | Login | Signup for a new account | 2013-05-21 23:26 CEST | ![]() |
| Main | My View | View Issues | Change Log | Roadmap |
| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | |||||||
| 0005770 | OCaml | OCaml general | public | 2012-10-01 20:20 | 2012-10-16 16:05 | |||||||
| Reporter | mauny | |||||||||||
| Assigned To | frisch | |||||||||||
| Priority | low | Severity | minor | Reproducibility | always | |||||||
| Status | resolved | Resolution | fixed | |||||||||
| Platform | all | OS | all | OS Version | all | |||||||
| Product Version | 4.00.0 | |||||||||||
| Target Version | later | Fixed in Version | 4.01.0+dev | |||||||||
| Summary | 0005770: Syntax error messages involving unclosed parens are sometimes incorrect | |||||||||||
| Description | Parsing invalid patterns inside well-balanced parentheses produces an error message saying that parens don't match. I provide a patch that fixes the problem by adding a few more error rules in the parser and relevant error messages. | |||||||||||
| Steps To Reproduce | $ echo 'match x with (1 as ) -> 0;;' > /tmp/a.ml $ ocamlc /tmp/a.ml -o /tmp/a File "/tmp/a.ml", line 1, characters 19-20: Error: Syntax error: ')' expected File "/tmp/a.ml", line 1, characters 13-14: Error: This '(' might be unmatched | |||||||||||
| Additional Information | After applying the patch, the new error message is File "/tmp/a.ml", line 1, characters 19-20: Error: Syntax error: identifier expected. | |||||||||||
| Tags | No tags attached. | |||||||||||
| Attached Files | ||||||||||||
Notes |
|
|
(0008268) frisch (developer) 2012-10-16 16:05 |
Thanks, I've applied your patch on the trunk (commit 13019) and adapted labltk/browser to support the new error. |
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2012-10-01 20:20 | mauny | New Issue | |
| 2012-10-01 20:20 | mauny | File Added: parser-msgs.patch | |
| 2012-10-16 16:05 | frisch | Note Added: 0008268 | |
| 2012-10-16 16:05 | frisch | Status | new => resolved |
| 2012-10-16 16:05 | frisch | Fixed in Version | => 4.01.0+dev |
| 2012-10-16 16:05 | frisch | Resolution | open => fixed |
| 2012-10-16 16:05 | frisch | Assigned To | => frisch |
| Copyright © 2000 - 2011 MantisBT Group |