# HG changeset patch
# User Tim Ringenbach <marv@pidgin.im>
# Date 1085613884 0
# Node ID 13e809489bebc7516811bf8664f9bcd71819a93b
# Parent  aad506cde4f32d11e6af0a9d9b9bca180d692fab
[gaim-migrate @ 9853]
nosnilmot found that there continued to be bugs in our linkification on
yahoo (and probably msn). He wrote:
"Tim,

Something's still not quite right, this is on Yahoo - insert link ->
http://www.google.com/ -> send (the signals-test plugin is quite useful
here!) - nested links now instead of duplicate links:

(07:12:48) signals test: writing-im-msg (nosnilmot26, nosnilmot26, <a href="http://www.google.com/"><A HREF="http://www.google.com/">http://www.google.com/</A></a>)

Anyway, that problem is NOT in imhtml, so I managed to do something
about it - gaim_markup_linkify was never actually detecting the "<A" tag
because it was an else-if on checking for "<".

The patch attached fixes that so gaim_markup_linkify will not try to
linkify stuff that's already inside a link

Regards,


Stu."

committer: Tailor Script <tailor@pidgin.im>

diff -r aad506cde4f3 -r 13e809489beb src/util.c
--- a/src/util.c	Wed May 26 20:23:07 2004 +0000
+++ b/src/util.c	Wed May 26 23:24:44 2004 +0000
@@ -1341,15 +1341,17 @@
 			}
 		} else if(*c == '<') {
 			inside_html = TRUE;
-		} else if (!g_ascii_strncasecmp(c, "<A", 2)) {
-			while (1) {
-				if (!g_ascii_strncasecmp(c, "/A>", 3)) {
-					break;
+			if (!g_ascii_strncasecmp(c, "<A", 2)) {
+				while (1) {
+					if (!g_ascii_strncasecmp(c, "/A>", 3)) {
+						inside_html = FALSE;
+						break;
+					}
+					ret = g_string_append_c(ret, *c);
+					c++;
+					if (!(*c))
+						break;
 				}
-				ret = g_string_append_c(ret, *c);
-				c++;
-				if (!(*c))
-					break;
 			}
 		} else if ((*c=='h') && (!g_ascii_strncasecmp(c, "http://", 7) ||
 					(!g_ascii_strncasecmp(c, "https://", 8)))) {