Re: [PATCH] xterm titlebar
- From: Tomas Styblo <tripie cpan org>
- To: mc-devel gnome org
- Subject: Re: [PATCH] xterm titlebar
- Date: Mon, 16 Dec 2002 18:09:17 +0100
* 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]