[Date Prev][Date Next][Thread Prev][Thread Next][Thread Index]
RE: [XaraXtreme-dev] Problem in wxWidgets 2.6.3...
- From: "Neil Howe" <NeilH@xxxxxxxx>
- Date: Fri, 19 May 2006 14:32:04 +0100
- Subject: RE: [XaraXtreme-dev] Problem in wxWidgets 2.6.3...
> I wrote re dragmgr.cpp:
> > Firstly, the ScreenToClient() is wrong - the call takes and has
> > always taken screen coords not client coords. If this was working,
> > it was only by accident (perhaps due to the other bug).
> >
> > Secondly, ChildWindowUnderPoint == WindowUnderPoint seems
unnecessary.
> > The routine has returned that a child window of the target window
> > exists which is the deepest window that contains the mouse pointer.
> > So it's a drag hit. Why do we care whether or not the child window
> > is also the one we found before?
>
> On reflection, I think the second check /is/ actually correct, and is
> designed to ensure that there is no window which is NOT a child of
> the target window which obscures the target window or its child.
>
> I can only assume the first ScreenToClient() was necessary to work
> around the bug we've fixed (in which case removal is correct) or
> that the routine never worked properly (ditto). In any case I've
> checked it in on this basis and fixed dlgmgr, camview & ccolbar too.
>
> It all seems to work.
>
> We should now not need the wxWidgets patch unless wxWidgets itself
> is using the call in a broken way. My only worry is whether it
> internally relies on the broken behaviour (I hope not).
Luke, I think you said you'd applied the patch to the build server so
can you remove it please and then confirm.
Thanks
Neil