Re: isearch "wrap around" enhancements

[Lloyd Zusman <ljz@xxxxxxxxxx>]

At Fri, 19 Nov 2004 05:23:46 -0500,
Lloyd Zusman wrote:
> 
> At Fri, 19 Nov 2004 02:04:27 -0500,
> John E. Davis wrote:
> > 
> > Lloyd Zusman <ljz@xxxxxxxxxx> wrote:
> > >To John Davis: this new behavior is optional, so would you be willing to
> > >consider this new functionality to become part of the official isearch
> > >capabilities for jed?  Read on for details ...
> > 
> > I have added the patch.  In doing so I wanted to clean up the code
> > involving the placing of marks on the interpreter's stack.  That
> > portion of code is very confusing and dates from a time when slang did
> > not have structured types.  I believe that my new implementation is
> > cleaner and more maintainable.  However, there is always the
> > possibility that I broke something.  I would appreciate it if you try
> > this version because it will be the one that I will release.  
> > Thanks, --John

One more thing: the following code near the bottom of the isearch.sl
file is superfluous ...

  % This needs to be reset in case we break out of the loop right
  % after a failed isearch attempt.
  Last_Search_Failed = 0;
                                                      
That's because immediately thereafer, EXECUTE_ERROR_BLOCK is invoked,
and that block also contains a "Last_Search_Failed = 0" statement.

I had noticed this earlier, but I forgot to remove the superfluous code
before submitting my patch.
  

-- 
 Lloyd Zusman
 ljz@xxxxxxxxxx
 God bless you.

--------------------------
To unsubscribe send email to <jed-users-request@xxxxxxxxxxx> with
the word "unsubscribe" in the message body.
Need help? Email <jed-users-owner@xxxxxxxxxxx>.