Re: [Lynx-dev] Lynx 2.8.9dev.12 - entered link numbers don't echo to screen

[Thomas Dickey]

On Sat, Apr 29, 2017 at 03:56:02PM +0100, Larry Hynes wrote:
> Thomas Dickey <address@hidden> wrote:
> > On Sat, Apr 29, 2017 at 02:32:03PM +0100, Larry Hynes wrote:
> > > Thomas Dickey <address@hidden> wrote:
> > > > On Sat, Apr 29, 2017 at 12:42:26PM +0100, Larry Hynes wrote:
> > > > > Hi
> > > > > 
> > > > > I built and installed Lynx 2.8.9dev.12 (on Darwin, OS X "Yosemite"
> > > > > 10.10.5) this morning.
> > > > > 
> > > > > Entering a link number opens the
> > > > > 
> > > > >       'Follow link (or goto link or page) number:'
> > > > > 
> > > > > dialog, but without 'pre-filling' it with the number that has been
> > > > > entered. All subsequently entered numbers do not echo to the dialog
> > > > > prompt. Hitting return selects the entered link number correctly.
> > > > 
> > > > I see (will fix, and put out a new version)
> > > 
> > > Great - thanks.
> > 
> > It's a small change -
> > 
> > diff -u -r1.266 src/LYStrings.c
> > --- src/LYStrings.c 2017/04/29 00:32:21     1.266
> > +++ src/LYStrings.c 2017/04/29 14:38:32
> > @@ -3112,6 +3112,8 @@
> >         have = LYstrExtent2(s, pos);
> >         CTRACE_EDIT((tfp, "  %2d:%2d:%.*s\n", pos, have, pos, s));
> >         if (have >= cells) {
> > +           if (cells <= 0)
> > +               break;
> >             /* the best solution is the one with the most bytes */
> >             best = pos;
> >         }
> > 
> > ...which doesn't appear to break last night's change :-)
> 
> Heh.
> 
> That's working here now, thanks.

no problem - the need for a special case wasn't apparent
until studying the debug-log

(I'll make a .13 to fix the regression)

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

signature.asc
Description: Digital signature