The following commit has been merged in the openafs-stable-1_8_x branch:
commit 599cf163d10d775a18e2b8a947ad3abd6d1a4021
Author: Andrew Deason <[email protected]>
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 <[email protected]>
    Reviewed-by: Michael Meffie <[email protected]>
    Reviewed-by: Marcio Brito Barbosa <[email protected]>
    Reviewed-by: Cheyenne Wills <[email protected]>
    Reviewed-by: Benjamin Kaduk <[email protected]>
    Tested-by: Mark Vitale <[email protected]>
    (cherry picked from commit aa46af6ae35e4f026a8ed94012c3bc18c954de23)
    
    Change-Id: I33f445ed253f1991896afff33e7cc14bf5e50b18
    Reviewed-on: https://gerrit.openafs.org/13524
    Reviewed-by: Michael Meffie <[email protected]>
    Reviewed-by: Andrew Deason <[email protected]>
    Reviewed-by: Cheyenne Wills <[email protected]>
    Reviewed-by: Mark Vitale <[email protected]>
    Reviewed-by: Marcio Brito Barbosa <[email protected]>
    Tested-by: BuildBot <[email protected]>
    Reviewed-by: Stephan Wiesand <[email protected]>

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

-- 
OpenAFS Master Repository
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs

Reply via email to