OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-devel-1_5_77-8-gfb5c2b4

Gerrit Code Review gerrit@openafs.org
Mon, 27 Sep 2010 08:26:16 -0700 (PDT)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit fb5c2b4c1f05743f392890b6b2b90fe94e8df599
Author: Marc Dionne <marc.c.dionne@gmail.com>
Date:   Thu Sep 9 21:02:05 2010 -0400

    Warning fix for gcc 4.5 "operation may be undefined" warnings
    
    The inc_header_word and set_header_word macros make repeated use of their
    argument, which triggers many (~30) warnings with gcc 4.5, like this one:
    
    ./ptutils.c:473:6: warning: operation on ‘cheader.foreigncount’ may be undefined
    
    Removing the cast to afs_int32 in the macros gets rid of the warning,
    and should be safe since we're just getting a small positive integer value
    - the offset of the member in the structure - and passing it to the
    pr_Write function which expects an afs_int32.
    
    Change-Id: Ie493520f874cddbb4b7f9f7ebe4b3922a848d0d0
    Reviewed-on: http://gerrit.openafs.org/2729
    Tested-by: Marc Dionne <marc.c.dionne@gmail.com>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    (cherry picked from commit d69914a5f0580f8048072d3b734558ea5b7b2df5)
    Reviewed-on: http://gerrit.openafs.org/2789
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/ptserver/ptserver.h |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

-- 
OpenAFS Master Repository