[clang-tools-extra] r258098 - [clang-tidy] Fixed wording ("clang-tidy check", not "clang-tidy checker")

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 18 17:21:32 PST 2016


No worries, I reviewed at least some of these changes and haven't found the
issues, but was later pointed to them by a community member ;)

On Mon, Jan 18, 2016 at 11:58 PM, Aaron Ballman <aaron at aaronballman.com>
wrote:

> On Mon, Jan 18, 2016 at 5:51 PM, Alexander Kornienko via cfe-commits
> <cfe-commits at lists.llvm.org> wrote:
> > Author: alexfh
> > Date: Mon Jan 18 16:51:09 2016
> > New Revision: 258098
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=258098&view=rev
> > Log:
> > [clang-tidy] Fixed wording ("clang-tidy check", not "clang-tidy checker")
>
> Sorry about that -- community terminology can be hard to keep straight
> sometimes. ;-)
>
> ~Aaron
>
> >
> > Modified:
> >     clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl03-c.rst
> >     clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl54-cpp.rst
> >     clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl59-cpp.rst
> >     clang-tools-extra/trunk/docs/clang-tidy/checks/cert-err61-cpp.rst
> >     clang-tools-extra/trunk/docs/clang-tidy/checks/cert-fio38-c.rst
> >     clang-tools-extra/trunk/docs/clang-tidy/checks/cert-oop11-cpp.rst
> >
>  clang-tools-extra/trunk/docs/clang-tidy/checks/google-build-namespaces.rst
> >
>  clang-tools-extra/trunk/docs/clang-tidy/checks/misc-move-constructor-init.rst
> >
>  clang-tools-extra/trunk/docs/clang-tidy/checks/misc-new-delete-overloads.rst
> >
>  clang-tools-extra/trunk/docs/clang-tidy/checks/misc-non-copyable-objects.rst
> >     clang-tools-extra/trunk/docs/clang-tidy/checks/misc-static-assert.rst
> >
>  clang-tools-extra/trunk/docs/clang-tidy/checks/misc-throw-by-value-catch-by-reference.rst
> >
> > Modified: clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl03-c.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl03-c.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > --- clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl03-c.rst
> (original)
> > +++ clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl03-c.rst Mon
> Jan 18 16:51:09 2016
> > @@ -5,5 +5,5 @@
> >  cert-dcl03-c
> >  ============
> >
> > -The cert-dcl03-c checker is an alias, please see
> > +The cert-dcl03-c check is an alias, please see
> >  `misc-static-assert <misc-static-assert.html>`_ for more information.
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl54-cpp.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl54-cpp.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > --- clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl54-cpp.rst
> (original)
> > +++ clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl54-cpp.rst
> Mon Jan 18 16:51:09 2016
> > @@ -5,6 +5,6 @@
> >  cert-dcl54-cpp
> >  ==============
> >
> > -The cert-dcl54-cpp checker is an alias, please see
> > +The cert-dcl54-cpp check is an alias, please see
> >  `misc-new-delete-overloads <misc-new-delete-overloads.html>`_ for more
> >  information.
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl59-cpp.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl59-cpp.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > --- clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl59-cpp.rst
> (original)
> > +++ clang-tools-extra/trunk/docs/clang-tidy/checks/cert-dcl59-cpp.rst
> Mon Jan 18 16:51:09 2016
> > @@ -5,5 +5,5 @@
> >  cert-dcl59-cpp
> >  ==============
> >
> > -The cert-dcl59-cpp checker is an alias, please see
> > +The cert-dcl59-cpp check is an alias, please see
> >  `google-build-namespaces <google-build-namespaces.html>`_ for more
> information.
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/cert-err61-cpp.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/cert-err61-cpp.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > --- clang-tools-extra/trunk/docs/clang-tidy/checks/cert-err61-cpp.rst
> (original)
> > +++ clang-tools-extra/trunk/docs/clang-tidy/checks/cert-err61-cpp.rst
> Mon Jan 18 16:51:09 2016
> > @@ -5,6 +5,6 @@
> >  cert-err61-cpp
> >  ==============
> >
> > -The cert-err61-cpp checker is an alias, please see
> > +The cert-err61-cpp check is an alias, please see
> >  `misc-throw-by-value-catch-by-reference
> <misc-throw-by-value-catch-by-reference.html>`_
> >  for more information.
> >
> > Modified: clang-tools-extra/trunk/docs/clang-tidy/checks/cert-fio38-c.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/cert-fio38-c.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > --- clang-tools-extra/trunk/docs/clang-tidy/checks/cert-fio38-c.rst
> (original)
> > +++ clang-tools-extra/trunk/docs/clang-tidy/checks/cert-fio38-c.rst Mon
> Jan 18 16:51:09 2016
> > @@ -5,6 +5,6 @@
> >  cert-fio38-c
> >  ============
> >
> > -The cert-fio38-c checker is an alias, please see
> > +The cert-fio38-c check is an alias, please see
> >  `misc-non-copyable-objects <misc-non-copyable-objects.html>`_ for more
> >  information.
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/cert-oop11-cpp.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/cert-oop11-cpp.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > --- clang-tools-extra/trunk/docs/clang-tidy/checks/cert-oop11-cpp.rst
> (original)
> > +++ clang-tools-extra/trunk/docs/clang-tidy/checks/cert-oop11-cpp.rst
> Mon Jan 18 16:51:09 2016
> > @@ -5,6 +5,6 @@
> >  cert-oop11-cpp
> >  ==============
> >
> > -The cert-oop11-cpp checker is an alias, please see
> > +The cert-oop11-cpp check is an alias, please see
> >  `misc-move-constructor-init <misc-move-constructor-init.html>`_ for more
> >  information.
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/google-build-namespaces.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/google-build-namespaces.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > ---
> clang-tools-extra/trunk/docs/clang-tidy/checks/google-build-namespaces.rst
> (original)
> > +++
> clang-tools-extra/trunk/docs/clang-tidy/checks/google-build-namespaces.rst
> Mon Jan 18 16:51:09 2016
> > @@ -3,7 +3,7 @@
> >  google-build-namespaces
> >  =======================
> >
> > -"cert-dcl59-cpp" redirects here as an alias for this checker.
> > +"cert-dcl59-cpp" redirects here as an alias for this check.
> >
> >  Finds anonymous namespaces in headers.
> >
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-move-constructor-init.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/misc-move-constructor-init.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > ---
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-move-constructor-init.rst
> (original)
> > +++
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-move-constructor-init.rst
> Mon Jan 18 16:51:09 2016
> > @@ -3,7 +3,7 @@
> >  misc-move-constructor-init
> >  ==========================
> >
> > -"cert-oop11-cpp" redirects here as an alias for this checker.
> > +"cert-oop11-cpp" redirects here as an alias for this check.
> >
> >  The check flags user-defined move constructors that have a
> ctor-initializer
> >  initializing a member or base class through a copy constructor instead
> of a
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-new-delete-overloads.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/misc-new-delete-overloads.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > ---
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-new-delete-overloads.rst
> (original)
> > +++
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-new-delete-overloads.rst
> Mon Jan 18 16:51:09 2016
> > @@ -3,7 +3,7 @@
> >  misc-new-delete-overloads
> >  =========================
> >
> > -"cert-dcl54-cpp" redirects here as an alias for this checker.
> > +"cert-dcl54-cpp" redirects here as an alias for this check.
> >
> >  The check flags overloaded operator new() and operator delete()
> functions that
> >  do not have a corresponding free store function defined within the same
> scope.
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-non-copyable-objects.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/misc-non-copyable-objects.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > ---
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-non-copyable-objects.rst
> (original)
> > +++
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-non-copyable-objects.rst
> Mon Jan 18 16:51:09 2016
> > @@ -3,7 +3,7 @@
> >  misc-non-copyable-objects
> >  =========================
> >
> > -"cert-fio38-c" redirects here as an alias for this checker.
> > +"cert-fio38-c" redirects here as an alias for this check.
> >
> >  The check flags dereferences and non-pointer declarations of objects
> that are
> >  not meant to be passed by value, such as C FILE objects or POSIX
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-static-assert.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/misc-static-assert.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > ---
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-static-assert.rst
> (original)
> > +++
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-static-assert.rst Mon
> Jan 18 16:51:09 2016
> > @@ -3,7 +3,7 @@
> >  misc-static-assert
> >  ==================
> >
> > -"cert-dcl03-c" redirects here as an alias for this checker.
> > +"cert-dcl03-c" redirects here as an alias for this check.
> >
> >  Replaces ``assert()`` with ``static_assert()`` if the condition is
> evaluatable
> >  at compile time.
> >
> > Modified:
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-throw-by-value-catch-by-reference.rst
> > URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/clang-tidy/checks/misc-throw-by-value-catch-by-reference.rst?rev=258098&r1=258097&r2=258098&view=diff
> >
> ==============================================================================
> > ---
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-throw-by-value-catch-by-reference.rst
> (original)
> > +++
> clang-tools-extra/trunk/docs/clang-tidy/checks/misc-throw-by-value-catch-by-reference.rst
> Mon Jan 18 16:51:09 2016
> > @@ -3,7 +3,7 @@
> >  misc-throw-by-value-catch-by-reference
> >  ======================================
> >
> > -"cert-err61-cpp" redirects here as an alias for this checker.
> > +"cert-err61-cpp" redirects here as an alias for this check.
> >
> >  Finds violations of the rule "Throw by value, catch by reference"
> presented for example in "C++ Coding Standards" by H. Sutter and A.
> Alexandrescu. This check also has the option to find violations of the rule
> "Throw anonymous temporaries" (
> https://www.securecoding.cert.org/confluence/display/cplusplus/ERR09-CPP.+Throw+anonymous+temporaries).
> The option is named "CheckThrowTemporaries" and it's on by default.
> >
> >
> >
> > _______________________________________________
> > cfe-commits mailing list
> > cfe-commits at lists.llvm.org
> > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160119/fdc71a47/attachment-0001.html>


More information about the cfe-commits mailing list