diff options
author | Thomas Haller <thaller@redhat.com> | 2020-02-13 14:55:26 +0100 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2020-02-13 17:17:07 +0100 |
commit | cd314370248b2bb199d8643fee09aa05d15a1859 (patch) | |
tree | 677326b4b14cd5ddd64b56bfc7aee3252426849d /src/devices/nm-device-macsec.c | |
parent | 17d9b852c80ceefa37478eea03e844d0d057c4de (diff) | |
download | NetworkManager-cd314370248b2bb199d8643fee09aa05d15a1859.tar.gz |
shared: drop _STATIC variant of macros that define functions
Several macros are used to define function. They had a "_STATIC" variant,
to define the function as static.
I think those macros should not try to abstract entirely what they do.
They should not accept the function scope as argument (or have two
variants per scope). This also because it might make sense to add
additional __attribute__(()) to the function. That only works, if
the macro does not pretend to *not* define a plain function.
Instead, embrace what the function does and let the users place the
function scope as they see fit.
This also follows what is already done with
static NM_CACHED_QUARK_FCN ("autoconnect-root", autoconnect_root_quark)
Diffstat (limited to 'src/devices/nm-device-macsec.c')
-rw-r--r-- | src/devices/nm-device-macsec.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/devices/nm-device-macsec.c b/src/devices/nm-device-macsec.c index c65bfd1566..dbb4ac8efe 100644 --- a/src/devices/nm-device-macsec.c +++ b/src/devices/nm-device-macsec.c @@ -77,7 +77,8 @@ static void macsec_secrets_cancel (NMDeviceMacsec *self); /******************************************************************/ -NM_UTILS_LOOKUP_STR_DEFINE_STATIC (validation_mode_to_string, guint8, +static +NM_UTILS_LOOKUP_STR_DEFINE (validation_mode_to_string, guint8, NM_UTILS_LOOKUP_DEFAULT_WARN ("<unknown>"), NM_UTILS_LOOKUP_STR_ITEM (0, "disable"), NM_UTILS_LOOKUP_STR_ITEM (1, "check"), |