Re: [Lynx-dev] patch pretty-src: fix attr w/o value, 1 other problem still persistent

[Thomas Dickey]

On Sat, Feb 17, 2007 at 05:46:34PM +0100, Rado S wrote:
> Moin moin,
> 
> the attached patch fixes the previously mentioned problem that the
> coloring of pretty-src gets messed up when an attr without a value
> appears and is not the last one before the closing '>'.
> Please verify, then enjoy.

thanks
 
> However, there is another strangeness that I have no idea for:
> 
> Setup: 150x109 linux-xterm (== + color) remote ssh to solaris 8,
> TERM=nxterm, ncurses5.2, different fg/bg colors for all the
> different pretty-src items.
> 
> See the source of
>       http://www.w3.org/TR/html401/index/attributes.html
> That should be all OK.
> Compare with
>       http://www.w3.org/TR/html401/index/elements.html
> 
> At the beginning the
>       '<A HREF="attributes.html[10]"> attributes</A>'
> before
>       '<H1 ALIGN="center">Index of Elements<A
> NAME="h-2">&nbsp;</A></H1>'
> 
> The color of the href value is already wrong, and the following
> text of inside the H1 and all text outside of any <tag> has wrong
> color. I have no idea what triggers this.
> When I break that long line with the HREF in the source in 2
> lines, then the colors are OK. Maybe something about line-wrapping
> or bad luck with processing at right margin?
> 
> -- 
> ? Rado S. -- You must provide YOUR effort for your goal!
> EVERY effort counts: at least to show your attitude.
> You're responsible for ALL of it: you get what you give.

> --- ../SGML.c Sat Feb 17 16:51:47 2007
> +++ ./SGML.c  Sat Feb 17 16:52:43 2007
> @@ -3521,9 +3521,11 @@
>                       LYUpperCase(string->data);
>               }
>               PUTS(string->data);
> +             if (c == ' ')
> +                 PUTC(' ');
>               if (c == '=')
>                   PUTC('=');
> -             if (c == '=' || c == '>') {
> +             if (c == ' ' || c == '=' || c == '>') {
>                   if (context->current_attribute_number == INVALID)
>                       PSRCSTOP(badattr);
>                   else

> _______________________________________________
> Lynx-dev mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/lynx-dev


-- 
Thomas E. Dickey <address@hidden>
http://invisible-island.net
ftp://invisible-island.net