[clang-tools-extra] r231938 - [clang-tidy] Clean up misc-use-override warning. NFC
David Blaikie
dblaikie at gmail.com
Wed Mar 11 09:57:06 PDT 2015
On Wed, Mar 11, 2015 at 9:47 AM, Alexander Kornienko <alexfh at google.com>
wrote:
> Author: alexfh
> Date: Wed Mar 11 11:47:27 2015
> New Revision: 231938
>
> URL: http://llvm.org/viewvc/llvm-project?rev=231938&view=rev
> Log:
> [clang-tidy] Clean up misc-use-override warning. NFC
>
> Modified:
> clang-tools-extra/trunk/clang-tidy/ClangTidy.h
>
> Modified: clang-tools-extra/trunk/clang-tidy/ClangTidy.h
> URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/ClangTidy.h?rev=231938&r1=231937&r2=231938&view=diff
>
> ==============================================================================
> --- clang-tools-extra/trunk/clang-tidy/ClangTidy.h (original)
> +++ clang-tools-extra/trunk/clang-tidy/ClangTidy.h Wed Mar 11 11:47:27 2015
> @@ -113,7 +113,7 @@ public:
> assert(!CheckName.empty());
> }
>
> - virtual ~ClangTidyCheck() {}
> + ~ClangTidyCheck() override = default;
>
Just remove this, then - since it's the implicit default anyway?
>
> /// \brief Override this to register \c PPCallbacks with \c Compiler.
> ///
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150311/176eed88/attachment.html>
More information about the cfe-commits
mailing list