all: clean-up usage of GError
authorThomas Haller <thaller@redhat.com>
Sun, 28 Feb 2016 15:25:36 +0000 (16:25 +0100)
committerThomas Haller <thaller@redhat.com>
Thu, 3 Mar 2016 17:54:20 +0000 (18:54 +0100)
commit01b9b4104c365002fae5e45b564956062fe146d4
treede72d05b95706d94a8750d9870768b61edf8b98a
parent7871d850f50975b46f7909bc6bd3cce434c7c60e
all: clean-up usage of GError

Functions that take a GError** MUST fill it in on error. There is no
need to check whether error is NULL if the function it was passed to
had a failing return value.

Likewise, a proper GError must have a non-NULL message, so there's no
need to double-check that either.

Based-on-patch-by: Dan Winship <danw@gnome.org>
32 files changed:
clients/cli/connections.c
libnm-glib/nm-client.c
libnm-glib/nm-device.c
libnm-glib/nm-remote-connection.c
libnm-glib/nm-secret-agent.c
libnm-util/nm-utils.c
libnm/nm-remote-connection.c
shared/nm-test-utils.h
src/devices/bluetooth/nm-bluez-device.c
src/devices/bluetooth/nm-bluez5-manager.c
src/devices/bluetooth/nm-device-bt.c
src/devices/nm-device-ethernet.c
src/devices/nm-device-factory.c
src/devices/nm-device.c
src/devices/wifi/nm-device-wifi.c
src/devices/wwan/nm-device-modem.c
src/devices/wwan/nm-modem-broadband.c
src/devices/wwan/nm-modem-manager.c
src/devices/wwan/nm-modem.c
src/dhcp-manager/nm-dhcp-dhclient.c
src/dns-manager/nm-dns-dnsmasq.c
src/dns-manager/nm-dns-plugin.c
src/main.c
src/nm-activation-request.c
src/nm-manager.c
src/nm-policy.c
src/ppp-manager/nm-pppd-plugin.c
src/settings/nm-agent-manager.c
src/settings/nm-settings-connection.c
src/settings/nm-settings.c
src/settings/plugins/ifupdown/nm-ifupdown-connection.c
src/settings/plugins/keyfile/plugin.c