Re: lynx-dev lynx 2.8.2 dev8 bug

[Klaus Weide]

On Fri, 4 Dec 1998, Larry W. Virden wrote:

> >   http://www.heise.de/newsticker/
> one pecularity I see on that page with lynx 2.8.2rel8 is that link 2 on
> the page is numbered (I use numbered links) but I cannot get to it by
> moving the cursor.  Also , when I type a g 2, I find the cursor moves
> to link 3.

Tom has included my original patch for HText_endAnchor 
   <URL: http://www.flora.org/lynx-dev/html/month1298/msg00002.html>
into 2.8.2dev.8, but not (yet) the corrected patch
   <URL: http://www.flora.org/lynx-dev/html/month1298/msg00036.html>.
The buggy original patch creates the behavior you observe.

This should update dev.8 to the corrected version:

*** lynx2-8-1.wrong/src/GridText.c      Mon Nov 30 18:47:04 1998
--- lynx2-8-1/src/GridText.c    Tue Dec  1 16:38:48 1998
***************
*** 2820,2826 ****
             */
            i = a->extent;
        }
!       j = (last->size - i);
        while (j < last->size) {
            if (!IsSpecialAttrChar(last->data[j]) &&
                !isspace((unsigned char)last->data[j]) &&
--- 2820,2826 ----
             */
            i = a->extent;
        }
!       k = j = (last->size - i);
        while (j < last->size) {
            if (!IsSpecialAttrChar(last->data[j]) &&
                !isspace((unsigned char)last->data[j]) &&
***************
*** 2859,2869 ****
        while (i == 0 &&
               (a->extent > CurBlankExtent ||
                (a->extent == CurBlankExtent &&
                 prev != text->last_line &&
                 (prev->size == 0 ||
                  prev->data[prev->size - 1] == ']')))) {
            start = prev;
!           j = prev->size - a->extent + CurBlankExtent;
            if (j < 0) {
                /*
                 *  The anchor starts on a preceding line,
--- 2859,2870 ----
        while (i == 0 &&
               (a->extent > CurBlankExtent ||
                (a->extent == CurBlankExtent &&
+                k == 0 && 
                 prev != text->last_line &&
                 (prev->size == 0 ||
                  prev->data[prev->size - 1] == ']')))) {
            start = prev;
!           k = j = prev->size - a->extent + CurBlankExtent;
            if (j < 0) {
                /*
                 *  The anchor starts on a preceding line,
***************
*** 2888,2894 ****
            }
            if (i == 0) {
                if (a->extent > (CurBlankExtent + prev->size) ||
!                   (a->extent == CurBlankExtent + prev->size &&
                     prev->prev != text->last_line &&
                     (prev->prev->size == 0 ||
                      prev->prev->data[prev->prev->size - 1] == ']'))) {
--- 2889,2896 ----
            }
            if (i == 0) {
                if (a->extent > (CurBlankExtent + prev->size) ||
!                   (a->extent == CurBlankExtent + (int)prev->size &&
!                    k == 0 &&
                     prev->prev != text->last_line &&
                     (prev->prev->size == 0 ||
                      prev->prev->data[prev->prev->size - 1] == ']'))) {