OpenAFS Master Repository branch, master, updated. openafs-devel-1_9_1-153-gf9c96d0
Gerrit Code Review
gerrit@openafs.org
Fri, 13 May 2022 02:38:52 -0400
The following commit has been merged in the master branch:
commit f9c96d0fd609e14fcb8ff7d9269024e026f742cb
Author: Marcio Barbosa <mbarbosa@sinenomine.net>
Date: Wed Mar 23 17:58:41 2022 -0300
DARWIN: Declare/include functions before using them
Every function should be explicitly declared before it can be called.
Since -Wimplicit-function-declaration is usually a warning and not an
error, calling undeclared functions does not prevent us from building
the code.
However, apple-clang 12 makes this an error by default, prohibiting the
build of the current version on macOS 11 (Big Sur).
To fix this problem, declare functions before calling them. Also,
include mach/thread_act.h into afs_call.c so the declaration of
thread_terminate() can be found on macOS. Last, given that the third
argument of PIOCTL() (if UKERNEL is defined) is a pointer, cast it to
'long'. Doing so, we can avoid another inhibited warning. Notice that
this PIOCTL definition is scoped to a single file (src/auth/ktc.c).
Change-Id: I6d796c10ea4dd81b13ae5feb9f42608aa445560d
Reviewed-on: https://gerrit.openafs.org/14744
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Cheyenne Wills <cwills@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
CODING | 1 -
src/afs/UKERNEL/afs_usrops.c | 1 -
src/afs/UKERNEL/afs_usrops.h | 3 ++-
src/afs/UKERNEL/afsd_uafs.c | 1 -
src/afs/afs_call.c | 4 ++++
src/afs/afs_prototypes.h | 6 ++++++
src/auth/ktc.c | 5 +++--
src/libafs/MakefileProto.DARWIN.in | 27 ++++++++++++++-------------
src/libuafs/Makefile.common.in | 2 +-
src/rx/DARWIN/rx_kmutex.c | 1 +
src/rx/DARWIN/rx_kmutex.h | 2 ++
src/rx/rx_prototypes.h | 4 +++-
12 files changed, 36 insertions(+), 21 deletions(-)
--
OpenAFS Master Repository