diff options
author | Lennart Poettering <lennart@poettering.net> | 2014-07-17 01:07:17 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2014-07-17 01:41:52 +0200 |
commit | 0014a4ad505d119c7ac4346d9d774c3f17f663a5 (patch) | |
tree | 78670c7fa0bb772e1e334c6f520478067b9c5531 /src/network | |
parent | 260b6323cc2aebe722d5b8e43fa63a502f906774 (diff) |
sd-network: fix parameter order for sd_network_monitor_new()
Constructors should return the object they created as first parameter,
except when they are generated as a child/member object of some other
object in which case that should be first.
Diffstat (limited to 'src/network')
-rw-r--r-- | src/network/networkd-wait-online.c | 2 | ||||
-rw-r--r-- | src/network/sd-network.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/network/networkd-wait-online.c b/src/network/networkd-wait-online.c index c6038c63bf..c4783ca2e9 100644 --- a/src/network/networkd-wait-online.c +++ b/src/network/networkd-wait-online.c @@ -252,7 +252,7 @@ int main(int argc, char *argv[]) { goto out; } - r = sd_network_monitor_new(NULL, &m->monitor); + r = sd_network_monitor_new(&m->monitor, NULL); if (r < 0) { log_error("Could not create monitor: %s", strerror(-r)); goto out; diff --git a/src/network/sd-network.c b/src/network/sd-network.c index 3ebc5d8b13..2057e53774 100644 --- a/src/network/sd-network.c +++ b/src/network/sd-network.c @@ -316,7 +316,7 @@ static inline sd_network_monitor* FD_TO_MONITOR(int fd) { return (sd_network_monitor*) (unsigned long) (fd + 1); } -_public_ int sd_network_monitor_new(const char *category, sd_network_monitor **m) { +_public_ int sd_network_monitor_new(sd_network_monitor **m, const char *category) { int fd, k; bool good = false; |