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

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

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

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

vicuna opened this issue Feb 15, 2012 · 1 comment
Labels
feature-wish ocamldoc

Comments

@vicuna
Copy link

vicuna 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
Copy link
Author

vicuna 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.

@vicuna vicuna closed this as completed May 7, 2012
@vicuna vicuna mentioned this issue Mar 14, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-wish ocamldoc
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vicuna