From: Benjamin Kaduk <kaduk@mit.edu>
Date: Thu, 6 Feb 2014 22:27:28 +0000 (-0500)
Subject: fstrace: only declare 'rval' when it is used
X-Git-Tag: upstream/1.6.17^2~12
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=4c17087b8d9f8bd2ad34f308c1208443d60f4e3e;p=packages%2Fo%2Fopenafs.git

fstrace: only declare 'rval' when it is used

... to avoid compiler warnings about unused variables.

Found by clang on FreeBSD 10.0.

Reviewed-on: http://gerrit.openafs.org/10822
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Perry Ruiter <pruiter@sinenomine.net>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
(cherry picked from commit 63291be2216762dd89072f41c9a016608b736ceb)

Change-Id: Ib5d7e14d6077ec2377180b9308d99f49ff79cccc
Reviewed-on: http://gerrit.openafs.org/11777
Reviewed-by: Jeffrey Altman <jaltman@auristor.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
---

diff --git a/src/venus/fstrace.c b/src/venus/fstrace.c
index d5a54cbdc..1d89d0f17 100644
--- a/src/venus/fstrace.c
+++ b/src/venus/fstrace.c
@@ -1117,7 +1117,10 @@ int
 afs_syscall(long call, long parm0, long parm1, long parm2, long parm3,
 	    long parm4, long parm5, long parm6)
 {
-    int code, rval;
+    int code;
+#if defined(AFS_DARWIN80_ENV) || defined(AFS_LINUX20_ENV)
+    int rval;
+#endif
 #ifdef AFS_LINUX20_ENV
 #if defined AFS_LINUX_64BIT_KERNEL
     long long eparm[4];