Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 12 Jan 2021 18:40:50 GMT
From:      Mariusz Zaborski <oshogbo@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: d2ceee38ca26 - main - casper: convert macros to inline functions
Message-ID:  <202101121840.10CIeoQd038386@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by oshogbo:

URL: https://cgit.FreeBSD.org/src/commit/?id=d2ceee38ca26d8ea48b8dfd9018636eadd41fbc4

commit d2ceee38ca26d8ea48b8dfd9018636eadd41fbc4
Author:     Mariusz Zaborski <oshogbo@FreeBSD.org>
AuthorDate: 2021-01-12 18:38:10 +0000
Commit:     Mariusz Zaborski <oshogbo@FreeBSD.org>
CommitDate: 2021-01-12 18:40:45 +0000

    casper: convert macros to inline functions
    
    In libcasper, the first argument to the function is a structure that
    represents a connection to Casper. On systems without Casper, macros
    are used to interpose the Casper functions to standard libc ones.
    This may cause errors/warnings that the variable is not used.
    With the inline function, there is no such problem.
    
    I omitted this file in: 8c121177f063a187534dcd475b136c34474802cd
---
 lib/libcasper/libcasper/libcasper.h | 25 ++++++++++++++++++++++---
 1 file changed, 22 insertions(+), 3 deletions(-)

diff --git a/lib/libcasper/libcasper/libcasper.h b/lib/libcasper/libcasper/libcasper.h
index c8ed184d92cd..b55619b01506 100644
--- a/lib/libcasper/libcasper/libcasper.h
+++ b/lib/libcasper/libcasper/libcasper.h
@@ -123,8 +123,21 @@ cap_channel_t	*cap_service_open(const cap_channel_t *chan, const char *name);
 int		 cap_service_limit(const cap_channel_t *chan,
 		    const char * const *names, size_t nnames);
 #else
-#define	cap_service_open(chan, name)		(cap_init())
-#define	cap_service_limit(chan, names, nnames)	(0)
+static inline cap_channel_t *
+cap_service_open(const cap_channel_t *chan __unused,
+    const char *name __unused)
+{
+
+	return (cap_init());
+}
+
+static inline int
+cap_service_limit(const cap_channel_t *chan __unused,
+    const char * const *names __unused, size_t nnames __unused)
+{
+
+	return (0);
+}
 #endif
 
 /*
@@ -229,7 +242,13 @@ int	cap_sock(const cap_channel_t *chan);
 #ifdef WITH_CASPER
 int	cap_limit_set(const cap_channel_t *chan, nvlist_t *limits);
 #else
-#define	cap_limit_set(chan, limits)	(0)
+static inline int
+cap_limit_set(const cap_channel_t *chan __unused,
+    nvlist_t *limits __unused)
+{
+
+	return (0);
+}
 #endif
 
 /*



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202101121840.10CIeoQd038386>