workaround broken links in html
[claws.git] / src / html.c
index 455bc15528f8110730346181a6e7aa1f2f2c7ae8..19cb52c843ff64ed81d4a04ebc5ac5fcd9a5e199 100644 (file)
@@ -1,10 +1,10 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2007 Hiroyuki Yamamoto and the Claws Mail team
+ * Copyright (C) 1999-2012 Hiroyuki Yamamoto and the Claws Mail team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
+ * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
@@ -13,8 +13,8 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * 
  */
 
 #include <glib.h>
@@ -38,94 +38,270 @@ struct _SC_HTMLSymbol
 };
 
 static SC_HTMLSymbol symbol_list[] = {
-       {"&lt;", "\74"},
-       {"&gt;", "\76"},
-       {"&amp;", "\46"},
-       {"&quot;", "\42"},
-       {"&lsquo;", "\47"},
-       {"&rsquo;", "\47"},
-       {"&ldquo;", "\42"},
-       {"&rdquo;", "\42"},
-       {"&laquo;", "\302\253"},
-       {"&raquo;", "\302\273"},
-       {"&nbsp;", "\40"},
-       {"&trade;", "\50\124\115\51"},
-       {"&hellip;", "\56\56\56"},
-       {"&bull;", "\52"},
-       {"&ndash;", "\55"},
-       {"&mdash;", "\55\55"},
-       {"&euro;", "\105\125\122"},
-       {"&cent;", "\302\242"},
-       {"&pound;", "\302\243"},
-       {"&curren;", "\302\244"},
-       {"&yen;", "\302\245"},
-       {"&copy;", "\302\251"},
-       {"&reg;", "\302\256"},
-       {"&iquest;", "\302\277"},
-       {"&iexcl;", "\302\241"}
-};
-
-static SC_HTMLSymbol ascii_symbol_list[] = {
-       {"&iexcl;" , "\302\241"},
-       {"&brvbar;", "\302\246"},
-       {"&copy;"  , "\302\251"},
-       {"&laquo;" , "\302\253"},
-       {"&reg;"   , "\302\256"},
-
-       {"&sup2;"  , "\302\262"},
-       {"&sup3;"  , "\302\263"},
-       {"&acute;" , "\302\264"},
-       {"&cedil;" , "\302\270"},
-       {"&sup1;"  , "\302\271"},
-       {"&raquo;" , "\302\273"},
-       {"&frac14;", "\302\274"},
-       {"&frac12;", "\302\275"},
-       {"&frac34;", "\302\276"},
-       {"&iquest;", "\302\277"},
-
-       {"&Agrave;", "\303\200"},
-       {"&Aacute;", "\303\201"},
-       {"&Acirc;" , "\303\202"},
-       {"&Atilde;", "\303\203"},
-       {"&AElig;" , "\303\206"},
-       {"&Egrave;", "\303\210"},
-       {"&Eacute;", "\303\211"},
-       {"&Ecirc;" , "\303\212"},
-       {"&Igrave;", "\303\214"},
-       {"&Iacute;", "\303\215"},
-       {"&Icirc;" , "\303\216"},
-
-       {"&Ntilde;", "\303\221"},
-       {"&Ograve;", "\303\222"},
-       {"&Oacute;", "\303\223"},
-       {"&Ocirc;" , "\303\224"},
-       {"&Otilde;", "\303\225"},
-       {"&Ugrave;", "\303\231"},
-       {"&Uacute;", "\303\232"},
-       {"&Ucirc;" , "\303\233"},
-       {"&Yacute;", "\303\235"},
-
-       {"&agrave;", "\303\240"},
-       {"&aacute;", "\303\241"},
-       {"&acirc;" , "\303\242"},
-       {"&atilde;", "\303\243"},
-       {"&aelig;" , "\303\246"},
-       {"&egrave;", "\303\250"},
-       {"&eacute;", "\303\251"},
-       {"&ecirc;" , "\303\252"},
-       {"&igrave;", "\303\254"},
-       {"&iacute;", "\303\255"},
-       {"&icirc;" , "\303\256"},
-
-       {"&ntilde;", "\303\261"},
-       {"&ograve;", "\303\262"},
-       {"&oacute;", "\303\263"},
-       {"&ocirc;" , "\303\264"},
-       {"&otilde;", "\303\265"},
-       {"&ugrave;", "\303\271"},
-       {"&uacute;", "\303\272"},
-       {"&ucirc;" , "\303\273"},
-       {"&yacute;", "\303\275"}
+ {"&#34;", "\42"},
+ {"&#38;", "\46"},
+ {"&#39;", "\47"},
+ {"&#60;", "\74"},
+ {"&#62;", "\76"},
+ {"&#034;", "\42"},
+ {"&#038;", "\46"},
+ {"&#039;", "\47"},
+ {"&#060;", "\74"},
+ {"&#062;", "\76"},
+ {"&#146;", "\47"},
+ {"&#153;", "\342\204\242"},
+ {"&#160;", "\40"},
+ {"&#161;", "\302\241"},
+ {"&#162;", "\302\242"},
+ {"&#163;", "\302\243"},
+ {"&#164;", "\302\244"},
+ {"&#165;", "\302\245"},
+ {"&#166;", "\302\246"},
+ {"&#167;", "\302\247"},
+ {"&#168;", "\302\250"},
+ {"&#169;", "\302\251"},
+ {"&#170;", "\302\252"},
+ {"&#171;", "\302\253"},
+ {"&#172;", "\302\254"},
+ {"&#173;", "\302\255"},
+ {"&#174;", "\302\256"},
+ {"&#175;", "\302\257"},
+ {"&#176;", "\302\260"},
+ {"&#177;", "\302\261"},
+ {"&#178;", "\302\262"},
+ {"&#179;", "\302\263"},
+ {"&#180;", "\302\264"},
+ {"&#181;", "\302\265"},
+ {"&#182;", "\302\266"},
+ {"&#183;", "\302\267"},
+ {"&#184;", "\302\270"},
+ {"&#185;", "\302\271"},
+ {"&#186;", "\302\272"},
+ {"&#187;", "\302\273"},
+ {"&#188;", "\302\274"},
+ {"&#189;", "\302\275"},
+ {"&#190;", "\302\276"},
+ {"&#191;", "\302\277"},
+ {"&#192;", "\303\200"},
+ {"&#193;", "\303\201"},
+ {"&#194;", "\303\202"},
+ {"&#195;", "\303\203"},
+ {"&#196;", "\303\204"},
+ {"&#197;", "\303\205"},
+ {"&#198;", "\303\206"},
+ {"&#199;", "\303\207"},
+ {"&#200;", "\303\210"},
+ {"&#201;", "\303\211"},
+ {"&#202;", "\303\212"},
+ {"&#203;", "\303\213"},
+ {"&#204;", "\303\214"},
+ {"&#205;", "\303\215"},
+ {"&#206;", "\303\216"},
+ {"&#207;", "\303\217"},
+ {"&#208;", "\303\220"},
+ {"&#209;", "\303\221"},
+ {"&#210;", "\303\222"},
+ {"&#211;", "\303\223"},
+ {"&#212;", "\303\224"},
+ {"&#213;", "\303\225"},
+ {"&#214;", "\303\226"},
+ {"&#215;", "\303\227"},
+ {"&#216;", "\303\230"},
+ {"&#217;", "\303\231"},
+ {"&#218;", "\303\232"},
+ {"&#219;", "\303\233"},
+ {"&#220;", "\303\234"},
+ {"&#221;", "\303\235"},
+ {"&#222;", "\303\236"},
+ {"&#223;", "\303\237"},
+ {"&#224;", "\303\240"},
+ {"&#225;", "\303\241"},
+ {"&#226;", "\303\242"},
+ {"&#227;", "\303\243"},
+ {"&#228;", "\303\244"},
+ {"&#229;", "\303\245"},
+ {"&#230;", "\303\246"},
+ {"&#231;", "\303\247"},
+ {"&#232;", "\303\250"},
+ {"&#233;", "\303\251"},
+ {"&#234;", "\303\252"},
+ {"&#235;", "\303\253"},
+ {"&#236;", "\303\254"},
+ {"&#237;", "\303\255"},
+ {"&#238;", "\303\256"},
+ {"&#239;", "\303\257"},
+ {"&#240;", "\303\260"},
+ {"&#241;", "\303\261"},
+ {"&#242;", "\303\262"},
+ {"&#243;", "\303\263"},
+ {"&#244;", "\303\264"},
+ {"&#245;", "\303\265"},
+ {"&#246;", "\303\266"},
+ {"&#247;", "\303\267"},
+ {"&#248;", "\303\270"},
+ {"&#249;", "\303\271"},
+ {"&#250;", "\303\272"},
+ {"&#251;", "\303\273"},
+ {"&#252;", "\303\274"},
+ {"&#253;", "\303\275"},
+ {"&#254;", "\303\276"},
+ {"&#255;", "\303\277"},
+ {"&#338;", "\305\222"},
+ {"&#339;", "\305\223"},
+ {"&#352;", "\305\240"},
+ {"&#353;", "\305\241"},
+ {"&#376;", "\305\270"},
+ {"&#710;", "\313\206"},
+ {"&#732;", "\313\234"},
+ {"&#8194;", "\342\200\202"},
+ {"&#8195;", "\342\200\203"},
+ {"&#8201;", "\342\200\211"},
+ {"&#8211;", "\342\200\223"},
+ {"&#8212;", "\342\200\224"},
+ {"&#8216;", "\342\200\230"},
+ {"&#8217;", "\342\200\231"},
+ {"&#8218;", "\342\200\232"},
+ {"&#8220;", "\342\200\234"},
+ {"&#8221;", "\342\200\235"},
+ {"&#8222;", "\342\200\236"},
+ {"&#8224;", "\342\200\240"},
+ {"&#8225;", "\342\200\241"},
+ {"&#8226;", "\342\200\242"},
+ {"&#8230;", "\342\200\246"},
+ {"&#8240;", "\342\200\260"},
+ {"&#8249;", "\342\200\271"},
+ {"&#8250;", "\342\200\272"},
+ {"&#8364;", "\342\202\254"},
+ {"&#8482;", "\342\204\242"},
+ {"&quot;", "\42"},
+ {"&amp;", "\46"},
+ {"&apos;", "\47"},
+ {"&lt;", "\74"},
+ {"&gt;", "\76"},
+ {"&squot;", "\47"},
+ {"&nbsp;", "\40"},
+ {"&iexcl;", "\302\241"},
+ {"&cent;", "\302\242"},
+ {"&pound;", "\302\243"},
+ {"&curren;", "\302\244"},
+ {"&yen;", "\302\245"},
+ {"&brvbar;", "\302\246"},
+ {"&sect;", "\302\247"},
+ {"&uml;", "\302\250"},
+ {"&copy;", "\302\251"},
+ {"&ordf;", "\302\252"},
+ {"&laquo;", "\302\253"},
+ {"&not;", "\302\254"},
+ {"&shy;", "\302\255"},
+ {"&reg;", "\302\256"},
+ {"&macr;", "\302\257"},
+ {"&deg;", "\302\260"},
+ {"&plusmn;", "\302\261"},
+ {"&sup2;", "\302\262"},
+ {"&sup3;", "\302\263"},
+ {"&acute;", "\302\264"},
+ {"&micro;", "\302\265"},
+ {"&para;", "\302\266"},
+ {"&middot;", "\302\267"},
+ {"&cedil;", "\302\270"},
+ {"&sup1;", "\302\271"},
+ {"&ordm;", "\302\272"},
+ {"&raquo;", "\302\273"},
+ {"&frac14;", "\302\274"},
+ {"&frac12;", "\302\275"},
+ {"&frac34;", "\302\276"},
+ {"&iquest;", "\302\277"},
+ {"&Agrave;", "\303\200"},
+ {"&Aacute;", "\303\201"},
+ {"&Acirc;", "\303\202"},
+ {"&Atilde;", "\303\203"},
+ {"&Auml;", "\303\204"},
+ {"&Aring;", "\303\205"},
+ {"&AElig;", "\303\206"},
+ {"&Ccedil;", "\303\207"},
+ {"&Egrave;", "\303\210"},
+ {"&Eacute;", "\303\211"},
+ {"&Ecirc;", "\303\212"},
+ {"&Euml;", "\303\213"},
+ {"&Igrave;", "\303\214"},
+ {"&Iacute;", "\303\215"},
+ {"&Icirc;", "\303\216"},
+ {"&Iuml;", "\303\217"},
+ {"&ETH;", "\303\220"},
+ {"&Ntilde;", "\303\221"},
+ {"&Ograve;", "\303\222"},
+ {"&Oacute;", "\303\223"},
+ {"&Ocirc;", "\303\224"},
+ {"&Otilde;", "\303\225"},
+ {"&Ouml;", "\303\226"},
+ {"&times;", "\303\227"},
+ {"&Oslash;", "\303\230"},
+ {"&Ugrave;", "\303\231"},
+ {"&Uacute;", "\303\232"},
+ {"&Ucirc;", "\303\233"},
+ {"&Uuml;", "\303\234"},
+ {"&Yacute;", "\303\235"},
+ {"&THORN;", "\303\236"},
+ {"&szlig;", "\303\237"},
+ {"&agrave;", "\303\240"},
+ {"&aacute;", "\303\241"},
+ {"&acirc;", "\303\242"},
+ {"&atilde;", "\303\243"},
+ {"&auml;", "\303\244"},
+ {"&aring;", "\303\245"},
+ {"&aelig;", "\303\246"},
+ {"&ccedil;", "\303\247"},
+ {"&egrave;", "\303\250"},
+ {"&eacute;", "\303\251"},
+ {"&ecirc;", "\303\252"},
+ {"&euml;", "\303\253"},
+ {"&igrave;", "\303\254"},
+ {"&iacute;", "\303\255"},
+ {"&icirc;", "\303\256"},
+ {"&iuml;", "\303\257"},
+ {"&eth;", "\303\260"},
+ {"&ntilde;", "\303\261"},
+ {"&ograve;", "\303\262"},
+ {"&oacute;", "\303\263"},
+ {"&ocirc;", "\303\264"},
+ {"&otilde;", "\303\265"},
+ {"&ouml;", "\303\266"},
+ {"&divide;", "\303\267"},
+ {"&oslash;", "\303\270"},
+ {"&ugrave;", "\303\271"},
+ {"&uacute;", "\303\272"},
+ {"&ucirc;", "\303\273"},
+ {"&uuml;", "\303\274"},
+ {"&yacute;", "\303\275"},
+ {"&thorn;", "\303\276"},
+ {"&yuml;", "\303\277"},
+ {"&OElig;", "\305\222"},
+ {"&oelig;", "\305\223"},
+ {"&Scaron;", "\305\240"},
+ {"&scaron;", "\305\241"},
+ {"&Yuml;", "\305\270"},
+ {"&circ;", "\313\206"},
+ {"&tilde;", "\313\234"},
+ {"&ensp;", "\342\200\202"},
+ {"&emsp;", "\342\200\203"},
+ {"&thinsp;", "\342\200\211"},
+ {"&ndash;", "\342\200\223"},
+ {"&mdash;", "\342\200\224"},
+ {"&lsquo;", "\342\200\230"},
+ {"&rsquo;", "\342\200\231"},
+ {"&sbquo;", "\342\200\232"},
+ {"&ldquo;", "\342\200\234"},
+ {"&rdquo;", "\342\200\235"},
+ {"&bdquo;", "\342\200\236"},
+ {"&dagger;", "\342\200\240"},
+ {"&Dagger;", "\342\200\241"},
+ {"&bull;", "\342\200\242"},
+ {"&hellip;", "\342\200\246"},
+ {"&permil;", "\342\200\260"},
+ {"&lsaquo;", "\342\200\271"},
+ {"&rsaquo;", "\342\200\272"},
+ {"&euro;", "\342\202\254"},
+ {"&trade;", "\342\204\242"}
 };
 
 typedef struct _SC_HTMLAltSymbol       SC_HTMLAltSymbol;
@@ -136,53 +312,7 @@ struct _SC_HTMLAltSymbol
        gchar *const val;
 };
 
-/* http://www.w3schools.com/html/html_entitiesref.asp */
-static SC_HTMLAltSymbol alternate_symbol_list[] = {
-       {  96, "\140"},                    /* backtick */
-       { 153, "\50\124\115\51"},  /* trademark */
-       { 161, "\302\241"},        /* inverted exclamation mark &iexcl */
-       { 162, "\302\242"},        /* cent (currency) &cent */
-       { 163, "\302\243"},        /* pound (currency) &pound */
-       { 164, "\342\202\254"},    /* currency sign &curren */
-       { 165, "\302\245"},        /* yen (currency) &yen */
-       { 169, "\302\251"},        /* copyright sign &copy */
-       { 174, "\302\256"},        /* registered sign &reg */
-       { 191, "\302\277"},        /* inverted question mark &iquest */
-       { 338, "\117\105"},        /* capital ligature OE &OElig */
-       { 339, "\157\145"},        /* small ligature OE &oelig */
-       { 352, NULL},                      /* capital S w/caron &Scaron */
-       { 353, NULL},                      /* small S w/caron &scaron */
-       { 376, NULL},                      /* cap Y w/ diaeres &Yuml */
-       { 710, "\136"},                    /* circumflex accent &circ */
-       { 732, "\176"},                    /* small tilde &tilde */
-       {8194, "\40"},                     /* en space &ensp */
-       {8195, "\40"},                     /* em space &emsp */
-       {8201, "\40"},                     /* thin space &thinsp */
-       {8204, NULL},                      /* zero width non-joiner &zwnj */
-       {8205, NULL},                      /* zero width joiner &zwj */
-       {8206, NULL},                      /* l-t-r mark &lrm */
-       {8207, NULL},                      /* r-t-l mark &rlm */
-       {8211, "\55"},                     /* en dash &ndash */
-       {8212, "\55\55"},                  /* em dash &mdash */
-       {8216, "\47"},                     /* l single quot mark &lsquo */
-       {8217, "\47"},                     /* r single quot mark &rsquo */
-       {8218, "\54"},                     /* single low-9 quot &sbquo */
-       {8220, "\134"},                    /* l double quot mark &ldquo */
-       {8221, "\134"},                    /* r double quot mark &rdquo */
-       {8222, "\42"},                     /* double low-9 quot &bdquo */
-       {8224, NULL},                      /* dagger &dagger */
-       {8225, NULL},                      /* double dagger &Dagger */
-       {8226, "\52"},                     /* bullet &bull */
-       {8230, "\56\56\56"},       /* horizontal ellipsis &hellip */
-       {8240, "\45\157"},                 /* per mile &permil */
-       {8249, "\74"},                     /* l-pointing angle quot &lsaquo */
-       {8250, "\76"},                     /* r-pointing angle quot &rsaquo */
-       {8364, "\105\125\122"},    /* euro &euro */
-       {8482, "\50\124\115\51"}   /* trademark &trade */
-};
-
 static GHashTable *default_symbol_table;
-static GHashTable *alternate_symbol_table;
 
 static SC_HTMLState sc_html_read_line  (SC_HTMLParser  *parser);
 static void sc_html_append_char                        (SC_HTMLParser  *parser,
@@ -201,8 +331,8 @@ SC_HTMLParser *sc_html_parser_new(FILE *fp, CodeConverter *conv)
 {
        SC_HTMLParser *parser;
 
-       g_return_val_if_fail(fp != NULL, NULL);
-       g_return_val_if_fail(conv != NULL, NULL);
+       cm_return_val_if_fail(fp != NULL, NULL);
+       cm_return_val_if_fail(conv != NULL, NULL);
 
        parser = g_new0(SC_HTMLParser, 1);
        parser->fp = fp;
@@ -236,19 +366,12 @@ SC_HTMLParser *sc_html_parser_new(FILE *fp, CodeConverter *conv)
                default_symbol_table =
                        g_hash_table_new(g_str_hash, g_str_equal);
                SYMBOL_TABLE_ADD(default_symbol_table, symbol_list);
-               SYMBOL_TABLE_ADD(default_symbol_table, ascii_symbol_list);
-       }
-       if (!alternate_symbol_table) {
-               alternate_symbol_table =
-                       g_hash_table_new(g_int_hash, g_int_equal);
-               SYMBOL_TABLE_REF_ADD(alternate_symbol_table, alternate_symbol_list);
        }
 
 #undef SYMBOL_TABLE_ADD
 #undef SYMBOL_TABLE_REF_ADD
 
        parser->symbol_table = default_symbol_table;
-       parser->alt_symbol_table = alternate_symbol_table;
 
        return parser;
 }
@@ -317,11 +440,17 @@ static SC_HTMLState sc_html_read_line(SC_HTMLParser *parser)
        gchar buf[SC_HTMLBUFSIZE];
        gchar buf2[SC_HTMLBUFSIZE];
        gint index;
+       gint n;
 
-       if (fgets(buf, sizeof(buf), parser->fp) == NULL) {
+       if (parser->fp == NULL)
+               return SC_HTML_EOF;
+
+       n = fread(buf, 1, sizeof(buf) - 1, parser->fp);
+       if (n == 0) {
                parser->state = SC_HTML_EOF;
                return SC_HTML_EOF;
-       }
+       } else
+               buf[n] = '\0';
 
        if (conv_convert(parser->conv, buf2, sizeof(buf2), buf) < 0) {
                index = parser->bufp - parser->buf->str;
@@ -396,7 +525,7 @@ static SC_HTMLTag *sc_html_get_tag(const gchar *str)
        gchar *tmp;
        guchar *tmpp;
 
-       g_return_val_if_fail(str != NULL, NULL);
+       cm_return_val_if_fail(str != NULL, NULL);
 
        if (*str == '\0' || *str == '!') return NULL;
 
@@ -408,13 +537,11 @@ static SC_HTMLTag *sc_html_get_tag(const gchar *str)
                ;
 
        if (*tmpp == '\0') {
-               g_strdown(tmp);
-               tag->name = g_strdup(tmp);
+               tag->name = g_utf8_strdown(tmp, -1);
                return tag;
        } else {
                *tmpp++ = '\0';
-               g_strdown(tmp);
-               tag->name = g_strdup(tmp);
+               tag->name = g_utf8_strdown(tmp, -1);
        }
 
        while (*tmpp != '\0') {
@@ -445,7 +572,7 @@ static SC_HTMLTag *sc_html_get_tag(const gchar *str)
                                tmpp++;
                                attr_value = tmpp;
                                if ((p = strchr(attr_value, quote)) == NULL) {
-                                       g_warning("sc_html_get_tag(): syntax error in tag: '%s'\n", str);
+                                       g_warning("sc_html_get_tag(): syntax error in tag: '%s'", str);
                                        return tag;
                                }
                                tmpp = p;
@@ -462,9 +589,8 @@ static SC_HTMLTag *sc_html_get_tag(const gchar *str)
                        attr_value = "";
 
                g_strchomp(attr_name);
-               g_strdown(attr_name);
                attr = g_new(SC_HTMLAttr, 1);
-               attr->name = g_strdup(attr_name);
+               attr->name = g_utf8_strdown(attr_name, -1);
                attr->value = g_strdup(attr_value);
                tag->attr = g_list_append(tag->attr, attr);
        }
@@ -487,6 +613,24 @@ static void sc_html_free_tag(SC_HTMLTag *tag)
        g_free(tag);
 }
 
+static void decode_href(SC_HTMLParser *parser)
+{
+       gchar *tmp;
+       SC_HTMLParser *tparser = g_new0(SC_HTMLParser, 1);
+
+       tparser->str = g_string_new(NULL);
+       tparser->buf = g_string_new(parser->href);
+       tparser->bufp = tparser->buf->str;
+       tparser->symbol_table = default_symbol_table;
+       
+       tmp = sc_html_parse(tparser);
+       
+       g_free(parser->href);
+       parser->href = g_strdup(tmp);
+
+       sc_html_parser_destroy(tparser);
+}
+
 static SC_HTMLState sc_html_parse_tag(SC_HTMLParser *parser)
 {
        gchar buf[SC_HTMLBUFSIZE];
@@ -499,7 +643,7 @@ static SC_HTMLState sc_html_parse_tag(SC_HTMLParser *parser)
        parser->state = SC_HTML_UNKNOWN;
        if (!tag) return SC_HTML_UNKNOWN;
 
-       if (!strcmp(tag->name, "br")) {
+       if (!strcmp(tag->name, "br") || !strcmp(tag->name, "br/")) {
                parser->space = FALSE;
                sc_html_append_char(parser, '\n');
                parser->state = SC_HTML_BR;
@@ -509,10 +653,12 @@ static SC_HTMLState sc_html_parse_tag(SC_HTMLParser *parser)
                        if (cur->data && !strcmp(((SC_HTMLAttr *)cur->data)->name, "href")) {
                                g_free(parser->href);
                                parser->href = g_strdup(((SC_HTMLAttr *)cur->data)->value);
+                               decode_href(parser);
                                parser->state = SC_HTML_HREF_BEG;
                                break;
                        }
                }
+               parser->state = SC_HTML_HREF_BEG;
        } else if (!strcmp(tag->name, "/a")) {
                parser->state = SC_HTML_HREF;
        } else if (!strcmp(tag->name, "p")) {
@@ -579,7 +725,7 @@ static void sc_html_parse_special(SC_HTMLParser *parser)
        const gchar *val;
 
        parser->state = SC_HTML_UNKNOWN;
-       g_return_if_fail(*parser->bufp == '&');
+       cm_return_if_fail(*parser->bufp == '&');
 
        /* &foo; */
        for (n = 0; parser->bufp[n] != '\0' && parser->bufp[n] != ';'; n++)
@@ -598,35 +744,33 @@ static void sc_html_parse_special(SC_HTMLParser *parser)
                sc_html_append_str(parser, val, -1);
                parser->state = SC_HTML_NORMAL;
                return;
-       } else if (symbol_name[1] == '#' && g_ascii_isdigit(symbol_name[2])) {
-               gint ch;
-
-               ch = atoi(symbol_name + 2);
-               if ((ch > 0 && ch <= 127) ||
-                   (ch >= 128 && ch <= 255 &&
-                    parser->conv->charset == C_ISO_8859_1)) {
-                       sc_html_append_char(parser, ch);
-                       parser->state = SC_HTML_NORMAL;
-                       return;
-               } else {
-                       const gchar *symb = g_hash_table_lookup(parser->alt_symbol_table, &ch);
-                       if (symb) {
-                               sc_html_append_str(parser, symb, -1);
-                               parser->state = SC_HTML_NORMAL;
-                               return;
-                       }
-               }
-       }
+       } 
 
        sc_html_append_str(parser, symbol_name, -1);
 }
 
+static gchar *sc_html_find_tag(SC_HTMLParser *parser, const gchar *tag)
+{
+       gchar *cur = parser->bufp;
+       gint len = strlen(tag);
+
+       if (cur == NULL)
+               return NULL;
+
+       while ((cur = strstr(cur, "<")) != NULL) {
+               if (!g_ascii_strncasecmp(cur, tag, len))
+                       return cur;
+               cur += 2;
+       }
+       return NULL;
+}
+
 static void sc_html_get_parenthesis(SC_HTMLParser *parser, gchar *buf, gint len)
 {
        gchar *p;
 
        buf[0] = '\0';
-       g_return_if_fail(*parser->bufp == '<');
+       cm_return_if_fail(*parser->bufp == '<');
 
        /* ignore comment / CSS / script stuff */
        if (!strncmp(parser->bufp, "<!--", 4)) {
@@ -638,14 +782,14 @@ static void sc_html_get_parenthesis(SC_HTMLParser *parser, gchar *buf, gint len)
        }
        if (!g_ascii_strncasecmp(parser->bufp, "<style", 6)) {
                parser->bufp += 6;
-               while ((p = strcasestr(parser->bufp, "</style>")) == NULL)
+               while ((p = sc_html_find_tag(parser, "</style>")) == NULL)
                        if (sc_html_read_line(parser) == SC_HTML_EOF) return;
                parser->bufp = p + 8;
                return;
        }
        if (!g_ascii_strncasecmp(parser->bufp, "<script", 7)) {
                parser->bufp += 7;
-               while ((p = strcasestr(parser->bufp, "</script>")) == NULL)
+               while ((p = sc_html_find_tag(parser, "</script>")) == NULL)
                        if (sc_html_read_line(parser) == SC_HTML_EOF) return;
                parser->bufp = p + 9;
                return;