This revision was automatically updated to reflect the committed changes.
Closed by commit rL289524: [clang-tidy] Add check for redundant function
pointer dereferences (authored by malcolm.parsons).
Changed prior to commit:
https://reviews.llvm.org/D27520?vs=80642&id=81190#toc
Repository:
rL
hokein accepted this revision.
hokein added a comment.
This revision is now accepted and ready to land.
Nice, the patch looks good to me.
Comment at:
docs/clang-tidy/checks/readability-redundant-function-ptr-dereference.rst:25
+ int i = (*p)(10, 50);
+
Nit: a
malcolm.parsons updated this revision to Diff 80642.
malcolm.parsons added a comment.
Add to release notes.
Remove youtube link.
https://reviews.llvm.org/D27520
Files:
clang-tidy/readability/CMakeLists.txt
clang-tidy/readability/ReadabilityTidyModule.cpp
clang-tidy/readability/RedundantFu
Eugene.Zelenko added a comment.
Please mention this check in docs/ReleaseNotes.rst (in alphabetical order).
Comment at:
docs/clang-tidy/checks/readability-redundant-function-ptr-dereference.rst:7
+Finds redundant dereferences of a function pointer.
+See https://youtu.be/6eX9gP
malcolm.parsons created this revision.
malcolm.parsons added reviewers: alexfh, aaron.ballman, hokein.
malcolm.parsons added a subscriber: cfe-commits.
Herald added subscribers: JDevlieghere, mgorny.
See https://youtu.be/6eX9gPithBo
https://reviews.llvm.org/D27520
Files:
clang-tidy/readabilit