[tor-commits] [tor/release-0.4.6] Stop using the "x$FOO" idiom in git-resquash.sh

nickm at torproject.org nickm at torproject.org
Fri May 28 12:04:00 UTC 2021


commit 8b6e919086ad6dde681516fe52d744afa3ffcc89
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri May 28 08:02:08 2021 -0400

    Stop using the "x$FOO" idiom in git-resquash.sh
    
    The new version of shellcheck says that that this idiom is
    unnecessary, and its presence here is making the shellcheck tests
    fail.
    
    No changes file needed, since this is not user-facing code.
---
 scripts/git/git-resquash.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/scripts/git/git-resquash.sh b/scripts/git/git-resquash.sh
index e0f26ecdc4..f37950c9b0 100755
--- a/scripts/git/git-resquash.sh
+++ b/scripts/git/git-resquash.sh
@@ -7,7 +7,7 @@ set -e
 
 PARENT="$1"
 
-if test "x$PARENT" = "x"; then
+if test "$PARENT" = ""; then
     echo "You must specify the parent branch."
     exit 1
 fi
@@ -32,12 +32,12 @@ else
     USE_KEEP_BASE=1
 fi
 
-if test "x$USE_KEEP_BASE" = "x1" ; then
+if test "$USE_KEEP_BASE" = "1" ; then
     exec git rebase -i --autosquash --keep-base "${PARENT}"
 else
     REV=$(git log --reverse --format='%H' "${PARENT}..HEAD" | head -1)
 
-    if test "x${REV}" = "x"; then
+    if test "${REV}" = ""; then
         echo "No changes here since ${PARENT}"
         exit 1
     fi





More information about the tor-commits mailing list