OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-3056-ge93b6dc
Gerrit Code Review
gerrit@openafs.org
Mon, 29 Oct 2012 09:12:42 -0700 (PDT)
The following commit has been merged in the master branch:
commit e93b6dca2f593e479499122eedef1e65452aaaac
Author: Simon Wilkinson <sxw@your-file-system.com>
Date: Thu Oct 25 11:27:33 2012 +0100
rx: Don't double check conn->call
We currently have
call = conn->call[channel]
if (call) {
...
} else {
call = conn->call[channel]
if (call) {
...
}
}
As we don't drop (or acquire) any locks between the first and the
second check of call, there's no way that the result can be different
from the first time we checked. So just get rid of the uneccessary
code, and reindent the following block to match.
Change-Id: If21a1d0af461fe0c0651e713dfb99a7c5f01cba4
Reviewed-on: http://gerrit.openafs.org/8290
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
src/rx/rx.c | 66 +++++++++++++++++++++++++++-------------------------------
1 files changed, 31 insertions(+), 35 deletions(-)
--
OpenAFS Master Repository