Re: lynx-dev Is this as intended? - #3 force_empty_hrefless_a

[Vlad Harchev]

On Sat, 4 Dec 1999, Klaus Weide wrote:

> In dev.15 / dev.16, HTML.c has the appended section.
> Something is missing.  The comments says the bahvior depends
> on force_empty_hrefless_a, but no related variable is tested.
>   
>    Klaus

 Sorry, I overlooed that. That bug is in lynx from dev14...
 I wonder that nobody noted/reported change in colors of lynx's screen (all
hypherlink targets don't have emphasis, if requested). Fix provided.
 
> 
>         if (me->inBoldA == TRUE && me->inBoldH == FALSE)
>             HText_appendCharacter(me->text, LY_BOLD_START_CHAR);
> #if defined(NOTUSED_FOTEMODS)
>         /*
>          *  Close an HREF-less NAMED-ed now if we aren't making their
>          *  content bold, and let the check in HTML_end_element() deal
>          *  with any dangling end tag this creates. - FM
>          */
>         if (href == NULL && me->inBoldA == FALSE) {
>             SET_SKIP_STACK(HTML_A);
>             HTML_end_element(me, HTML_A, include);
>         }
> #else
>         /*Close an HREF-less NAMED-ed now if force_empty_hrefless_a was
>             requested - VH*/
-         if (href == NULL) {
+         if (href == NULL && force_empty_hrefless_a) {
>             SET_SKIP_STACK(HTML_A);
>             HTML_end_element(me, HTML_A, include);
>         }
> #endif
> 

 Best regards,
  -Vlad