[PATCH] D27520: [clang-tidy] Add check for redundant function pointer dereferences

2016-12-13 Thread Malcolm Parsons via Phabricator via cfe-commits
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

[PATCH] D27520: [clang-tidy] Add check for redundant function pointer dereferences

2016-12-12 Thread Haojian Wu via Phabricator via cfe-commits
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

[PATCH] D27520: [clang-tidy] Add check for redundant function pointer dereferences

2016-12-07 Thread Malcolm Parsons via Phabricator via cfe-commits
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

[PATCH] D27520: [clang-tidy] Add check for redundant function pointer dereferences

2016-12-07 Thread Eugene Zelenko via Phabricator via cfe-commits
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

[PATCH] D27520: [clang-tidy] Add check for redundant function pointer dereferences

2016-12-07 Thread Malcolm Parsons via Phabricator via cfe-commits
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