[sylpheed:34666] Re: Short tags in displaying xhtml-only mails
Hiroyuki Yamamoto
hiro-y at kcn.ne.jp
Wed Jul 20 17:11:09 JST 2011
Hello,
On Tue, 19 Jul 2011 21:42:38 +0200
Ervin Peters <ervin.peters at ervnet.de> wrote:
> There is something in html.c line 470 which looks like
>
> if (!strcmp(tag->name, "br")) {
> parser->space = FALSE;
> html_append_char(parser, '\n');
> parser->state = HTML_BR;
>
> and mayby could be changed to
>
> if (strcasecmp(tag->name, "br", 2 ) <= 0) {
> parser->space = FALSE;
> html_append_char(parser, '\n');
> parser->state = HTML_BR;
>
> similar with the hr tag on line 505.
>
> line numbers from sylpheed-3.2.0beta1, currently downloaded
I have fixed the html parser at svn r2927. Thanks for pointing it out.
I modified html_get_tag() instead to handle the following case
correctly:
<br/>
<name value="foo" />
--
Hiroyuki Yamamoto <hiro-y at kcn.ne.jp>
More information about the Sylpheed
mailing list