machine: Clean up includes

Split out of #37344.

diff --git a/src/machine/image-dbus.c b/src/machine/image-dbus.c
index 5ad765237cb0a11e7097bfb0dc836b486d94a811..c91cedf12de54929272b40795d380b4c3136abd3 100644 (file)
--- a/src/machine/image-dbus.c
+++ b/src/machine/image-dbus.c
@@ -1,30 +1,24 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include <sys/file.h>
-#include <sys/mount.h>
+#include "sd-bus.h"
 
 #include "alloc-util.h"
 #include "bus-get-properties.h"
 #include "bus-label.h"
+#include "bus-object.h"
 #include "bus-polkit.h"
 #include "bus-util.h"
-#include "copy.h"
 #include "discover-image.h"
-#include "dissect-image.h"
 #include "fd-util.h"
-#include "fileio.h"
-#include "fs-util.h"
+#include "hashmap.h"
 #include "image-dbus.h"
+#include "image-policy.h"
 #include "io-util.h"
-#include "loop-util.h"
 #include "machined.h"
-#include "mount-util.h"
 #include "operation.h"
 #include "os-util.h"
 #include "process-util.h"
-#include "raw-clone.h"
 #include "strv.h"
-#include "user-util.h"
 
 static BUS_DEFINE_PROPERTY_GET_ENUM(property_get_type, image_type, ImageType);
 

diff --git a/src/machine/image-dbus.h b/src/machine/image-dbus.h
index 38626cffe28961be6cc890506ce71a6e6d16d791..7218a1a095759befe031fa8e797ff10fe0e34be9 100644 (file)
--- a/src/machine/image-dbus.h
+++ b/src/machine/image-dbus.h
@@ -1,7 +1,7 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include "bus-object.h"
+#include "machine-forward.h"
 
 extern const BusObjectImplementation image_object;
 

diff --git a/src/machine/image-varlink.c b/src/machine/image-varlink.c
index c7bc684ab55b85a1c9e921ad3f612eb801fe3a11..82ea7122e33c260b188da12872411a64f290799b 100644 (file)
--- a/src/machine/image-varlink.c
+++ b/src/machine/image-varlink.c
@@ -3,21 +3,19 @@
 #include "sd-json.h"
 #include "sd-varlink.h"
 
-#include "btrfs-util.h"
 #include "bus-polkit.h"
 #include "discover-image.h"
+#include "errno-util.h"
 #include "fd-util.h"
 #include "fileio.h"
 #include "image.h"
 #include "image-varlink.h"
 #include "io-util.h"
 #include "json-util.h"
-#include "machine.h"
 #include "machine-pool.h"
 #include "machined.h"
 #include "operation.h"
 #include "process-util.h"
-#include "string-util.h"
 
 typedef struct ImageUpdateParameters {
         const char *name;

diff --git a/src/machine/image-varlink.h b/src/machine/image-varlink.h
index ab91b5eee149dbaaca5cc0ffa06d30928cd63267..267c4c6da5b5dd84f6b14566accedbc0ffd4fe1d 100644 (file)
--- a/src/machine/image-varlink.h
+++ b/src/machine/image-varlink.h
@@ -1,7 +1,7 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include "sd-varlink.h"
+#include "machine-forward.h"
 
 #define VARLINK_ERROR_MACHINE_IMAGE_NO_SUCH_IMAGE               "io.systemd.MachineImage.NoSuchImage"
 #define VARLINK_ERROR_MACHINE_IMAGE_TOO_MANY_OPERATIONS         "io.systemd.MachineImage.TooManyOperations"

diff --git a/src/machine/image.c b/src/machine/image.c
index 41b40854d795252b3868c55ec12f96411d100231..3e903b3f3f7ca9e81ce579c9919c6bf8596c3628 100644 (file)
--- a/src/machine/image.c
+++ b/src/machine/image.c
@@ -1,12 +1,15 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include "sd-json.h"
+#include "sd-bus.h"
 
 #include "discover-image.h"
+#include "errno-util.h"
 #include "fd-util.h"
 #include "fileio.h"
+#include "hashmap.h"
 #include "image.h"
 #include "io-util.h"
+#include "log.h"
 #include "machined.h"
 #include "operation.h"
 #include "process-util.h"

diff --git a/src/machine/image.h b/src/machine/image.h
index fc8de564e8b3fe6839f8ed27e3def4d9d56e249e..353cdd68281fa09be5a9db31bfbd6f9beb38c923 100644 (file)
--- a/src/machine/image.h
+++ b/src/machine/image.h
@@ -1,14 +1,7 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include <errno.h>
-
-#include "sd-bus.h"
-
-#include "macro.h"
-
-typedef struct Manager Manager;
-typedef struct Operation Operation;
+#include "machine-forward.h"
 
 typedef enum ImageCleanPoolMode {
         IMAGE_CLEAN_POOL_REMOVE_ALL,

diff --git a/src/machine/machine-dbus.c b/src/machine/machine-dbus.c
index de839099e08b3a63645bd2dd0fb77f892460e531..c19b1a5beb0d5813553e9f48ac4fb7073fe1b98a 100644 (file)
--- a/src/machine/machine-dbus.c
+++ b/src/machine/machine-dbus.c
@@ -1,41 +1,30 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include <errno.h>
-#include <sys/mount.h>
-#include <sys/wait.h>
+#include "sd-bus.h"
 
 #include "alloc-util.h"
 #include "bus-common-errors.h"
 #include "bus-get-properties.h"
-#include "bus-internal.h"
 #include "bus-label.h"
-#include "bus-locator.h"
+#include "bus-object.h"
 #include "bus-polkit.h"
 #include "bus-util.h"
 #include "copy.h"
-#include "env-file.h"
 #include "env-util.h"
+#include "errno-util.h"
 #include "fd-util.h"
-#include "fileio.h"
-#include "format-util.h"
-#include "fs-util.h"
+#include "hashmap.h"
 #include "in-addr-util.h"
 #include "local-addresses.h"
 #include "machine.h"
 #include "machine-dbus.h"
 #include "machined.h"
-#include "mkdir.h"
 #include "mount-util.h"
-#include "mountpoint-util.h"
-#include "namespace-util.h"
 #include "operation.h"
-#include "os-util.h"
 #include "path-util.h"
-#include "process-util.h"
 #include "signal-util.h"
+#include "string-util.h"
 #include "strv.h"
-#include "terminal-util.h"
-#include "user-util.h"
 
 static BUS_DEFINE_PROPERTY_GET_ENUM(property_get_class, machine_class, MachineClass);
 static BUS_DEFINE_PROPERTY_GET2(property_get_state, "s", Machine, machine_get_state, machine_state_to_string);

diff --git a/src/machine/machine-dbus.h b/src/machine/machine-dbus.h
index 72b6984660dc4ebf0f855226b037f45eec636b24..7fd025f7b3102f0ab2d64612958c4e333088bd06 100644 (file)
--- a/src/machine/machine-dbus.h
+++ b/src/machine/machine-dbus.h
@@ -1,11 +1,7 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include "sd-bus.h"
-
-#include "bus-object.h"
-
-typedef struct Machine Machine;
+#include "machine-forward.h"
 
 typedef enum {
         MACHINE_COPY_REPLACE = 1 << 0, /* Public API via DBUS, do not change */

diff --git a/src/machine/machine-forward.h b/src/machine/machine-forward.h
new file mode 100644 (file)
index 0000000..81c3370
--- /dev/null
+++ b/src/machine/machine-forward.h
@@ -0,0 +1,8 @@
+/* SPDX-License-Identifier: LGPL-2.1-or-later */
+#pragma once
+
+#include "forward.h" // IWYU pragma: export
+
+typedef struct Machine Machine;
+typedef struct Manager Manager;
+typedef struct Operation Operation;

diff --git a/src/machine/machine-varlink.c b/src/machine/machine-varlink.c
index 7ee1a356cab7f8ac4f13e0d2829af9e0abb0c6e7..1c2013de51baea13383dfe82f38017b5b8948ddf 100644 (file)
--- a/src/machine/machine-varlink.c
+++ b/src/machine/machine-varlink.c
@@ -1,14 +1,14 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include <limits.h>
-
-#include "sd-id128.h"
 #include "sd-json.h"
 #include "sd-varlink.h"
 
 #include "bus-polkit.h"
 #include "copy.h"
+#include "errno-util.h"
 #include "fd-util.h"
+#include "format-util.h"
+#include "hashmap.h"
 #include "hostname-util.h"
 #include "json-util.h"
 #include "machine.h"
@@ -16,10 +16,7 @@
 #include "machined.h"
 #include "mount-util.h"
 #include "operation.h"
-#include "path-util.h"
 #include "pidref.h"
-#include "process-util.h"
-#include "signal-util.h"
 #include "socket-util.h"
 #include "string-table.h"
 #include "string-util.h"

diff --git a/src/machine/machine-varlink.h b/src/machine/machine-varlink.h
index 83f396d708bb702b49c077e58a4e5622da8a8c54..7a0004ea9024ce39d0b589eb1e9df9b28ae9c336 100644 (file)
--- a/src/machine/machine-varlink.h
+++ b/src/machine/machine-varlink.h
@@ -3,10 +3,7 @@
 
 #include "sd-varlink.h"
 
-#include "pidref.h"
-
-typedef struct Machine Machine;
-typedef struct Manager Manager;
+#include "machine-forward.h"
 
 #define VARLINK_ERROR_MACHINE_NO_SUCH_MACHINE           "io.systemd.Machine.NoSuchMachine"
 #define VARLINK_ERROR_MACHINE_EXISTS                    "io.systemd.Machine.MachineExists"

diff --git a/src/machine/machine.c b/src/machine/machine.c
index 09c10dc3501676a7968d3374512f55950660e3b9..9f78a9c7d33b0a2c3de973416a691358fd953650 100644 (file)
--- a/src/machine/machine.c
+++ b/src/machine/machine.c
@@ -1,9 +1,11 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include <errno.h>
+#include <sched.h>
 #include <sys/stat.h>
 #include <unistd.h>
 
+#include "sd-bus.h"
+#include "sd-event.h"
 #include "sd-messages.h"
 
 #include "alloc-util.h"
@@ -11,7 +13,6 @@
 #include "bus-internal.h"
 #include "bus-locator.h"
 #include "bus-unit-util.h"
-#include "bus-util.h"
 #include "env-file.h"
 #include "errno-util.h"
 #include "escape.h"
@@ -21,6 +22,7 @@
 #include "format-util.h"
 #include "fs-util.h"
 #include "hashmap.h"
+#include "log.h"
 #include "machine.h"
 #include "machine-dbus.h"
 #include "machined.h"
@@ -36,6 +38,7 @@
 #include "stdio-util.h"
 #include "string-table.h"
 #include "string-util.h"
+#include "strv.h"
 #include "terminal-util.h"
 #include "tmpfile-util.h"
 #include "uid-range.h"

diff --git a/src/machine/machine.h b/src/machine/machine.h
index a4b375ca32e5ebbe8bbf9d3317caca7581b2bc4f..870146fc2251e90cc718a728f09037a05369ea5e 100644 (file)
--- a/src/machine/machine.h
+++ b/src/machine/machine.h
@@ -1,19 +1,14 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include "sd-bus.h"
-#include "sd-event.h"
 #include "sd-id128.h"
 
 #include "copy.h"
 #include "list.h"
+#include "machine-forward.h"
 #include "pidref.h"
 #include "time-util.h"
 
-typedef struct Machine Machine;
-typedef struct Manager Manager;
-typedef struct Operation Operation;
-
 typedef enum MachineState {
         MACHINE_OPENING,    /* Machine is being registered */
         MACHINE_RUNNING,    /* Machine is running */
@@ -37,7 +32,7 @@ typedef enum KillWhom {
         _KILL_WHOM_INVALID = -EINVAL,
 } KillWhom;
 
-struct Machine {
+typedef struct Machine {
         Manager *manager;
 
         char *name;
@@ -75,7 +70,7 @@ struct Machine {
         LIST_HEAD(Operation, operations);
 
         LIST_FIELDS(Machine, gc_queue);
-};
+} Machine;
 
 int machine_new(MachineClass class, const char *name, Machine **ret);
 int machine_link(Manager *manager, Machine *machine);

diff --git a/src/machine/machinectl.c b/src/machine/machinectl.c
index 4cd670748e1aa2e61dbea46d6eb3e7a3e796296d..07fdb996e9b533c672d4b125adfe7bfa45a938ef 100644 (file)
--- a/src/machine/machinectl.c
+++ b/src/machine/machinectl.c
@@ -1,17 +1,14 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include <arpa/inet.h>
-#include <errno.h>
-#include <fcntl.h>
 #include <getopt.h>
-#include <math.h>
+#include <locale.h>
 #include <net/if.h>
-#include <netinet/in.h>
-#include <sys/mount.h>
 #include <sys/socket.h>
 #include <unistd.h>
 
 #include "sd-bus.h"
+#include "sd-event.h"
+#include "sd-journal.h"
 
 #include "alloc-util.h"
 #include "ask-password-agent.h"
@@ -28,24 +25,19 @@
 #include "bus-wait-for-jobs.h"
 #include "cgroup-show.h"
 #include "cgroup-util.h"
-#include "constants.h"
-#include "copy.h"
 #include "edit-util.h"
 #include "env-util.h"
-#include "fd-util.h"
+#include "format-util.h"
 #include "format-ifname.h"
 #include "format-table.h"
 #include "hostname-util.h"
 #include "import-util.h"
 #include "in-addr-util.h"
 #include "label-util.h"
-#include "locale-util.h"
 #include "log.h"
 #include "logs-show.h"
 #include "machine-dbus.h"
-#include "macro.h"
 #include "main-func.h"
-#include "mkdir.h"
 #include "nulstr-util.h"
 #include "osc-context.h"
 #include "pager.h"
@@ -56,16 +48,15 @@
 #include "pretty-print.h"
 #include "process-util.h"
 #include "ptyfwd.h"
-#include "rlimit-util.h"
-#include "signal-util.h"
-#include "sort-util.h"
+#include "runtime-scope.h"
 #include "stdio-util.h"
 #include "string-table.h"
+#include "string-util.h"
 #include "strv.h"
 #include "terminal-util.h"
+#include "time-util.h"
 #include "unit-name.h"
 #include "verbs.h"
-#include "web-util.h"
 
 typedef enum MachineRunner {
         RUNNER_NSPAWN,

diff --git a/src/machine/machined-core.c b/src/machine/machined-core.c
index d5804b54abd084fa47910818cb6ef5f2c9075c18..b87974a5dc2eefb2b1abd92345c027acd93c2b58 100644 (file)
--- a/src/machine/machined-core.c
+++ b/src/machine/machined-core.c
@@ -1,19 +1,20 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
+#include "sd-event.h"
 #include "cgroup-util.h"
 #include "copy.h"
 #include "discover-image.h"
 #include "env-file.h"
 #include "fd-util.h"
 #include "fileio.h"
-#include "iovec-util.h"
+#include "hashmap.h"
+#include "local-addresses.h"
 #include "log.h"
 #include "machine.h"
 #include "machined.h"
 #include "namespace-util.h"
 #include "os-util.h"
 #include "process-util.h"
-#include "socket-util.h"
 #include "strv.h"
 #include "user-util.h"
 

diff --git a/src/machine/machined-dbus.c b/src/machine/machined-dbus.c
index 3be1ec0cddd8861f2624c5931da406f82eef5d7b..014c0267637c7c9ab5220ebef84cb6dd3ba4fa07 100644 (file)
--- a/src/machine/machined-dbus.c
+++ b/src/machine/machined-dbus.c
@@ -1,8 +1,8 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include <errno.h>
 #include <unistd.h>
 
+#include "sd-bus.h"
 #include "sd-id128.h"
 
 #include "alloc-util.h"
@@ -11,6 +11,7 @@
 #include "bus-get-properties.h"
 #include "bus-locator.h"
 #include "bus-message-util.h"
+#include "bus-object.h"
 #include "bus-polkit.h"
 #include "bus-util.h"
 #include "cgroup-util.h"
@@ -19,6 +20,7 @@
 #include "fd-util.h"
 #include "fileio.h"
 #include "format-util.h"
+#include "hashmap.h"
 #include "hostname-util.h"
 #include "image.h"
 #include "image-dbus.h"
@@ -30,10 +32,9 @@
 #include "operation.h"
 #include "os-util.h"
 #include "path-util.h"
-#include "process-util.h"
-#include "stdio-util.h"
+#include "string-util.h"
 #include "strv.h"
-#include "unit-name.h"
+#include "unit-def.h"
 #include "user-util.h"
 
 static BUS_DEFINE_PROPERTY_GET_GLOBAL(property_get_pool_path, "s", "/var/lib/machines");

diff --git a/src/machine/machined-varlink.c b/src/machine/machined-varlink.c
index 5aeb0f797c17103afcf81ca58ded85cc87ab973c..b429ceb974c69eb04959442d5b4d34eb5ce7c8bd 100644 (file)
--- a/src/machine/machined-varlink.c
+++ b/src/machine/machined-varlink.c
@@ -1,20 +1,25 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
+#include <linux/vm_sockets.h>
+
+#include "sd-event.h"
 #include "sd-varlink.h"
 
 #include "bus-polkit.h"
 #include "discover-image.h"
+#include "errno-util.h"
 #include "format-util.h"
-#include "hostname-util.h"
+#include "hashmap.h"
+#include "image-policy.h"
 #include "image-varlink.h"
 #include "json-util.h"
+#include "local-addresses.h"
 #include "machine.h"
 #include "machine-varlink.h"
 #include "machined.h"
 #include "machined-varlink.h"
-#include "mkdir.h"
-#include "process-util.h"
-#include "socket-util.h"
+#include "string-util.h"
+#include "strv.h"
 #include "user-util.h"
 #include "varlink-io.systemd.Machine.h"
 #include "varlink-io.systemd.MachineImage.h"

diff --git a/src/machine/machined-varlink.h b/src/machine/machined-varlink.h
index b96700719bf4721580235820b282d457277b4eaa..cdf3a582dc8876de871113119f4f070e670f0c14 100644 (file)
--- a/src/machine/machined-varlink.h
+++ b/src/machine/machined-varlink.h
@@ -1,7 +1,7 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-typedef struct Manager Manager;
+#include "machine-forward.h"
 
 int manager_varlink_init(Manager *m);
 void manager_varlink_done(Manager *m);

diff --git a/src/machine/machined.c b/src/machine/machined.c
index 722b2363dcc3736b6057c9d8d9849e2ccf6bd5b7..7f463f174bdf477eedf707e450d71dc59bd8983d 100644 (file)
--- a/src/machine/machined.c
+++ b/src/machine/machined.c
@@ -1,24 +1,25 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include <errno.h>
 #include <string.h>
 #include <sys/stat.h>
-#include <sys/types.h>
-#include <unistd.h>
+
+#include "sd-bus.h"
+#include "sd-event.h"
+#include "sd-varlink.h"
 
 #include "alloc-util.h"
-#include "bus-error.h"
 #include "bus-locator.h"
 #include "bus-log-control-api.h"
-#include "bus-polkit.h"
+#include "bus-object.h"
 #include "bus-util.h"
-#include "cgroup-util.h"
 #include "common-signal.h"
+#include "constants.h"
 #include "daemon-util.h"
 #include "dirent-util.h"
-#include "discover-image.h"
+#include "errno-util.h"
 #include "fd-util.h"
-#include "format-util.h"
+#include "hash-funcs.h"
+#include "hashmap.h"
 #include "hostname-util.h"
 #include "machine.h"
 #include "machined.h"
@@ -26,11 +27,11 @@
 #include "main-func.h"
 #include "mkdir-label.h"
 #include "operation.h"
-#include "process-util.h"
 #include "service-util.h"
 #include "signal-util.h"
 #include "socket-util.h"
 #include "special.h"
+#include "string-util.h"
 
 static Manager* manager_unref(Manager *m);
 DEFINE_TRIVIAL_CLEANUP_FUNC(Manager*, manager_unref);

diff --git a/src/machine/machined.h b/src/machine/machined.h
index 10de674e7f7bdebce6b56f168e8ffb56dcb9ef7a..b1d9ce1d5534fc4f59207005e6c28a942f9e5c7e 100644 (file)
--- a/src/machine/machined.h
+++ b/src/machine/machined.h
@@ -1,23 +1,10 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include <stdbool.h>
-
-#include "sd-bus.h"
-#include "sd-event.h"
-#include "sd-varlink.h"
-
-#include "bus-object.h"
-#include "hashmap.h"
 #include "list.h"
-#include "local-addresses.h"
-#include "pidref.h"
+#include "machine-forward.h"
 #include "runtime-scope.h"
 
-typedef struct Image Image;
-typedef struct Machine Machine;
-typedef struct Operation Operation;
-
 typedef struct Manager {
         sd_event *event;
         sd_bus *bus;

diff --git a/src/machine/operation.c b/src/machine/operation.c
index 22168a3670f3c3ed8a6159e8e798fb8e1235986f..a624cb35fd72967f573450b9fa83258c2718e8e0 100644 (file)
--- a/src/machine/operation.c
+++ b/src/machine/operation.c
@@ -3,8 +3,13 @@
 #include <sys/wait.h>
 #include <unistd.h>
 
+#include "sd-bus.h"
+#include "sd-event.h"
+#include "sd-varlink.h"
+
 #include "alloc-util.h"
 #include "fd-util.h"
+#include "format-util.h"
 #include "log.h"
 #include "machine.h"
 #include "machined.h"
@@ -158,3 +163,21 @@ Operation *operation_free(Operation *o) {
 
         return mfree(o);
 }
+
+void operation_attach_bus_reply(Operation *op, sd_bus_message *message) {
+        assert(op);
+        assert(!op->message);
+        assert(!op->link);
+        assert(message);
+
+        op->message = sd_bus_message_ref(message);
+}
+
+void operation_attach_varlink_reply(Operation *op, sd_varlink *link) {
+        assert(op);
+        assert(!op->message);
+        assert(!op->link);
+        assert(link);
+
+        op->link = sd_varlink_ref(link);
+}

diff --git a/src/machine/operation.h b/src/machine/operation.h
index 330ef0bf98b8f143e2944fe9e5299d97cef87938..2425694aff1a7ff057575ca1756782689a92ad00 100644 (file)
--- a/src/machine/operation.h
+++ b/src/machine/operation.h
@@ -1,22 +1,12 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include <sys/types.h>
-
-#include "sd-bus.h"
-#include "sd-event.h"
-#include "sd-varlink.h"
-
-#include "assert-util.h"
 #include "list.h"
-
-typedef struct Machine Machine;
-typedef struct Manager Manager;
-typedef struct Operation Operation;
+#include "machine-forward.h"
 
 #define OPERATIONS_MAX 64
 
-struct Operation {
+typedef struct Operation {
         Manager *manager;
         Machine *machine;
         pid_t pid;
@@ -31,28 +21,13 @@ struct Operation {
         int (*done)(Operation *o, int ret, sd_bus_error *error);
         LIST_FIELDS(Operation, operations);
         LIST_FIELDS(Operation, operations_by_machine);
-};
+} Operation;
 
 int operation_new(Manager *manager, Machine *machine, pid_t child, int errno_fd, Operation **ret);
 Operation *operation_free(Operation *o);
 
-static inline void operation_attach_bus_reply(Operation *op, sd_bus_message *message) {
-        assert(op);
-        assert(!op->message);
-        assert(!op->link);
-        assert(message);
-
-        op->message = sd_bus_message_ref(message);
-}
-
-static inline void operation_attach_varlink_reply(Operation *op, sd_varlink *link) {
-        assert(op);
-        assert(!op->message);
-        assert(!op->link);
-        assert(link);
-
-        op->link = sd_varlink_ref(link);
-}
+void operation_attach_bus_reply(Operation *op, sd_bus_message *message);
+void operation_attach_varlink_reply(Operation *op, sd_varlink *link);
 
 static inline int operation_new_with_bus_reply(
                 Manager *manager,