Thread (10 messages) 10 messages, 3 authors, 2025-05-22

Re: [PATCH v2 2/2] new: Replace "status=0; exit 0" with _exit 0

From: Ritesh Harjani (IBM) <ritesh.list@gmail.com>
Date: 2025-05-17 06:03:41
Also in: fstests, linux-xfs

"Nirjhar Roy (IBM)" [off-list ref] writes:
We should now start using _exit 0 for every new test
that we add.
Right every test now sources common/exit in _begin_test() preamble.
So we may as well start using _exit for new tests.

Looks good to me. Please feel free to add:
Reviewed-by: Ritesh Harjani (IBM) <ritesh.list@gmail.com>

quoted hunk ↗ jump to hunk
Signed-off-by: Nirjhar Roy (IBM) <redacted>
Reviewed-by: Zorro Lang <redacted>
---
 new | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/new b/new
index 636648e2..dff69265 100755
--- a/new
+++ b/new
@@ -176,8 +176,7 @@ exit
 #echo "If failure, check \$seqres.full (this) and \$seqres.full.ok (reference)"
 
 # success, all done
-status=0
-exit
+_exit 0
 End-of-File
 
 sleep 2		# latency to read messages to this point
-- 
2.34.1
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help