--- libedataserverui/e-passwords.c.orig	2007-10-18 15:33:33.000000000 -0400
+++ libedataserverui/e-passwords.c	2007-10-18 15:51:21.000000000 -0400
@@ -348,7 +348,7 @@ ep_clear_passwords_keyring(EPassMsg *msg
 }
 #endif
 static void
-ep_clear_passwords_file(EPassMsg *msg)
+ep_clear_passwords_keyfile(EPassMsg *msg)
 {
 	gchar *group;
 	GError *error = NULL;
@@ -369,6 +369,19 @@ ep_clear_passwords_file(EPassMsg *msg)
 		e_msgport_reply(&msg->msg);
 }
 
+static void
+ep_clear_passwords (EPassMsg *msg)
+{
+#ifdef WITH_GNOME_KEYRING
+	if (gnome_keyring_is_available ())
+		ep_clear_passwords_keyring (msg);
+	else
+		ep_clear_passwords_keyfile (msg);
+#else
+	ep_clear_passwords_keyfile (msg);
+#endif
+}
+
 static gboolean
 free_entry (gpointer key, gpointer value, gpointer user_data)
 {
@@ -426,7 +439,7 @@ ep_forget_passwords_keyring(EPassMsg *ms
 #endif
 
 static void
-ep_forget_passwords_file(EPassMsg *msg)
+ep_forget_passwords_keyfile(EPassMsg *msg)
 {
 	gchar **groups;
 	gsize length, ii;
@@ -446,6 +459,19 @@ ep_forget_passwords_file(EPassMsg *msg)
 		e_msgport_reply(&msg->msg);
 }
 
+static void
+ep_forget_passwords (EPassMsg *msg)
+{
+#ifdef WITH_GNOME_KEYRING
+	if (gnome_keyring_is_available ())
+		ep_forget_passwords_keyring (msg);
+	else
+		ep_forget_passwords_keyfile (msg);
+#else
+	ep_forget_passwords_keyfile (msg);
+#endif
+}
+
 #ifdef WITH_GNOME_KEYRING
 static void
 ep_remember_password_keyring(EPassMsg *msg)
@@ -500,7 +526,7 @@ ep_remember_password_keyring(EPassMsg *m
 #endif
 
 static void
-ep_remember_password_file(EPassMsg *msg)
+ep_remember_password_keyfile(EPassMsg *msg)
 {
 	gchar *group, *key, *password;
 
@@ -526,6 +552,19 @@ ep_remember_password_file(EPassMsg *msg)
 		e_msgport_reply(&msg->msg);
 }
 
+static void
+ep_remember_password (EPassMsg *msg)
+{
+#ifdef WITH_GNOME_KEYRING
+	if (gnome_keyring_is_available ())
+		ep_remember_password_keyring (msg);
+	else
+		ep_remember_password_keyfile (msg);
+#else
+	ep_remember_password_keyfile (msg);
+#endif
+}
+
 #ifdef WITH_GNOME_KEYRING
 static void
 ep_forget_password_keyring (EPassMsg *msg)
@@ -614,7 +653,7 @@ exit:
 #endif
 
 static void
-ep_forget_password_file (EPassMsg *msg)
+ep_forget_password_keyfile (EPassMsg *msg)
 {
 	gchar *group, *key;
 	GError *error = NULL;
@@ -639,6 +678,18 @@ ep_forget_password_file (EPassMsg *msg)
 		e_msgport_reply(&msg->msg);
 }
 
+static void
+ep_forget_password (EPassMsg *msg)
+{
+#ifdef WITH_GNOME_KEYRING
+	if (gnome_keyring_is_available ())
+		ep_forget_password_keyring (msg);
+	else
+		ep_forget_password_keyfile (msg);
+#else
+	ep_forget_password_keyfile (msg);
+#endif
+}
 
 #ifdef WITH_GNOME_KEYRING
 static void
@@ -722,7 +773,7 @@ ep_get_password_keyring (EPassMsg *msg)
 #endif
 
 static void
-ep_get_password_file (EPassMsg *msg)
+ep_get_password_keyfile (EPassMsg *msg)
 {
 	gchar *group, *key, *password;
 	GError *error = NULL;
@@ -753,6 +804,19 @@ ep_get_password_file (EPassMsg *msg)
 }
 
 static void
+ep_get_password (EPassMsg *msg)
+{
+#ifdef WITH_GNOME_KEYRING
+	if (gnome_keyring_is_available ())
+		ep_get_password_keyring (msg);
+	else
+		ep_get_password_keyfile (msg);
+#else
+	ep_get_password_keyfile (msg);
+#endif
+}
+
+static void
 ep_add_password (EPassMsg *msg)
 {
 	g_hash_table_insert (passwords, g_strdup (msg->key), g_strdup (msg->oldpass));
@@ -785,17 +849,8 @@ pass_response(GtkDialog *dialog, int res
 				msg->oldpass = msg->password;
 				ep_add_password(msg);
 			}
-#ifdef WITH_GNOME_KEYRING
-			if (*msg->remember && type == E_PASSWORDS_REMEMBER_FOREVER) {
-				if (gnome_keyring_is_available())
-					ep_remember_password_keyring(msg);
-				else
-					ep_remember_password_file(msg);
-			}
-#else
 			if (*msg->remember && type == E_PASSWORDS_REMEMBER_FOREVER)
-				ep_remember_password_file(msg);				    
-#endif				    
+				ep_remember_password(msg);				    
 
 			msg->noreply = noreply;
 		}
@@ -812,16 +867,8 @@ pass_response(GtkDialog *dialog, int res
 	mw = (EPassMsg *)request_list.head;
 	mn = (EPassMsg *)mw->msg.ln.next;
 	while (mn) {
-#ifdef WITH_GNOME_KEYRING		
-		if ((mw->dispatch == (gnome_keyring_is_available() ? ep_forget_password_keyring : ep_forget_password_file)
-#else
-		if ((mw->dispatch == ep_forget_password_file		     
-#endif
-#ifdef WITH_GNOME_KEYRING				     
-		     || mw->dispatch == (gnome_keyring_is_available() ? ep_get_password_keyring : ep_get_password_file)
-#else
-		     || mw->dispatch == ep_get_password_file		     
-#endif
+		if ((mw->dispatch == ep_forget_password
+		     || mw->dispatch == ep_get_password
 		     || mw->dispatch == ep_ask_password)		    
 		    && (strcmp(mw->component, msg->component) == 0
 			&& strcmp(mw->key, msg->key) == 0)) {
@@ -880,14 +927,7 @@ ep_ask_password(EPassMsg *msg)
 	gtk_widget_grab_focus (msg->entry);
 	
 	if ((msg->flags & E_PASSWORDS_REPROMPT)) {
-#ifdef WITH_GNOME_KEYRING
-		if (gnome_keyring_is_available())
-			ep_get_password_keyring(msg);
-		else
-			ep_get_password_file(msg);			
-#else
-		ep_get_password_file(msg);
-#endif
+		ep_get_password(msg);
 		if (msg->password) {
 			gtk_entry_set_text ((GtkEntry *) msg->entry, msg->password);
 			g_free (msg->password);
@@ -1031,11 +1071,7 @@ e_passwords_set_online(int state)
 void
 e_passwords_forget_passwords (void)
 {
-#ifdef WITH_GNOME_KEYRING	
-	EPassMsg *msg = ep_msg_new(gnome_keyring_is_available() ? ep_forget_passwords_keyring : ep_forget_passwords_file);
-#else
-	EPassMsg *msg = ep_msg_new(ep_forget_passwords_file);
-#endif
+	EPassMsg *msg = ep_msg_new(ep_forget_passwords);
 	
 	ep_msg_send(msg);
 	ep_msg_free(msg);
@@ -1049,11 +1085,7 @@ e_passwords_forget_passwords (void)
 void
 e_passwords_clear_passwords (const char *component_name)
 {
-#ifdef WITH_GNOME_KEYRING	
-	EPassMsg *msg = ep_msg_new(gnome_keyring_is_available() ? ep_clear_passwords_keyring : ep_clear_passwords_file);
-#else
-	EPassMsg *msg = ep_msg_new(ep_clear_passwords_file);		
-#endif
+	EPassMsg *msg = ep_msg_new(ep_clear_passwords);		
 
 	msg->component = component_name;
 	ep_msg_send(msg);
@@ -1074,11 +1106,7 @@ e_passwords_remember_password (const cha
 	g_return_if_fail(component_name != NULL);
 	g_return_if_fail(key != NULL);
 
-#ifdef WITH_GNOME_KEYRING
-	msg = ep_msg_new(gnome_keyring_is_available() ? ep_remember_password_keyring : ep_remember_password_file);
-#else
-	msg = ep_msg_new(ep_remember_password_file);
-#endif	
+	msg = ep_msg_new(ep_remember_password);
 	msg->component = component_name;
 	msg->key = key;
 
@@ -1100,11 +1128,7 @@ e_passwords_forget_password (const char 
 	g_return_if_fail(component_name != NULL);
 	g_return_if_fail(key != NULL);
 
-#ifdef WITH_GNOME_KEYRING	
-	msg = ep_msg_new(gnome_keyring_is_available() ? ep_forget_password_keyring : ep_forget_password_file);
-#else	
-	msg = ep_msg_new(ep_forget_password_file);
-#endif	
+	msg = ep_msg_new(ep_forget_password);
 	msg->component = component_name;
 	msg->key = key;
 
@@ -1128,12 +1152,7 @@ e_passwords_get_password (const char *co
 	g_return_val_if_fail(component_name != NULL, NULL);
 	g_return_val_if_fail(key != NULL, NULL);
 
-#ifdef WITH_GNOME_KEYRING	
-	msg = ep_msg_new(gnome_keyring_is_available() ? ep_get_password_keyring : ep_get_password_file);
-#else
-	msg = ep_msg_new(ep_get_password_file);
-#endif	
-
+	msg = ep_msg_new(ep_get_password);
 	msg->component = component_name;
 	msg->key = key;
 
