rsyslog/runtime/prop.h
Rainer Gerhards 63a233a1c8
build: normalize object prototypes and remove decl noise
Why:
Keep strict warning builds useful without relying on broad
diagnostic suppressions.

Impact: removes redundant-decls noise while keeping
missing-prototypes at zero in full check builds.

Before/After:
Before mixed macro/local/header declarations; after a
layered prototype model with no warning class regressions.

Technical Overview:
Introduce PROTOTYPEobjConstruct() and standardize new
declarations on PROTOTYPE* macros in obj-types.h.
Split object prototype groups into PROTOTYPEObjBasic() and
PROTOTYPEObjFull(), keeping PROTOTYPEObj() as a compatibility
alias to Basic and using Full where typed ctor/dtor visibility
is valid.
Remove redundant local macro predeclarations for
QueryInterface/Destruct/Construct/DebugPrint/Set methods and
add header-owned prototypes where required.
Add BEGINmodInitNoPredecl() and apply it to built-in tool
modules that already publish modInit prototypes in headers.
Drop the obsolete -Wredundant-decls suppression pragma.

Closes https://github.com/rsyslog/rsyslog/issues/5700

With the help of AI-Agents: Codex
2026-02-07 11:32:00 +01:00

82 lines
3.1 KiB
C

/* The prop object.
*
* This implements props within rsyslog.
*
* Copyright 2009-2026 Adiscon GmbH.
*
* This file is part of the rsyslog runtime library.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
* -or-
* see COPYING.ASL20 in the source distribution
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
#ifndef INCLUDED_PROP_H
#define INCLUDED_PROP_H
#include "atomic.h"
/* the prop object */
struct prop_s {
BEGINobjInstance
; /* Data to implement generic object - MUST be the first data element! */
int iRefCount; /* reference counter */
union {
uchar *psz; /* stored string */
uchar sz[CONF_PROP_BUFSIZE];
} szVal;
int len; /* we use int intentionally, otherwise we may get some troubles... */
DEF_ATOMIC_HELPER_MUT(mutRefCount);
};
/* interfaces */
BEGINinterface(prop) /* name must also be changed in ENDinterface macro! */
INTERFACEObjDebugPrint(prop);
rsRetVal (*Construct)(prop_t **ppThis);
rsRetVal (*ConstructFinalize)(prop_t *pThis);
rsRetVal (*Destruct)(prop_t **ppThis);
rsRetVal (*SetString)(prop_t *pThis, const uchar *psz, const int len);
rsRetVal (*GetString)(prop_t *pThis, uchar **ppsz, int *plen);
int (*GetStringLen)(prop_t *pThis);
rsRetVal (*AddRef)(prop_t *pThis);
rsRetVal (*CreateStringProp)(prop_t **ppThis, const uchar *psz, const int len);
rsRetVal (*CreateOrReuseStringProp)(prop_t **ppThis, const uchar *psz, const int len);
ENDinterface(prop)
#define propCURR_IF_VERSION 1 /* increment whenever you change the interface structure! */
/* get classic c-style string */
/* Note: I know that "static inline" is not the right thing from a C99
* PoV, but some environments treat, even in C99 mode, compile
* non-static inline into the source even if not defined as "extern". This
* obviously results in linker errors. Using "static inline" as below together
* with "__attribute__((unused))" works in all cases. Note also that we
* cannot work around this as we would otherwise need to evaluate
* pThis more than once.
*/
static inline uchar *__attribute__((unused)) ATTR_NONNULL(1) propGetSzStr(prop_t *pThis) {
return (pThis->len < CONF_PROP_BUFSIZE) ? pThis->szVal.sz : pThis->szVal.psz;
}
static inline const char *__attribute__((unused)) propGetSzStrOrDefault(prop_t *pThis, const char *dflt) {
return (pThis != NULL) ? (const char *)propGetSzStr(pThis) : dflt;
}
static inline const char *__attribute__((unused)) szStrOrDefault(const uchar *psz, const char *dflt) {
return (psz != NULL) ? (const char *)psz : dflt;
}
/* prototypes */
PROTOTYPEObjFull(prop);
PROTOTYPEObjDebugPrint(prop);
#endif /* #ifndef INCLUDED_PROP_H */