From 47ac3261fff7588ea4cda15cdfdf24366a4f5bf8 Mon Sep 17 00:00:00 2001 From: Jeffrey Stedfast Date: Tue, 21 May 2002 21:45:58 +0000 Subject: Use a new macro, STR_EQUAL, to tell if the 2 values are equal or not 2002-05-21 Jeffrey Stedfast * mail-composer-prefs.c (spell_save_values): Use a new macro, STR_EQUAL, to tell if the 2 values are equal or not rather than using strcmp since one or both strings could be NULL. svn path=/trunk/; revision=16971 --- mail/mail-composer-prefs.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'mail/mail-composer-prefs.c') diff --git a/mail/mail-composer-prefs.c b/mail/mail-composer-prefs.c index c196d4635e..15f87826c6 100644 --- a/mail/mail-composer-prefs.c +++ b/mail/mail-composer-prefs.c @@ -623,6 +623,8 @@ spell_load_values (MailComposerPrefs *prefs) #define SET(t,x,prop) \ gconf_client_set_ ## t (prefs->gconf, GNOME_SPELL_GCONF_DIR x, prop, NULL); +#define STR_EQUAL(str1, str2) ((str1 == NULL && str2 == NULL) || (str1 && str2 && !strcmp (str1, str2))) + static void spell_save_values (MailComposerPrefs *prefs, gboolean force) { @@ -631,7 +633,7 @@ spell_save_values (MailComposerPrefs *prefs, gboolean force) SET (int, "/spell_error_color_green", prefs->spell_error_color.green); SET (int, "/spell_error_color_blue", prefs->spell_error_color.blue); } - if (force || strcmp (prefs->language_str, prefs->language_str_orig)) { + if (force || !STR_EQUAL (prefs->language_str, prefs->language_str_orig)) { SET (string, "/language", prefs->language_str); } -- cgit v1.2.3