[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