From 58be19d792e6b8e93fbd49bc4ef04bbb8f4560ef Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Fri, 30 Mar 2012 19:30:18 +0100 Subject: [PATCH] vol: Remove unneeded braces Doing if ((a==b)) is unecessary. It's also potentially dangerous, as that's the syntax required to do assignment within an if statement. clang now issues warnings (errors in -Werror mode) when it encounters these. Remove pointless braces from vol to make clang happy. Change-Id: I805c52fd76de4d028741ae4382b747917581d2f2 Reviewed-on: http://gerrit.openafs.org/7082 Reviewed-by: Derrick Brashear Tested-by: Derrick Brashear --- src/vol/volume.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/vol/volume.c b/src/vol/volume.c index 5a1d5e1b9..d84609245 100644 --- a/src/vol/volume.c +++ b/src/vol/volume.c @@ -8493,7 +8493,7 @@ VLookupVolume_r(Error * ec, VolId volumeId, Volume * hint) /* search the chain for this volume id */ for(queue_Scan(head, vp, np, Volume)) { looks++; - if ((vp->hashid == volumeId)) { + if (vp->hashid == volumeId) { break; } } -- 2.39.5