groff-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[groff] 02/02: src/roff/groff/tests/*: Remove redundant syntax.


From: G. Branden Robinson
Subject: [groff] 02/02: src/roff/groff/tests/*: Remove redundant syntax.
Date: Fri, 6 Sep 2019 10:28:56 -0400 (EDT)

gbranden pushed a commit to branch master
in repository groff.

commit 6b513fb1e16687f46aadd767c321cef264a7c60a
Author: G. Branden Robinson <address@hidden>
Date:   Sat Sep 7 00:05:26 2019 +1000

    src/roff/groff/tests/*: Remove redundant syntax.
    
    "exit $?" at the end of a shell script means the same thing as just
    plain "exit" which means the same thing as just falling off the end
    of the script.  The exit status of the script is that of the last
    command executed.
    
    I can't find the exact spot in POSIX 2017 that mandates this, but none
    of the following shells screw it up:
        bash 5.0
        dash 0.5.10.2
        ksh 93u+20120801
        mksh 57
        posh 0.13.2
        yash 2.48
        zsh 5.7.1
---
 src/roff/groff/tests/on-latin1-device-oq-is-0x27.sh | 1 -
 src/roff/groff/tests/regression-56555.sh            | 2 --
 2 files changed, 3 deletions(-)

diff --git a/src/roff/groff/tests/on-latin1-device-oq-is-0x27.sh 
b/src/roff/groff/tests/on-latin1-device-oq-is-0x27.sh
index 76076ec..a0d81f1 100755
--- a/src/roff/groff/tests/on-latin1-device-oq-is-0x27.sh
+++ b/src/roff/groff/tests/on-latin1-device-oq-is-0x27.sh
@@ -29,4 +29,3 @@ EOF
 )
 
 diff -u <(echo "$expected") <(echo "$actual")
-exit $?
diff --git a/src/roff/groff/tests/regression-56555.sh 
b/src/roff/groff/tests/regression-56555.sh
index 2beaf3a..009990b 100755
--- a/src/roff/groff/tests/regression-56555.sh
+++ b/src/roff/groff/tests/regression-56555.sh
@@ -24,5 +24,3 @@ groff="${abs_top_builddir:-.}/test-groff"
 "$groff" >/dev/null <<EOF
 \!ta
 EOF
-
-exit $?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]