From bd4494363230705c3efaa13f3c94a73fadab88e4 Mon Sep 17 00:00:00 2001 From: Johan Hedberg Date: Fri, 24 Oct 2008 13:44:16 +0300 Subject: Use g_clear_error(&err) instead of g_error_free(err); err = NULL --- audio/a2dp.c | 15 +++++---------- audio/headset.c | 3 +-- audio/manager.c | 12 ++++-------- network/manager.c | 26 +++++++++----------------- src/main.c | 18 ++++++------------ 5 files changed, 25 insertions(+), 49 deletions(-) diff --git a/audio/a2dp.c b/audio/a2dp.c index 49312a71..8c38a90b 100644 --- a/audio/a2dp.c +++ b/audio/a2dp.c @@ -1071,8 +1071,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config) if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { if (strstr(str, "Sink")) source = FALSE; @@ -1084,8 +1083,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config) str = g_key_file_get_string(config, "A2DP", "SBCSources", &err); if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { sbc_srcs = atoi(str); g_free(str); @@ -1094,8 +1092,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config) str = g_key_file_get_string(config, "A2DP", "MPEG12Sources", &err); if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { mpeg12_srcs = atoi(str); g_free(str); @@ -1104,8 +1101,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config) str = g_key_file_get_string(config, "A2DP", "SBCSinks", &err); if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { sbc_sinks = atoi(str); g_free(str); @@ -1114,8 +1110,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config) str = g_key_file_get_string(config, "A2DP", "MPEG12Sinks", &err); if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { mpeg12_sinks = atoi(str); g_free(str); diff --git a/audio/headset.c b/audio/headset.c index 0a7f18ed..9ba2138a 100644 --- a/audio/headset.c +++ b/audio/headset.c @@ -2040,8 +2040,7 @@ uint32_t headset_config_init(GKeyFile *config) &err); if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { if (strcmp(str, "PCM") == 0) sco_hci = FALSE; diff --git a/audio/manager.c b/audio/manager.c index 4441b90f..816e6821 100644 --- a/audio/manager.c +++ b/audio/manager.c @@ -510,8 +510,7 @@ static int headset_server_init(struct audio_adapter *adapter) &err); if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else master = tmp; } @@ -595,8 +594,7 @@ static int gateway_server_init(struct audio_adapter *adapter) &err); if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else master = tmp; } @@ -918,8 +916,7 @@ int audio_manager_init(DBusConnection *conn, GKeyFile *conf) &err); if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else enabled.hfp = b; @@ -928,8 +925,7 @@ int audio_manager_init(DBusConnection *conn, GKeyFile *conf) &err); if (err) { debug("audio.conf: %s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else max_connected_headsets = i; diff --git a/network/manager.c b/network/manager.c index 564af2ad..1b68ea2a 100644 --- a/network/manager.c +++ b/network/manager.c @@ -95,7 +95,7 @@ static void read_config(const char *file) if (!g_key_file_load_from_file(keyfile, file, 0, &err)) { error("Parsing %s failed: %s", file, err->message); - g_error_free(err); + g_clear_error(&err); goto done; } @@ -103,8 +103,7 @@ static void read_config(const char *file) "Disable", NULL, &err); if (err) { debug("%s: %s", file, err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { int i; for (i = 0; disabled[i] != NULL; i++) { @@ -120,56 +119,49 @@ static void read_config(const char *file) "DisableSecurity", &err); if (err) { debug("%s: %s", file, err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } conf.panu_script = g_key_file_get_string(keyfile, "PANU Role", "Script", &err); if (err) { debug("%s: %s", file, err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } conf.gn_script = g_key_file_get_string(keyfile, "GN Role", "Script", &err); if (err) { debug("%s: %s", file, err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } conf.nap_script = g_key_file_get_string(keyfile, "NAP Role", "Script", &err); if (err) { debug("%s: %s", file, err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } conf.iface_prefix = g_key_file_get_string(keyfile, "PANU Role", "Interface", &err); if (err) { debug("%s: %s", file, err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } conf.gn_iface = g_key_file_get_string(keyfile, "GN Role", "Interface", &err); if (err) { debug("%s: %s", file, err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } conf.nap_iface = g_key_file_get_string(keyfile, "NAP Role", "Interface", &err); if (err) { debug("%s: %s", file, err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } done: diff --git a/src/main.c b/src/main.c index 9bfeb512..407a1122 100644 --- a/src/main.c +++ b/src/main.c @@ -106,8 +106,7 @@ static void parse_config(GKeyFile *config) "OffMode", &err); if (err) { debug("%s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { debug("offmode=%s", str); if (g_str_equal(str, "DevDown")) @@ -120,8 +119,7 @@ static void parse_config(GKeyFile *config) &err); if (err) { debug("%s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { debug("discovto=%d", val); main_opts.discovto = val; @@ -133,8 +131,7 @@ static void parse_config(GKeyFile *config) &err); if (err) { debug("%s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { debug("pageto=%d", val); main_opts.pageto = val; @@ -145,8 +142,7 @@ static void parse_config(GKeyFile *config) "Name", &err); if (err) { debug("%s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { debug("name=%s", str); g_free(main_opts.name); @@ -159,8 +155,7 @@ static void parse_config(GKeyFile *config) "Class", &err); if (err) { debug("%s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { debug("class=%s", str); main_opts.class = strtol(str, NULL, 16); @@ -173,8 +168,7 @@ static void parse_config(GKeyFile *config) &err); if (err) { debug("%s", err->message); - g_error_free(err); - err = NULL; + g_clear_error(&err); } else { debug("inqmode=%d", val); main_opts.inqmode = val; -- cgit