Re: Handling dialog messages for non-template based dialogs

2004-04-16 Thread Dmitry Timoshkov
"Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > Could you take a look at this fix? I would think I have resolved the other > issue with SetDefButton as well. Tests with msg ran fine. The only thing that I'd suggest to remove is a misleading comment about WM_NCCREATE, otherwise the patch looks

RE: Handling dialog messages for non-template based dialogs

2004-04-16 Thread Santosh Siddheshwar
04 1:35 PM > To: Santosh Siddheshwar; Alexandre Julliard > Cc: Robert Shearman; [EMAIL PROTECTED] > Subject: Re: Handling dialog messages for non-template based dialogs > > "Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > > > This test fails with WINE even

Re: Handling dialog messages for non-template based dialogs

2004-04-16 Thread Dmitry Timoshkov
"Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > This test fails with WINE even after I create the DIALOGINFO structure the > first time it enters DefDlgProc. Not sure why this test should work on > Windows. Since I wrote it under Windows it apparently works there. > Basically at the stage whe

RE: Handling dialog messages for non-template based dialogs

2004-04-15 Thread Santosh Siddheshwar
Siddheshwar; Alexandre Julliard > Cc: Robert Shearman; [EMAIL PROTECTED] > Subject: Re: Handling dialog messages for non-template based dialogs > > "Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > > > OK.One place where this could be done is WM_NCCRE

RE: Handling dialog messages for non-template based dialogs

2004-04-15 Thread Santosh Siddheshwar
> Sent: Thursday, April 15, 2004 3:56 PM > To: Santosh Siddheshwar; Alexandre Julliard > Cc: Robert Shearman; [EMAIL PROTECTED] > Subject: Re: Handling dialog messages for non-template based dialogs > > "Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > &

Re: Handling dialog messages for non-template based dialogs

2004-04-15 Thread Dmitry Timoshkov
"Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > OK.One place where this could be done is WM_NCCREATE which comes before > WM_CREATE. Other place (possibly better) where it might be possible to do > this is in CBTProc hook procedure. Not sure about the second option though. > Any thoughts? I'll

RE: Handling dialog messages for non-template based dialogs

2004-04-14 Thread Santosh Siddheshwar
age- > From: Alexandre Julliard [SMTP:[EMAIL PROTECTED] > Sent: Thursday, April 15, 2004 9:43 AM > To: Santosh Siddheshwar > Cc: Dmitry Timoshkov; Robert Shearman; [EMAIL PROTECTED] > Subject: Re: Handling dialog messages for non-template based dialogs > > Santosh Siddh

Re: Handling dialog messages for non-template based dialogs

2004-04-14 Thread Alexandre Julliard
Santosh Siddheshwar <[EMAIL PROTECTED]> writes: > But dont you think that the structure is not really required till the > window has been created successfully? Well, I don't think so, but the only way to know for sure is to write a test program under Windows to determine the exact behavior. --

RE: Handling dialog messages for non-template based dialogs

2004-04-14 Thread Santosh Siddheshwar
moshkov > Cc: Santosh Siddheshwar; Robert Shearman; [EMAIL PROTECTED] > Subject: Re: Handling dialog messages for non-template based dialogs > > "Dmitry Timoshkov" <[EMAIL PROTECTED]> writes: > > > Yes, if Alexandre has no objections. Be patient, that

Re: Handling dialog messages for non-template based dialogs

2004-04-14 Thread Alexandre Julliard
"Dmitry Timoshkov" <[EMAIL PROTECTED]> writes: > Yes, if Alexandre has no objections. Be patient, that may take some time. Actually, I do have an objection... I think you should be allocating the structure as soon as it is needed, WM_CREATE is not the first message that the window will receive.

Re: Handling dialog messages for non-template based dialogs

2004-04-14 Thread Dmitry Timoshkov
"Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > So can I expect this to be checked into cvs soon? Yes, if Alexandre has no objections. Be patient, that may take some time. -- Dmitry.

RE: Handling dialog messages for non-template based dialogs

2004-04-14 Thread Santosh Siddheshwar
; Subject: Re: Handling dialog messages for non-template based dialogs > > "Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > > > Thanks for the feedback. Excellent catches! Hope my next mail to > > wine-patches gets through! > > Your new patch loo

Re: Handling dialog messages for non-template based dialogs

2004-04-14 Thread Dmitry Timoshkov
"Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > Thanks for the feedback. Excellent catches! Hope my next mail to > wine-patches gets through! Your new patch looks good for me. Thanks for doing this work! -- Dmitry.

RE: Handling dialog messages for non-template based dialogs

2004-04-14 Thread Santosh Siddheshwar
rman; [EMAIL PROTECTED] > Cc: [EMAIL PROTECTED] > Subject: Re: Handling dialog messages for non-template based dialogs > > "Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > > > I didnt get the second point that you mention > > Another one might be

Re: Handling dialog messages for non-template based dialogs

2004-04-13 Thread Dmitry Timoshkov
"Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > I didnt get the second point that you mention > Another one might be that WM_CREATE must be always > passed to DEFDLG_Proc regardless of what result an application > returned to DefDlgProc16/A/W. Here is a snippet of DefDlgProc16 (DefDlgProcA/W

RE: Handling dialog messages for non-template based dialogs

2004-04-13 Thread Santosh Siddheshwar
Dmitry Timoshkov [SMTP:[EMAIL PROTECTED] > Sent: Tuesday, April 13, 2004 5:46 PM > To: Robert Shearman; [EMAIL PROTECTED] > Cc: [EMAIL PROTECTED] > Subject: Re: Handling dialog messages for non-template based dialogs > > "Robert Shearman" <[EMAIL PROTECTED]> wr

Re: Handling dialog messages for non-template based dialogs

2004-04-13 Thread Dmitry Timoshkov
"Robert Shearman" <[EMAIL PROTECTED]> wrote: > > +if (!(dlgInfo = HeapAlloc( GetProcessHeap(), 0, > sizeof(*dlgInfo) ))) return 0; > > Minor point, but you probably want to return -1 here to say that window > creation failed. Good catch. Another one might be that WM_CREATE must be al

RE: Handling dialog messages for non-template based dialogs

2004-04-13 Thread Robert Shearman
Santosh Siddheshwar wrote: > > Change Log: > windows/defdlg.c - Added code for allocating DIALOGINFO structure and > storing it as an attribute of the window > windows/dialog.c - Removed code for allocating DIALOGINFO structure and > moved initialization of DIALOGINFO structure > to after CreateWin

RE: Handling dialog messages for non-template based dialogs

2004-04-12 Thread Santosh Siddheshwar
e: Handling dialog messages for non-template based dialogs > > "Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > > > Change Log: > > windows/defdlg.c - Added code for allocating DIALOGINFO structure and > > storing it as an attribute of the window

Re: Handling dialog messages for non-template based dialogs

2004-04-12 Thread Dmitry Timoshkov
"Santosh Siddheshwar" <[EMAIL PROTECTED]> wrote: > Change Log: > windows/defdlg.c - Added code for allocating DIALOGINFO structure and > storing it as an attribute of the window > windows/dialog.c - Removed code for allocating DIALOGINFO structure and > moved initialization of DIALOGINFO structure