From: Denys Vlasenko <vda.linux@googlemail.com>
Date: Sun, 29 Jan 2012 20:17:56 +0000 (+0100)
Subject: Trivial optimization
X-Git-Tag: v4.7~182
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3521884c91d0bbd8e796d72619da40bdad95789c;p=strace

Trivial optimization

* strace.c (cleanup): Read 'interrupted' volatile variable only once.

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
---

diff --git a/strace.c b/strace.c
index 8eca57ae..b739cd62 100644
--- a/strace.c
+++ b/strace.c
@@ -1974,7 +1974,12 @@ cleanup(void)
 {
 	int i;
 	struct tcb *tcp;
-	int fatal_sig = interrupted ? interrupted : SIGTERM;
+	int fatal_sig;
+
+	/* 'interrupted' is a volatile object, fetch it only once */
+	fatal_sig = interrupted;
+	if (!fatal_sig)
+		fatal_sig = SIGTERM;
 
 	for (i = 0; i < tcbtabsize; i++) {
 		tcp = tcbtab[i];