From 209d4bba0deb88e5224b8fedd973909d46c1e80d Mon Sep 17 00:00:00 2001 From: Philip Withnall Date: Thu, 23 Feb 2023 16:04:47 +0000 Subject: [PATCH] transaction: Remove unused error argument from flatpak_transaction_add_op() Introduces no functional changes, but simplifies the code a little. Signed-off-by: Philip Withnall --- common/flatpak-transaction.c | 30 ++++++++---------------------- 1 file changed, 8 insertions(+), 22 deletions(-) diff --git a/common/flatpak-transaction.c b/common/flatpak-transaction.c index 420e3d59..0cecd564 100644 --- a/common/flatpak-transaction.c +++ b/common/flatpak-transaction.c @@ -2113,8 +2113,7 @@ flatpak_transaction_add_op (FlatpakTransaction *self, const char *commit, GFile *bundle, FlatpakTransactionOperationType kind, - gboolean pin_on_deploy, - GError **error) + gboolean pin_on_deploy) { FlatpakTransactionPrivate *priv = flatpak_transaction_get_instance_private (self); FlatpakTransactionOperation *op; @@ -2252,10 +2251,7 @@ add_related (FlatpakTransaction *self, related_op = flatpak_transaction_add_op (self, rel->remote, rel->ref, NULL, NULL, NULL, NULL, FLATPAK_TRANSACTION_OPERATION_UNINSTALL, - FALSE, error); - if (related_op == NULL) - return FALSE; - + FALSE); related_op->non_fatal = TRUE; related_op->fail_if_op_fails = op; flatpak_transaction_operation_add_related_to_op (related_op, op); @@ -2284,10 +2280,7 @@ add_related (FlatpakTransaction *self, (const char **) rel->subpaths, NULL, NULL, NULL, FLATPAK_TRANSACTION_OPERATION_INSTALL_OR_UPDATE, - FALSE, error); - if (related_op == NULL) - return FALSE; - + FALSE); related_op->non_fatal = TRUE; related_op->fail_if_op_fails = op; flatpak_transaction_operation_add_related_to_op (related_op, op); @@ -2519,9 +2512,7 @@ add_new_dep_op (FlatpakTransaction *self, return FALSE; *dep_op = flatpak_transaction_add_op (self, dep_remote, dep_ref, NULL, NULL, NULL, NULL, - FLATPAK_TRANSACTION_OPERATION_INSTALL_OR_UPDATE, FALSE, error); - if (*dep_op == NULL) - return FALSE; + FLATPAK_TRANSACTION_OPERATION_INSTALL_OR_UPDATE, FALSE); } else { @@ -2531,9 +2522,7 @@ add_new_dep_op (FlatpakTransaction *self, g_info ("Updating dependency %s of %s", flatpak_decomposed_get_pref (dep_ref), flatpak_decomposed_get_pref (op->ref)); *dep_op = flatpak_transaction_add_op (self, dep_remote, dep_ref, NULL, NULL, NULL, NULL, - FLATPAK_TRANSACTION_OPERATION_UPDATE, FALSE, error); - if (*dep_op == NULL) - return FALSE; + FLATPAK_TRANSACTION_OPERATION_UPDATE, FALSE); (*dep_op)->non_fatal = TRUE; } } @@ -2746,9 +2735,7 @@ flatpak_transaction_add_ref (FlatpakTransaction *self, } op = flatpak_transaction_add_op (self, remote, ref, subpaths, previous_ids, - commit, bundle, kind, pin_on_deploy, error); - if (op == NULL) - return FALSE; + commit, bundle, kind, pin_on_deploy); if (external_metadata) op->external_metadata = g_bytes_new (external_metadata, strlen (external_metadata)); @@ -4983,9 +4970,8 @@ add_uninstall_unused_ops (FlatpakTransaction *self, uninstall_op = flatpak_transaction_add_op (self, origin, unused_ref, NULL, NULL, NULL, NULL, FLATPAK_TRANSACTION_OPERATION_UNINSTALL, - FALSE, NULL); - if (uninstall_op) - run_operation_last (uninstall_op); + FALSE); + run_operation_last (uninstall_op); } }