This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Support concepts in `bugprone-forwarding-reference-overload`
ClosedPublic

Authored by Izaron on Oct 7 2022, 12:01 PM.

Details

Diff Detail

Event Timeline

Izaron created this revision.Oct 7 2022, 12:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 7 2022, 12:01 PM
Izaron requested review of this revision.Oct 7 2022, 12:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 7 2022, 12:01 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Eugene.Zelenko added inline comments.
clang-tools-extra/docs/clang-tidy/checks/bugprone/forwarding-reference-overload.rst
53–54

Please follow 80 characters limit.

Izaron updated this revision to Diff 466175.Oct 7 2022, 2:01 PM

Follow 80 characters limit. Thanks to @Eugene.Zelenko!

Izaron marked an inline comment as done.Oct 7 2022, 2:01 PM
Izaron added inline comments.
clang-tools-extra/docs/clang-tidy/checks/bugprone/forwarding-reference-overload.rst
53–54

Thanks! I ran git clang-format HEAD~1 before commiting, but it didn't modify this file. Seems like it doesn't see .rst files.

Eugene.Zelenko added inline comments.
clang-tools-extra/docs/clang-tidy/checks/bugprone/forwarding-reference-overload.rst
53–54

If I my memory is not failing me, @MyDeveloperDay once tried to introduce documentation linter, but I don't think that it reached production.

LegalizeAdulthood resigned from this revision.Mar 29 2023, 8:17 AM
PiotrZSL accepted this revision.Jun 23 2023, 1:57 AM

May require rebase, but looks fine.
Release notes entry should be added.

This revision is now accepted and ready to land.Jun 23 2023, 1:57 AM
PiotrZSL updated this revision to Diff 543206.Jul 22 2023, 10:29 AM
PiotrZSL edited the summary of this revision. (Show Details)

rebase, add release notes, move tests to separate file