OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_3-17-g599cf16

Gerrit Code Review gerrit@openafs.org
Mon, 10 Jun 2019 07:48:15 -0400


The following commit has been merged in the openafs-stable-1_8_x branch:
commit 599cf163d10d775a18e2b8a947ad3abd6d1a4021
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Wed Aug 9 20:05:50 2017 -0500

    SOLARIS: Clean up some osi_vnodeops func defs
    
    Currently, the Solaris osi_vnodeops.c file forward-declares many of
    its function definitions, but doesn't declare the arguments. For
    example:
    
        int afs_nfsrdwr();
    
    This avoids type-checking for a few functions that are called before
    they are defined in this file. Furthermore, many of these functions
    are only used within this file, but are not declared 'static'.
    
    To fix this weirdness, remove most of the forward declarations (most
    are not referenced until the function is defined), and fully declare
    the rest. Declare functions 'static' that are not referenced outside
    of this file.
    
    This commit only changes functions up to the 'afs_getsecattr'
    definition. The rest of the file will be fixed in a future commit.
    
    Reviewed-on: https://gerrit.openafs.org/12692
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Marcio Brito Barbosa <mbarbosa@sinenomine.net>
    Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    Tested-by: Mark Vitale <mvitale@sinenomine.net>
    (cherry picked from commit aa46af6ae35e4f026a8ed94012c3bc18c954de23)
    
    Change-Id: I33f445ed253f1991896afff33e7cc14bf5e50b18
    Reviewed-on: https://gerrit.openafs.org/13524
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Marcio Brito Barbosa <mbarbosa@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 src/afs/SOLARIS/osi_vnodeops.c |   64 ++++++++++++++++++++--------------------
 1 files changed, 32 insertions(+), 32 deletions(-)

-- 
OpenAFS Master Repository