pidgin: 80e8cabf: disapproval of revision '0a9f6836dbe4fac...

[markdoliner at pidgin.im]

-----------------------------------------------------------------
Revision: 80e8cabf693147bcebc8857d383b6aab279d75ae
Ancestor: 0a9f6836dbe4facb38768025e830328a50c5ddd8
Author: markdoliner at pidgin.im
Date: 2009-07-19T19:33:43
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/80e8cabf693147bcebc8857d383b6aab279d75ae

Modified files:
        libpurple/util.c

ChangeLog: 

disapproval of revision '0a9f6836dbe4facb38768025e830328a50c5ddd8'
-------------- next part --------------
============================================================
--- libpurple/util.c	e04f24386263400d79b1229e14912f07182b0d27
+++ libpurple/util.c	6be57b7a1d754e29220193f907ba6449100ed933
@@ -1409,8 +1409,7 @@ struct purple_parse_tag {
 	gboolean ignore;
 };
 
-#define ALLOW_TAG_ALT(x, y) do { \
-					if(!g_ascii_strncasecmp(c, "<" x " ", strlen("<" x " "))) { \
+#define ALLOW_TAG_ALT(x, y) if(!g_ascii_strncasecmp(c, "<" x " ", strlen("<" x " "))) { \
 						const char *o = c + strlen("<" x); \
 						const char *p = NULL, *q = NULL, *r = NULL; \
 						GString *innards = g_string_new(""); \
@@ -1481,7 +1480,6 @@ struct purple_parse_tag {
 						c = strchr(c, '>') + 1; \
 						continue; \
 					} \
-				} while (0);
 #define ALLOW_TAG(x) ALLOW_TAG_ALT(x, x)
 void
 purple_markup_html_to_xhtml(const char *html, char **xhtml_out,
@@ -1574,8 +1572,9 @@ purple_markup_html_to_xhtml(const char *
 				ALLOW_TAG("h5");
 				ALLOW_TAG("h6");
 				/* we only allow html to start the message */
-				if(c == html)
+				if(c == html) {
 					ALLOW_TAG("html");
+				}
 				ALLOW_TAG_ALT("i", "em");
 				ALLOW_TAG_ALT("italic", "em");
 				ALLOW_TAG("li");