This is an archive of the discontinued LLVM Phabricator instance.

[clang-doc] Fix assert on startup
ClosedPublic

Authored by brettw on Aug 12 2022, 10:34 AM.

Details

Summary

When using clang-doc --format=html it will crash on startup because of an assertion doing a self-assignment of a SmallString. This patch removes the self-assignment by creating an intermediate copy.

Diff Detail

Event Timeline

brettw created this revision.Aug 12 2022, 10:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 12 2022, 10:34 AM
brettw requested review of this revision.Aug 12 2022, 10:34 AM
paulkirth accepted this revision.Aug 12 2022, 10:49 AM

This is LGTM, but can you add a tag to the summary title/commit message? ie. [clang-doc] Fix assert on startup in clang-doc

This revision is now accepted and ready to land.Aug 12 2022, 10:49 AM
brettw retitled this revision from Fix assert on startup in clang-doc to [clang-doc] Fix assert on startup.Aug 12 2022, 11:24 AM

This is LGTM, but can you add a tag to the summary title/commit message? ie. [clang-doc] Fix assert on startup in clang-doc

Done

This revision was automatically updated to reflect the committed changes.