On Fri, Jul 28, 2006 at 04:09:03PM +0200, Roman Zippel wrote:
> Hi,
>
> On Thu, 27 Jul 2006, Sam Ravnborg wrote:
>
> > Dedided to take another stamp on an old TODO item of making lxdialog
> > a built-in. Following patch is first step to do so.
> > The patch makes it a built-in - but with two open issues that I yet
> > have to address.
>
> Looks good. :)
> There is a NULL pointer problem with empty menus, item_cur is NULL and a
> select or exit will cause a segfault in item_set_selected().
Fixed. Introduced a dummy variable "item_nil" that item_cur points to in
the empty case. So no need for special cases all over.
>
> > I will during the weekend try to address the resize issue.
>
> Wasn't it working at some point?
> Anyway, it doesn't has to be overly complex either, e.g. if you delay it
> to the next key event, it's fine too. The signal handler would just set a
> flag and when wgetch returns, the display is reinitialized.
My experiments so far tells me that a resize generates KEY_RESIZE so it
is a simple matter of handling KEY_RESIZE correct in the different
dialog_* functions - and no signal handler needed. Thats looks like the
approach taken in the dialog package too.
>
> > The double ESC ESC thing I dunno how to fix.
>
> I think the easiest would be to just ignore the first ESC, it matches the
> documented behaviour and e.g. mc has the same behaviour. The delay of the
> single ESC makes it a bit annoying/confusing to use, so that sticking to
> the double ESC is IMO safer.
> I played with it a little and below is an example, which implements this
> behaviour for the menu window.
Thanks - will implment this for all the dialog_* functions.
Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
