Re: [PATCH] D14311: [Clang] Fix some Clang-tidy modernize warnings, other minor fixes

2015-11-04 Thread Hans Wennborg via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. lgtm Repository: rL LLVM http://reviews.llvm.org/D14311 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

Re: [PATCH] D14311: [Clang] Fix some Clang-tidy modernize warnings, other minor fixes

2015-11-04 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL252081: Fix some Clang-tidy modernize warnings, other minor fixes. (authored by eugenezelenko). Changed prior to commit: http://reviews.llvm.org/D14311?vs=39134=39252#toc Repository: rL LLVM

[PATCH] D14311: [Clang] Fix some Clang-tidy modernize warnings, other minor fixes

2015-11-03 Thread Eugene Zelenko via cfe-commits
Eugene.Zelenko created this revision. Eugene.Zelenko added a reviewer: hans. Eugene.Zelenko added a subscriber: cfe-commits. Eugene.Zelenko set the repository for this revision to rL LLVM. Herald added subscribers: dschuff, jfb. Fixed warnings are: modernize-use-override, modernize-use-nullptr