This is an archive of the discontinued LLVM Phabricator instance.

[FormatVariadic] Mark index as required in docstring (NFC)
ClosedPublic

Authored by kastiglione on Feb 2 2022, 11:00 AM.

Details

Summary

After looking at the formatv docstring, I thought the index was optional (as it
is in other languages). This changes the header docs to show index instead of
[index], to indicate that it is required.

Diff Detail

Event Timeline

kastiglione created this revision.Feb 2 2022, 11:00 AM
kastiglione requested review of this revision.Feb 2 2022, 11:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 2 2022, 11:00 AM
kastiglione retitled this revision from [FormatVariadic] Mark index as required in docstring to [FormatVariadic] Mark index as required in docstring (NFC).Feb 2 2022, 11:01 AM
aprantl accepted this revision.Feb 21 2022, 9:23 AM
This revision is now accepted and ready to land.Feb 21 2022, 9:23 AM
This revision was landed with ongoing or failed builds.Feb 23 2022, 3:34 PM
This revision was automatically updated to reflect the committed changes.