refactor: remove unnecessary file descriptor number 1 in redirections ➡️

Author: oldratlee

bin/cp-into-docker-run

@@ -29,7 +29,7 @@ redPrint() {
 }
 
 die() {
-  redPrint "Error: $*" 1>&2
+  redPrint "Error: $*" >&2
   exit 1
 }
 
@@ -242,7 +242,7 @@ trap cleanupWhenExit EXIT
 ########################################
 
 logAndRun() {
-  $verbose && printf '%s\n' "[$PROG] $*" 1>&2
+  $verbose && printf '%s\n' "[$PROG] $*" >&2
   "$@"
 }
 

bin/show-busy-java-threads

@@ -101,7 +101,7 @@ blueOutput() {
 }
 
 die() {
-  redOutput "Error: $*" 1>&2
+  redOutput "Error: $*" >&2
   exit 1
 }
 

bin/uq

@@ -60,7 +60,7 @@ yellowPrint() {
 }
 
 die() {
-  redPrint "Error: $*" 1>&2
+  redPrint "Error: $*" >&2
   exit 1
 }
 

legacy-bin/cp-svn-url

@@ -78,7 +78,7 @@ readonly dir="${1:-.}"
 # NOTE: DO NOT declare var url as readonly in ONE line!
 url="$(svn info "${dir}" | awk '/^URL: /{print $2}')"
 if [ -z "${url}" ]; then
-  echo "Fail to get svn url!" 1>&2
+  echo "Fail to get svn url!" >&2
   exit 1
 fi
 

test-cases/bump-scripts-version.sh

@@ -49,7 +49,7 @@ logAndRun() {
 }
 
 die() {
-  redPrint "Error: $*" 1>&2
+  redPrint "Error: $*" >&2
   exit 1
 }
 

test-cases/my_unit_test_lib.sh

@@ -40,7 +40,7 @@ fail() {
 }
 
 die() {
-  redEcho "Error: $*" 1>&2
+  redEcho "Error: $*" >&2
   exit 1
 }