]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Remove a redundant argument to xargs. 13093/head
authorDavid Wells <drwells@email.unc.edu>
Thu, 16 Dec 2021 18:02:56 +0000 (13:02 -0500)
committerDavid Wells <drwells@email.unc.edu>
Wed, 22 Dec 2021 17:19:06 +0000 (12:19 -0500)
On GNU, '-I {}' implies '-L 1', i.e., provide one line to each invocation. Since
we are using '-0' (NUL-delimited input) we already implicitly get one item per
line, so the -n 1 is redundant (and also raises a warning).

A simple example: running

    ls --zero | xargs -0 -I {} echo 'aa {}'

prints 'aa ' followed by each item in the current directory - i.e.,
NUL-delimited input really is represented as one line per item.

contrib/utilities/indent_common.sh

index da3144581e2f7aa64ec1eff03e6521791bf0bc2d..068c32e8bbd9682ef63d0c712025f66821f01283 100644 (file)
@@ -259,8 +259,8 @@ fix_permissions()
 export -f fix_permissions
 
 #
-# Collect all files found in a list of directories "${1}$" matching a
-# regular expression "${2}$", and process them with a command "${3}" on 10
+# Collect all files found in a list of directories "${1}" matching a
+# regular expression "${2}", and process them with a command "${3}" on 10
 # threads in parallel.
 #
 # The command line is a bit complicated, so let's discuss the more
@@ -271,9 +271,9 @@ export -f fix_permissions
 #   serves as a good candidate to separate individual file names.
 # - For 'xargs', -0 does the opposite: it separates filenames that are
 #   delimited by \0
-# - the options "-n 1 -P 10" make sure that the following script will be
-#   called exactly with one file name as argument at a time, but we allow
-#   execution for up to 10 times in parallel
+# - the option "-P 10" starts up to 10 processes in parallel. -0 implies '-L 1'
+#   (one argument to each command) so each launch of clang-format corresponds
+#   to exactly one file.
 #
 
 process()
@@ -282,11 +282,11 @@ process()
   case "${OSTYPE}" in
     darwin*)
       find -E ${directories} -regex "${2}" -print0 |
-        xargs -0 -n 1 -P 10 -I {} bash -c "${3} {}"
+        xargs -0 -P 10 -I {} bash -c "${3} {}"
       ;;
     *)
       find ${directories} -regextype egrep -regex "${2}" -print0 |
-        xargs -0 -n 1 -P 10 -I {} bash -c "${3} {}"
+        xargs -0 -P 10 -I {} bash -c "${3} {}"
       ;;
   esac
 }
@@ -318,7 +318,7 @@ process_changed()
       sort -u |
       xargs -n 1 ls -d 2>/dev/null |
       grep -E "^${2}$" |
-      ${XARGS} '\n' -n 1 -P 10 -I {} bash -c "${3} {}"
+      ${XARGS} '\n' -P 10 -I {} bash -c "${3} {}"
 }
 
 #

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.