[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