Re: lynx-dev [PATCH] "change directory" command

[Ismael Cordeiro]

On Mon, 12 Jun 2000, Vlad Harchev wrote:

> On Mon, 12 Jun 2000, Thomas Dickey wrote:
> 
> > On Mon, Jun 12, 2000 at 03:35:12PM +0500, Vlad Harchev wrote:
> > > 
> > >  Here is a first version of implementation of "change directory"
> > >  command.
> > 
> > you have to be careful with this - I have a hunch that it'll break a lot
> > of dired configuration.
> 
> What do you suspect exactly? I tested all dired ops and they worked fine.
> And one of the proofs: chdir'ing to /tmp and then 'g'oing to '.' shows the
> list of /tmp (so all seem correct) independant of the directory where lynx
> was started.
> Reloading of lynx.cfg works fine too, 'm' works too - seems most commands
> are ok (I haven't encountered op for which behaviour became broken
> changed).

It seems that Thomas is right, your patch broke something. In the prompt
"D)ownload, or C)ancel" the "c" and "C" commands don't work anymore.
Commenting out "#define "SUPPORT_CHDIR" in userdefs.h, as suggested in the
CHANGES file, doesn't re-enable the commands above.

By the way, I think that it would be better to make clear that what the
command does is to change the _current_ directory by calling it "change
current directory" everywhere and by changing SUPPORT_CHDIR to something like
SUPPORT_CHCURDIR.

Ismael
-- 
       ________________________________________________________________
      |                                                                |
      | ISMAEL CORDEIRO            | mailto:address@hidden        |
      | Production sound mixer     | http://www.pubnix.net/~ismael/    |
      | Montréal - Québec - Canada | ftp://ftp.pubnix.net/users/ismael |
      |________________________________________________________________|


; To UNSUBSCRIBE: Send "unsubscribe lynx-dev" to address@hidden