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.
Change-Id: I0f1558b42a487e1de722b77f8423500365a20eee
Reviewed-on: http://gerrit.openafs.org/7074
Reviewed-by: Derrick Brashear <shadow@dementix.org>
Tested-by: Derrick Brashear <shadow@dementix.org>
writelocked = (0 == NBObtainWriteLock(&afs_xdnlc, 3));
for (i = 0; i < NCSIZE; i++) {
- if ((nameCache[i].vp == avc)) {
+ if (nameCache[i].vp == avc) {
nameCache[i].dirp = nameCache[i].vp = NULL;
/* can't simply break; because of hard links -- might be two */
/* different entries with same vnode */