HomePhabricator

[clang-tidy] add IgnoreMacros option to readability-redundant-smartptr-get

Description

[clang-tidy] add IgnoreMacros option to readability-redundant-smartptr-get

And also enable it by default to be consistent with e.g. modernize-use-using.

This helps e.g. when running this check on client code where the macro is
provided by the system, so there is no easy way to modify it.

Reviewed By: JonasToth

Differential Revision: https://reviews.llvm.org/D53454

Details

Committed
vmiklosOct 21 2018, 12:16 PM
Reviewer
JonasToth
Differential Revision
D53454: [clang-tidy] add IgnoreMacros option to eadability-redundant-smartptr-get
Branches
Unknown
Tags
Unknown

Event Timeline

JonasToth added inline comments.
/clang-tools-extra/trunk/docs/ReleaseNotes.rst
70

Could you please move this part below the new checks? AFAIK there is no written rule on that but the new checks come first, and then improvements/changes below.

vmiklos marked an inline comment as done.Oct 21 2018, 11:38 PM
vmiklos added inline comments.
/clang-tools-extra/trunk/docs/ReleaseNotes.rst
70

Done in r344885.