From ff528b19387f103c1b22cb1d480a651c2f2b9074 Mon Sep 17 00:00:00 2001 From: Chris Toshok Date: Fri, 30 Apr 2004 18:23:16 +0000 Subject: don't use "email" twice for a variable name.. even though they don't 2004-04-30 Chris Toshok * util/e-destination.c (e_destination_set_contact): don't use "email" twice for a variable name.. even though they don't really conflict, it's ugly and confusing. svn path=/trunk/; revision=25720 --- addressbook/ChangeLog | 6 ++++++ addressbook/util/e-destination.c | 7 ++++--- 2 files changed, 10 insertions(+), 3 deletions(-) (limited to 'addressbook') diff --git a/addressbook/ChangeLog b/addressbook/ChangeLog index df5ea466aa..152faed889 100644 --- a/addressbook/ChangeLog +++ b/addressbook/ChangeLog @@ -1,3 +1,9 @@ +2004-04-30 Chris Toshok + + * util/e-destination.c (e_destination_set_contact): don't use + "email" twice for a variable name.. even though they don't really + conflict, it's ugly and confusing. + 2004-04-30 Enver ALTIN * gui/component/GNOME_Evolution_Addressbook.server.in.in: Fixed a diff --git a/addressbook/util/e-destination.c b/addressbook/util/e-destination.c index 1ab46425c4..d81cbd7fbb 100644 --- a/addressbook/util/e-destination.c +++ b/addressbook/util/e-destination.c @@ -332,7 +332,7 @@ e_destination_set_contact (EDestination *dest, EContact *contact, gint email_num GList *p; EDestination *list_dest = e_destination_new (); char *contact_uid = NULL; - char *email = NULL; + char *email_addr = NULL; char *name = NULL; int email_num = -1; gboolean html_pref = FALSE; @@ -358,7 +358,7 @@ e_destination_set_contact (EDestination *dest, EContact *contact, gint email_num else if (!g_ascii_strcasecmp (param_name, EVC_X_DEST_EMAIL)) { GList *v = e_vcard_attribute_param_get_values (param); - email = v ? v->data : NULL; + email_addr = v ? v->data : NULL; } else if (!g_ascii_strcasecmp (param_name, EVC_X_DEST_HTML_MAIL)) { @@ -369,7 +369,7 @@ e_destination_set_contact (EDestination *dest, EContact *contact, gint email_num if (contact_uid) e_destination_set_contact_uid (list_dest, contact_uid, email_num); if (name) e_destination_set_name (list_dest, name); - if (email) e_destination_set_email (list_dest, email); + if (email_addr) e_destination_set_email (list_dest, email_addr); e_destination_set_html_mail_pref (list_dest, html_pref); dest->priv->list_dests = g_list_append (dest->priv->list_dests, list_dest); @@ -381,6 +381,7 @@ e_destination_set_contact (EDestination *dest, EContact *contact, gint email_num } else { /* handle the normal contact case */ + /* is there anything to do here? */ } } } -- cgit