diff options
| author | Marco Barisione <marco.barisione@collabora.co.uk> | 2013-05-13 10:25:27 +0100 |
|---|---|---|
| committer | Marco Barisione <marco.barisione@collabora.co.uk> | 2013-08-20 11:03:07 +0100 |
| commit | df2ff24c6a132e4b6cd40c9f82a24cec8c04a4aa (patch) | |
| tree | 0c6d1e920b697f865b1db4146498658ee4f5962e /tp-account-widgets/tpaw-account-settings.c | |
| parent | 72b6c3a680852d93aba90bfd87db0b7477b3f4f5 (diff) | |
| download | telepathy-account-widgets-df2ff24c6a132e4b6cd40c9f82a24cec8c04a4aa.tar.gz | |
tp-account-widgets: use g_strcmp0 instead of strcmp
There was no bug in this code, but I noticed the usage of strcmp because
of compilation errors caused by some changes to the includes. g_strcmp0
is safer as it handles NULLs.
https://bugzilla.gnome.org/show_bug.cgi?id=699492
Diffstat (limited to 'tp-account-widgets/tpaw-account-settings.c')
| -rw-r--r-- | tp-account-widgets/tpaw-account-settings.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tp-account-widgets/tpaw-account-settings.c b/tp-account-widgets/tpaw-account-settings.c index fb914397..88204171 100644 --- a/tp-account-widgets/tpaw-account-settings.c +++ b/tp-account-widgets/tpaw-account-settings.c @@ -1538,7 +1538,7 @@ tpaw_account_settings_parameter_is_valid ( g_return_val_if_fail (TPAW_IS_ACCOUNT_SETTINGS (settings), FALSE); if (g_list_find_custom (settings->priv->required_params, param, - (GCompareFunc) strcmp)) + (GCompareFunc) g_strcmp0)) { /* first, look if it's set in our own parameters */ if (g_hash_table_lookup (settings->priv->parameters, param) != NULL) |