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