Re: lynx-dev br (ok, it's tiny)

[dickey]

> From: address@hidden 
> > I didn't notice on the usual spot checks (directory mode and help file), 
> > but did see that on my own webpage.  Will send out a fix today (unless 
> > someone else does first). 
>  
> I noticed it first on AltaVista's page in a minimal way.  Then I started 
> webbing around and noticed it worse. 
>  
> I took a look and even made a stab at changing things in src/HTML.c to 
> move back to the original code - but i saw no difference.  I am babysitting 
> and don't have a lot of time to dig, so I will bear up for tonite and 
> hopefully someone (TD or other) will take a stab soon at a better fix. 
It's a small fix: try it, and if it works for you too, I'll check it in.

--- HTML.c.orig Sat Aug 29 19:26:30 1998
+++ HTML.c      Sun Aug 30 19:38:46 1998
@@ -1552,13 +1552,13 @@
        CHECK_ID(HTML_GEN_ID);
          /* Add a \r (new line) if these three conditions are true:
           *   1. We are not collapsing BR's, and
-          *   2. This line has text on it, or
-          *   3. The previous line has text on it.
+          *   2. The previous line has text on it, or
+          *   3. This line has text on it.
           * Otherwise, don't do anything. -DH 980814, TD 980827
           */
-       if ((LYCollapseBRs == FALSE) &&
-           (HText_LastLineSize(me->text, FALSE) ||
-            HText_PreviousLineSize(me->text, FALSE))) {
+       if ((LYCollapseBRs == FALSE &&
+            HText_PreviousLineSize(me->text, FALSE)) ||
+           HText_LastLineSize(me->text, FALSE)) {
            HText_setLastChar(me->text, ' ');  /* absorb white space */
            HText_appendCharacter(me->text, '\r');
        }

> Larry W. Virden                 INET: address@hidden 

-- 
Thomas E. Dickey
address@hidden
http://www.clark.net/pub/dickey