This is an archive of the discontinued LLVM Phabricator instance.

Wrong link target in the documentation #62990
ClosedPublic

Authored by eddiep24 on May 29 2023, 5:42 PM.

Details

Summary

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.

Diff Detail

Event Timeline

eddiep24 created this revision.May 29 2023, 5:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 29 2023, 5:42 PM
eddiep24 requested review of this revision.May 29 2023, 5:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 29 2023, 5:42 PM
DavidSpickett accepted this revision.Sep 8 2023, 7:31 AM
DavidSpickett added a subscriber: DavidSpickett.

Sorry this got completely lost, built the HTML locally and the result is as expected. LGTM.

This revision is now accepted and ready to land.Sep 8 2023, 7:31 AM
This revision was automatically updated to reflect the committed changes.

I've taken the liberty of landing it for you with the details from your last commit.

Thanks again, we'll try not to take 3 months to review a simple fix next time :)