You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Original bug ID: 5692 Reporter: Hendrik Tews Assigned to:@diml Status: closed (set by @xavierleroy on 2015-12-11T18:07:23Z) Resolution: fixed Priority: normal Severity: major Version: 4.00.0+beta2/+rc1 Fixed in version: 4.00.0+beta2/+rc1 Category: -for Camlp4 use https://github.com/ocaml/camlp4/issues
Bug description
Somewhere inside the camlp4 machinery, end locations are not
propagated any more. The result is that for many nodes in the camlp4
ast the end location is identical to the start location.
Severity major, because this will very likely break otags. Because of
missing or wrong location info, otags reparses fragments of the source.
In some cases it has to take the last or second last word. This can of
course only work with correct end locations.
For example, for "exception A of int" I see for the top StExc node
Original bug ID: 5692
Reporter: Hendrik Tews
Assigned to: @diml
Status: closed (set by @xavierleroy on 2015-12-11T18:07:23Z)
Resolution: fixed
Priority: normal
Severity: major
Version: 4.00.0+beta2/+rc1
Fixed in version: 4.00.0+beta2/+rc1
Category: -for Camlp4 use https://github.com/ocaml/camlp4/issues
Bug description
Somewhere inside the camlp4 machinery, end locations are not
propagated any more. The result is that for many nodes in the camlp4
ast the end location is identical to the start location.
Severity major, because this will very likely break otags. Because of
missing or wrong location info, otags reparses fragments of the source.
In some cases it has to take the last or second last word. This can of
course only work with correct end locations.
For example, for "exception A of int" I see for the top StExc node
start_line 1, start_offset 0, end_line 1, end_offset 0
for "type t = int" I see for the top StTyp node
start_line 1, start_offset 0, end_line 1, end_offset 0
and there are many more cases...
The text was updated successfully, but these errors were encountered: