# HG changeset patch # User Richard Laager # Date 1150696631 0 # Node ID 378bba5011e7817cdd64a512eb2fcf2b0995fbbc # Parent 571aebbe5ec6114898af673ded4b612868e6a2de [gaim-migrate @ 16283] SF Patch #1500688 from Brian Chu (roast) He already committed this to his soc-2006-file-loggers branch. Here's the patch description: In util.c, gaim_markup_html_to_xhtml() doesn't trim spaces of parsed background colors, parsed font names, or the end result for styles (which inserted spaces for the purpose of many span.style attributes). For example, things like this get written to the log: Things like the following this _can_ written to the log given a tag like this: Same for: committer: Tailor Script diff -r 571aebbe5ec6 -r 378bba5011e7 COPYRIGHT --- a/COPYRIGHT Mon Jun 19 05:39:30 2006 +0000 +++ b/COPYRIGHT Mon Jun 19 05:57:11 2006 +0000 @@ -51,6 +51,7 @@ Christophe Chapuis Ka-Hing Cheung Sadrul Habib Chowdhury +Brian Chu Arturo Cisneros, Jr. Vincas Ciziunas Jonathan Clark diff -r 571aebbe5ec6 -r 378bba5011e7 src/util.c --- a/src/util.c Mon Jun 19 05:39:30 2006 +0000 +++ b/src/util.c Mon Jun 19 05:57:11 2006 +0000 @@ -1342,7 +1342,7 @@ face = g_string_append_c(face, *q); q++; } - g_string_append_printf(style, "font-family: %s; ", face->str); + g_string_append_printf(style, "font-family: %s; ", g_strstrip(face->str)); g_string_free(face, TRUE); p = q; } else if(!g_ascii_strncasecmp(p, "size=", strlen("size="))) { @@ -1392,7 +1392,7 @@ pt->dest_tag = "span"; tags = g_list_prepend(tags, pt); if(style->len) - g_string_append_printf(xhtml, "", style->str); + g_string_append_printf(xhtml, "", g_strstrip(style->str)); else pt->ignore = TRUE; g_string_free(style, TRUE); @@ -1412,7 +1412,7 @@ color = g_string_append_c(color, *q); q++; } - g_string_append_printf(xhtml, "", color->str); + g_string_append_printf(xhtml, "", g_strstrip(color->str)); g_string_free(color, TRUE); if ((c = strchr(c, '>')) != NULL) c++;