Re: [PATCH]: menu doesn't appear on right info panel
- From: Pavel Tsekov <ptsekov gmx net>
- To: Leonard den Ottolander <leonard den ottolander nl>
- Cc: MC Devel <mc-devel gnome org>
- Subject: Re: [PATCH]: menu doesn't appear on right info panel
- Date: Tue, 2 Nov 2004 12:13:27 +0100
Hello Leonard,
On Tue, 2 Nov 2004, Leonard den Ottolander wrote:
> On Tue, 2004-11-02 at 11:51, Pavel Tsekov wrote:
> > Me and Pavel Shirshov both agree that this new patch is indeed better.
> > What I wrote in the mail describing the problem is based on me reading the
> > source code, but there aren't any comments or something else to proof
> > that I was right. The new code introduced by Pavel Rosking seemed to try
> > to avoid an endless loop too. I think the new code is clear, fool proof
> > and better.
>
> I don't say the new code is bad, I only wonder why you haven't checked
> the rest of the code to make sure whether your original statement about
> at least one item accepting WIDGET_CODE is correct. If that statement is
It would be a hard task to proof it - really. But I tell you - it worked
that way for quite some time :)
> true you have introduced a redundant check for a situation that will
> never occur.
No - we don't know what kind of a dialog would be introduced in the
future. And it is not indicated clearly that a dialog should have
at least one widget wich accepts focus.
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]