[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