Skip to content

Wrong link target in the documentation #62990

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.

Sign up for GitHub

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
Closed
sedymrak opened this issue May 29, 2023 · 5 comments
Closed

Wrong link target in the documentation #62990

sedymrak opened this issue May 29, 2023 · 5 comments
Labels
confirmed Verified by a second party documentation lldb

Comments

@sedymrak
Copy link
Contributor

In the LDDB documentation you have the following sentence:

"The --format (which you can shorten to -f) option accepts a format name."

I believe that the link points to a wrong place.

I guess that you want it to point to the table that precedes the Type summary section.
@llvmbot
Copy link
Member

llvmbot commented May 29, 2023

@llvm/issue-subscribers-lldb

@eddiep24
Copy link
Contributor

Working on it.

@eddiep24
Copy link
Contributor

Awaiting review https://reviews.llvm.org/D151668

@sedymrak
Copy link
Contributor Author

sedymrak commented Jun 1, 2023

Thank you!

(I have learned something new. Before submitting the bug report, I have tried to fix this by using :ref: instruction, which is mentioned in the ReStructured Text documentation. However, after rendering the modified page, the link still did not work.)

@DavidSpickett DavidSpickett added the confirmed Verified by a second party label Sep 8, 2023
DavidSpickett pushed a commit that referenced this issue Sep 8, 2023
@eddiep24 @DavidSpickett
Wrong link target in the documentation #62990
9ce82a1 
In the LDDB documentation you have the following sentence:

"The --format (which you can shorten to -f) option accepts a format name."

The link points to the wrong place.

I pointed it to the table that precedes the Type summary section.

Reviewed By: DavidSpickett

Differential Revision: https://reviews.llvm.org/D151668
@DavidSpickett
Copy link
Collaborator

Fixed by the commit linked above, thanks @eddiep24 !

@michaelrj-google michaelrj-google mentioned this issue Sep 12, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
confirmed Verified by a second party documentation lldb
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@DavidSpickett @EugeneZelenko @llvmbot @eddiep24 @sedymrak