Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Wront error location with CamlP4 #6210

Closed
vicuna opened this issue Oct 21, 2013 · 3 comments
Closed

Wront error location with CamlP4 #6210

vicuna opened this issue Oct 21, 2013 · 3 comments
Labels
bug camlp4

Comments

@vicuna
Copy link

vicuna commented Oct 21, 2013

Original bug ID: 6210
Reporter: furuse
Assigned to: @bobzhang
Status: closed (set by @xavierleroy on 2015-12-11T18:24:18Z)
Resolution: fixed
Priority: normal
Severity: minor
Version: 4.01.0
Category: -for Camlp4 use https://github.com/ocaml/camlp4/issues
Monitored by: @gasche

Bug description

type t = (int, float) Unbound.t

The vanilla ocamlc reports Error: Unbound module Unbound at line 1, characters 9-30 "(int, float) Unbound.t", while ocamlc -pp camlp4o reports at line 1, characters 15-20, "float".

Sigh... P4 has so many location glitches.
@vicuna
Copy link
Author

vicuna commented Oct 21, 2013

Comment author: @bobzhang

Thanks for the report. This should be fixable.
Which version are you working against? I would prefer fix the bug against 4.01. if that's not the problem for you. (currently the parsetree in trunk is unstable so I would avoid the change if that is ok)

@vicuna
Copy link
Author

vicuna commented Oct 25, 2013

Comment author: furuse

Thanks, It happend with the pure 4.01.0.

@vicuna
Copy link
Author

vicuna commented Oct 28, 2013

Comment author: @bobzhang

Fixed in revision 14249. If you have any problem, please reopen the issue. Thanks

@vicuna vicuna closed this as completed Dec 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug camlp4
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vicuna