The following commit has been merged in the openafs-stable-1_6_x branch:
commit 27f32e81e4fb8bf7219d732fc9bb053234ba4290
Author: Simon Wilkinson <[email protected]>
Date: Fri Mar 30 19:39:51 2012 +0100
rx: Remove needless 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 the Unix CM to make clang happy.
Reviewed-on: http://gerrit.openafs.org/7088
Tested-by: Simon Wilkinson <[email protected]>
Reviewed-by: Derrick Brashear <[email protected]>
(cherry picked from commit 5e107724f3661254cfdb693ae2d4d1c5238eba7d)
Change-Id: I99a04d9a2c547e34a3daca6f9e6714f6c7b76b9c
Reviewed-on: http://gerrit.openafs.org/10732
Tested-by: BuildBot <[email protected]>
Reviewed-by: Perry Ruiter <[email protected]>
Reviewed-by: Andrew Deason <[email protected]>
Reviewed-by: Derrick Brashear <[email protected]>
Reviewed-by: Stephan Wiesand <[email protected]>
src/rx/rx_user.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
--
OpenAFS Master Repository
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs
