OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_9-15-g34d8699

Gerrit Code Review gerrit@openafs.org
Fri, 20 Jan 2023 08:58:15 -0500


The following commit has been merged in the openafs-stable-1_8_x branch:
commit 34d869996dda2de2449a1623de051b86e262212c
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Fri Jan 21 17:54:48 2022 -0600

    afs: Organize setpag() declarations better
    
    The current declarations for setpag() are a mess (not to be confused
    with many other setpag functions, like afs_setpag()), accumulated
    across various commits over time. Shuffle the #ifdef logic around, so
    this becomes a much more straightforward #ifdef ladder, with one
    declaration per conditional. And move the LINUX declaration to be next
    to all of the others (even the LINUX setpag() is declared in
    osi_groups.c, not osi_misc.c, so it's in the wrong spot anyway).
    
    The resulting #ifdef logic should be identical to the original code,
    but is now easier to follow. For the BSDs, it may look like we have
    changed the conditional for the case of "XBSD && !DFBSD && !FBSD &&
    !NBSD", but that's a very roundabout way of saying OBSD (OpenBSD).
    
    Reviewed-on: https://gerrit.openafs.org/14883
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit 738a6487ad088068d19a18c6330e01a40f2fd278)
    
    Change-Id: I9dc933fa96fc6e7839c010c5207e644179343898
    Reviewed-on: https://gerrit.openafs.org/15245
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Harish Sharma <Harish.Sharma1@ibm.com>
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 src/afs/afs_prototypes.h | 18 ++++++------------
 1 file changed, 6 insertions(+), 12 deletions(-)

-- 
OpenAFS Master Repository