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

please use locations of type Lexing.position in Odoc_info.location #5507

Closed
vicuna opened this issue Feb 15, 2012 · 1 comment

Comments

Projects
None yet
1 participant
@vicuna
Copy link

commented Feb 15, 2012

Original bug ID: 5507
Reporter: Hendrik Tews
Assigned to: @zoggy
Status: closed (set by @zoggy on 2012-05-07T10:33:47Z)
Resolution: fixed
Priority: normal
Severity: feature
Version: 3.13.0+dev
Category: ocamldoc

Bug description

The locations in ocamldoc provide not enough information to
locate the items, see the specification of pos_cnum in
lexing.mli. If ocamldoc could just copy the whole position of
type Lexing.position, then there would be at least an accurate
line number information. Copying the lexing position as whole
would also simplify the code, I believe.

@vicuna

This comment has been minimized.

Copy link
Author

commented May 7, 2012

Comment author: @zoggy

This is done now. This could not be done before, because the Location module was not installed before by "make install". This will change in next release.

Location.t is now used by OCamldoc since revision 12435.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.