Re: [libvirt] [PATCH 1/4] remote: dispatch: Remove return value from make_nonnull_* helpers

After convertion to g_strdup, the helpers now always return success.

Remove the return value to simplify the callers. Note that many occurences of these is in the code generated by

gendispatch.pl. Since gendispatch aggregates many cases together a

incremental conversion would require more invasive changes to gendispatch for the time of coversion which doesn't make sense.

Also in many cases the helper was the last place where the 'error:' label was used and thus also those conversions must be included in this patch. Signed-off-by: Peter Krempa <pkrempa(a)redhat.com&gt; --- src/admin/admin_server_dispatch.c | 6 +- src/remote/remote_daemon_dispatch.c | 321 ++++++---------------------- src/rpc/gendispatch.pl | 19 +- 3 files changed, 75 insertions(+), 271 deletions(-)

+++ b/src/remote/remote_daemon_dispatch.c @@ -95,18 +95,18 @@ static virNWFilterBindingPtr get_nonnull_nwfilter_binding(virConnectPtr conn, re static virDomainCheckpointPtr get_nonnull_domain_checkpoint(virDomainPtr dom, remote_nonnull_domain_checkpoint checkpoint); static virDomainSnapshotPtr get_nonnull_domain_snapshot(virDomainPtr dom, remote_nonnull_domain_snapshot snapshot); static virNodeDevicePtr get_nonnull_node_device(virConnectPtr conn, remote_nonnull_node_device dev); -static int make_nonnull_domain(remote_nonnull_domain *dom_dst, virDomainPtr dom_src) G_GNUC_WARN_UNUSED_RESULT; -static int make_nonnull_network(remote_nonnull_network *net_dst, virNetworkPtr net_src) G_GNUC_WARN_UNUSED_RESULT;

+static void make_nonnull_domain(remote_nonnull_domain *dom_dst, virDomainPtr dom_src); +static void make_nonnull_network(remote_nonnull_network *net_dst, virNetworkPtr net_src);