Re: [PATCH] xterm titlebar



* Adam Byrtek 'alpha' <alpha student uci agh edu pl> [Mon, 16 Dec 2002]:
> On Mon, Dec 16, 2002 at 03:52:56AM +0100, Tomas Styblo wrote:
> >     1) The malloced space was never freed. Fixed by replacing
> >        it with an automatic array.
> 
> Allocation is not necessary. cpanel->cwd gives us working dir.

Yeah, it's actually amusing how often the simplest solution is
the most correct one :)

Regarding the comments in the bug tracker, I also think that the
patch is already very useful. Configurability would be nice, but
it's not necessary, IMHO.

The only problem I see is that when you switch to the background
shell using CTRL+o, "cd" to some other directory, then the title
is not updated. I don't how difficult would be to correct it ...

-- 
Tomas Styblo <tripie cpan org>
PGP: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0xC97EA4B6



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]