This is an archive of the discontinued LLVM Phabricator instance.

[clang-format][docs] Fix incorrect 'clang-format 7' option markers
ClosedPublic

Authored by kuzkry on Apr 28 2022, 10:42 AM.

Details

Summary

Introduced by 23a5090c6, some style option markers indicated
'clang-format 7', though their respective options were available in
different releases.

Diff Detail

Event Timeline

kuzkry created this revision.Apr 28 2022, 10:42 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 28 2022, 10:42 AM
kuzkry requested review of this revision.Apr 28 2022, 10:42 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 28 2022, 10:42 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

Here's a list of earliest commits where each option changed in this PR was introduced:

AlwaysBreakTemplateDeclarations - 61e6bbf8508a46ad1269c8ea67dc0372b84e9a3b
IncludeBlocks - 4c2c9c3620e3bed9cb6fa709ec3b3cc454eb5748
IncludeCategories - introduced in 85c472dccdf536fb1853139f456a0763311a8a42 and configurable from 8ce1b8df7690bc25788c7da1b464c654461489b0
IncludeIsMainRegex - 9c8ff3551ace965a3232fc4a1b53a75015a706cf
IncludeIsMainSourceRegex - 335ac2eb662ce5f1888e2a50310b01fba2d40d68

MyDeveloperDay accepted this revision.Apr 28 2022, 12:40 PM
This revision is now accepted and ready to land.Apr 28 2022, 12:40 PM

I forgot to add "Differential revision" footer to the commit message and so I've got to manually close this review. Sorry

kuzkry closed this revision.Jul 16 2022, 9:28 AM