Wrong location for 'Unbound record field label' error #5073
Comments
|
Comment author: @alainfrisch Tried to do that... but stuck on updating Camlp4. I upload a proposal, in case someone wants to continue. |
|
Comment author: @alainfrisch It seems there has been a regression: now, even for expressions, the whole record is highlighted in case of an unbound field label. |
|
Comment author: @garrigue The syntax tree now contains all the locations necessary. |
|
Comment author: @garrigue Fixed in trunk and 4.00, revision 12607. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Original bug ID: 5073
Reporter: @alainfrisch
Assigned to: @garrigue
Status: closed (set by @garrigue on 2012-06-20T10:05:28Z)
Resolution: fixed
Priority: normal
Severity: minor
Fixed in version: 4.00.0+dev
Category: typing
Monitored by: @hcarty @alainfrisch
Bug description
In a record literal expression, an unbound record field error is located on the field expression, not on the label.
E.g.:
{x = 1};;
Error: Unbound record field label x
For pattern records, the whole pattern is highlighted. In a sense, the behavior for expressions is better (more precisely localized), but quite misleading.
The location for the faulty identifier itself is not kept in the parse tree. Maybe one should add it?
Alain
File attachments
The text was updated successfully, but these errors were encountered: