[llvm] r311632 - [docs] In the CMake primer, correct the description of the ARGV/ARGN variables.

Lang Hames via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 23 22:38:40 PDT 2017


Author: lhames
Date: Wed Aug 23 22:38:39 2017
New Revision: 311632

URL: http://llvm.org/viewvc/llvm-project?rev=311632&view=rev
Log:
[docs] In the CMake primer, correct the description of the ARGV/ARGN variables.

ARGN is the sublist of unnamed arguments, not the count of the arguments.

Modified:
    llvm/trunk/docs/CMakePrimer.rst

Modified: llvm/trunk/docs/CMakePrimer.rst
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/docs/CMakePrimer.rst?rev=311632&r1=311631&r2=311632&view=diff
==============================================================================
--- llvm/trunk/docs/CMakePrimer.rst (original)
+++ llvm/trunk/docs/CMakePrimer.rst Wed Aug 23 22:38:39 2017
@@ -336,15 +336,15 @@ to the ``macro`` block as well.
 CMake commands can have named arguments that are requried at every call site. In
 addition, all commands will implicitly accept a variable number of extra
 arguments (In C parlance, all commands are varargs functions). When a command is
-invoked with extra arguments (beyond the named ones) CMake will store the extra
-arguments in a list named ``ARGV``, and the count of the extra arguments in
-``ARGN``. Below is a trivial example of providing a wrapper function for CMake's
-built in function ``add_dependencies``.
+invoked with extra arguments (beyond the named ones) CMake will store the full
+list of arguments (both named and unnamed) in a list named ``ARGV``, and the
+sublist of unnamed arguments in ``ARGN``. Below is a trivial example of
+providing a wrapper function for CMake's built in function ``add_dependencies``.
 
 .. code-block:: cmake
 
    function(add_deps target)
-     add_dependencies(${target} ${ARGV})
+     add_dependencies(${target} ${ARGN})
    endfunction()
 
 This example defines a new macro named ``add_deps`` which takes a required first




More information about the llvm-commits mailing list