# HG changeset patch
# User qarkai <qarkai@gmail.com>
# Date 1490351100 -10800
# Node ID 8c0b26d1d508c5d98eae24f0148bc66571b5045f
# Parent  653645acccc8ff7f6208161e06968a19c0f48aab
Replace empty string comparison with null character comparison
More efficient way to identify an empty string

diff -r 653645acccc8 -r 8c0b26d1d508 libpurple/protocols/gg/lib/common.c
--- a/libpurple/protocols/gg/lib/common.c	Fri Mar 24 13:01:08 2017 +0300
+++ b/libpurple/protocols/gg/lib/common.c	Fri Mar 24 13:25:00 2017 +0300
@@ -161,7 +161,7 @@
 {
 	char *foo, *res;
 
-	if (!ptr || !*ptr || purple_strequal(*ptr, ""))
+	if (!ptr || !*ptr)
 		return NULL;
 
 	res = *ptr;
diff -r 653645acccc8 -r 8c0b26d1d508 libpurple/protocols/jabber/disco.c
--- a/libpurple/protocols/jabber/disco.c	Fri Mar 24 13:01:08 2017 +0300
+++ b/libpurple/protocols/jabber/disco.c	Fri Mar 24 13:25:00 2017 +0300
@@ -519,6 +519,7 @@
 	for (child = xmlnode_get_child(query, "identity"); child;
 	     child = xmlnode_get_next_twin(child)) {
 		const char *category, *type, *name;
+		const char *stun_ip;
 		category = xmlnode_get_attrib(child, "category");
 		type = xmlnode_get_attrib(child, "type");
 		if(purple_strequal(category, "pubsub") && purple_strequal(type, "pep")) {
@@ -538,17 +539,16 @@
 
 		g_free(js->server_name);
 		js->server_name = g_strdup(name);
+		stun_ip = purple_network_get_stun_ip();
 		if (purple_strequal(name, "Google Talk")) {
 			purple_debug_info("jabber", "Google Talk!\n");
 			js->googletalk = TRUE;
 
 			/* autodiscover stun and relays */
-			if (purple_network_get_stun_ip() == NULL ||
-		    	purple_strequal(purple_network_get_stun_ip(), "")) {
+			if (stun_ip == NULL || stun_ip[0] == '\0') {
 				jabber_google_send_jingle_info(js);
 			}
-		} else if (purple_network_get_stun_ip() == NULL ||
-		    purple_strequal(purple_network_get_stun_ip(), "")) {
+		} else if (stun_ip == NULL || stun_ip[0] == '\0') {
 			js->srv_query_data =
 				purple_srv_resolve_account(
 					purple_connection_get_account(js->gc), "stun", "udp",
diff -r 653645acccc8 -r 8c0b26d1d508 pidgin/gtkimhtml.c
--- a/pidgin/gtkimhtml.c	Fri Mar 24 13:01:08 2017 +0300
+++ b/pidgin/gtkimhtml.c	Fri Mar 24 13:25:00 2017 +0300
@@ -4793,7 +4793,7 @@
 	g_free(*edit_field);
 	*edit_field = NULL;
 
-	if (value && !purple_strequal(value, ""))
+	if (value && value[0] != '\0')
 	{
 		*edit_field = g_strdup(value);
 
diff -r 653645acccc8 -r 8c0b26d1d508 pidgin/gtkprefs.c
--- a/pidgin/gtkprefs.c	Fri Mar 24 13:01:08 2017 +0300
+++ b/pidgin/gtkprefs.c	Fri Mar 24 13:25:00 2017 +0300
@@ -1016,7 +1016,7 @@
 
 		gtk_tree_model_get(GTK_TREE_MODEL(prefs_blist_themes), &iter, 2, &name, -1);
 
-		if(!purple_strequal(name, ""))
+		if(!name || name[0] != '\0')
 			theme = PIDGIN_BLIST_THEME(purple_theme_manager_find_theme(name, "blist"));
 
 		g_free(name);
@@ -1037,7 +1037,7 @@
 
 		gtk_tree_model_get(GTK_TREE_MODEL(prefs_status_icon_themes), &iter, 2, &name, -1);
 
-		if(!purple_strequal(name, ""))
+		if(!name || name[0] != '\0')
 			theme = PIDGIN_STATUS_ICON_THEME(purple_theme_manager_find_theme(name, "status-icon"));
 
 		g_free(name);
diff -r 653645acccc8 -r 8c0b26d1d508 pidgin/plugins/pidginrc.c
--- a/pidgin/plugins/pidginrc.c	Fri Mar 24 13:01:08 2017 +0300
+++ b/pidgin/plugins/pidginrc.c	Fri Mar 24 13:25:00 2017 +0300
@@ -113,7 +113,7 @@
 	if (purple_prefs_get_bool("/plugins/gtk/purplerc/set/gtk-font-name")) {
 		const gchar *pref = purple_prefs_get_string("/plugins/gtk/purplerc/gtk-font-name");
 
-		if (pref != NULL && !purple_strequal(pref, "")) {
+		if (pref != NULL && pref[0] != '\0') {
 			g_string_append_printf(style_string,
 			                       "gtk-font-name = \"%s\"\n",
 			                       pref);
@@ -123,7 +123,7 @@
 	if (purple_prefs_get_bool("/plugins/gtk/purplerc/set/gtk-key-theme-name")) {
 		const gchar *pref = purple_prefs_get_string("/plugins/gtk/purplerc/gtk-key-theme-name");
 
-		if (pref != NULL && !purple_strequal(pref, "")) {
+		if (pref != NULL && pref[0] != '\0') {
 			g_string_append_printf(style_string,
 			                       "gtk-key-theme-name = \"%s\"\n",
 			                       pref);
@@ -141,7 +141,7 @@
 			const gchar *pref;
 
 			pref = purple_prefs_get_string(color_prefs[i]);
-			if (pref != NULL && !purple_strequal(pref, "")) {
+			if (pref != NULL && pref[0] != '\0') {
 				prefbase = g_path_get_basename(color_prefs[i]);
 				g_string_append_printf(style_string,
 				                       "\n\t%s = \"%s\"",
@@ -180,7 +180,7 @@
 			const gchar *pref;
 
 			pref = purple_prefs_get_string(font_prefs[i]);
-			if (pref != NULL && !purple_strequal(pref, "")) {
+			if (pref != NULL && pref[0] != '\0') {
 				prefbase = g_path_get_basename(font_prefs[i]);
 				g_string_append_printf(style_string,
 				                       "style \"%s_style\"\n{\n"
@@ -276,7 +276,7 @@
 
 	pref = purple_prefs_get_string(color_prefs[subscript]);
 
-	if (pref != NULL && !purple_strequal(pref, "")) {
+	if (pref != NULL && pref[0] != '\0') {
 		if (gdk_color_parse(pref, &color)) {
 #if GTK_CHECK_VERSION(2,14,0)
 			gtk_color_selection_set_current_color(GTK_COLOR_SELECTION(
@@ -337,7 +337,7 @@
 
 	pref = purple_prefs_get_string(prefpath);
 
-	if (pref != NULL && !purple_strequal(pref, "")) {
+	if (pref != NULL && pref[0] != '\0') {
 		gtk_font_selection_dialog_set_font_name(GTK_FONT_SELECTION_DIALOG(font_dialog), pref);
 	}