diff options
| author | Lubomir Rintel <lkundrak@v3.sk> | 2022-08-05 17:03:15 +0200 |
|---|---|---|
| committer | Lubomir Rintel <lkundrak@v3.sk> | 2022-08-05 17:47:55 +0200 |
| commit | 7881904eda6a7cd2af7d9670d56bed026d86b5bc (patch) | |
| tree | 62afa35934e341536be25f7b219c5251825ac65f /src/libnm-core-impl/nm-setting-bond.c | |
| parent | 3ce34510402517dfdffb8fc541c97ff8a457f0b9 (diff) | |
| download | NetworkManager-lr/strset.tar.gz | |
all: avoid useless use of NM_IN_STRSET()lr/strset
Sometimes NM_IN_STRSET() is used for comparing two strings. This hampers
readability, making it look like the argument, if it's a macro, expands
to multiple values.
Other than that it's just odd and useless.
Diffstat (limited to 'src/libnm-core-impl/nm-setting-bond.c')
| -rw-r--r-- | src/libnm-core-impl/nm-setting-bond.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/libnm-core-impl/nm-setting-bond.c b/src/libnm-core-impl/nm-setting-bond.c index 2984e6fc22..48a2438b18 100644 --- a/src/libnm-core-impl/nm-setting-bond.c +++ b/src/libnm-core-impl/nm-setting-bond.c @@ -335,11 +335,11 @@ _bond_get_option_normalized(NMSettingBond *self, const char *option, gboolean ge value = _bond_get_option(self, NM_SETTING_BOND_OPTION_NUM_GRAT_ARP); if (!value) value = _bond_get_option(self, NM_SETTING_BOND_OPTION_NUM_UNSOL_NA); - } else if (NM_IN_STRSET(option, NM_SETTING_BOND_OPTION_ACTIVE_SLAVE)) { + } else if (nm_streq(option, NM_SETTING_BOND_OPTION_ACTIVE_SLAVE)) { /* "active_slave" is deprecated, and an alias for "primary". The property * itself always normalizes to %NULL. */ value = NULL; - } else if (NM_IN_STRSET(option, NM_SETTING_BOND_OPTION_PRIMARY)) { + } else if (nm_streq(option, NM_SETTING_BOND_OPTION_PRIMARY)) { /* "active_slave" is deprecated, and an alias for "primary". */ value = _bond_get_option(self, NM_SETTING_BOND_OPTION_PRIMARY); if (!value) |