Re: user32: DdeClientTransaction should return 0 in the last param forDDE_DATA

2008-08-07 Thread Jeff Latimer
Dmitry Timoshkov wrote: > "Jeff Latimer" <[EMAIL PROTECTED]> wrote: > >> MSDN does not show DdeCientTransaction as returning in res >> DDE_FNOTPROCESSED. Is this the standard test to see if it is zero? >> Otherwise an actual test for 0 would be just as good. > > Existing tests already check for

Re: user32: DdeClientTransaction should return 0 in the last param forDDE_DATA

2008-08-07 Thread Dmitry Timoshkov
"Jeff Latimer" <[EMAIL PROTECTED]> wrote: > MSDN does not show DdeCientTransaction as returning in res > DDE_FNOTPROCESSED. Is this the standard test to see if it is zero? > Otherwise an actual test for 0 would be just as good. Existing tests already check for DDE_FNOTPROCESSED. Using just 0

Re: user32: DdeClientTransaction should return 0 in the last param forDDE_DATA

2008-08-07 Thread Jeff Latimer
Dmitry Timoshkov wrote: > "Jeff Latimer" <[EMAIL PROTECTED]> wrote: > >> >> + if (ack) *ack = ERROR_SUCCESS; >> >> >> ok(ret == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", ret); >> -todo_wine >> -{ >> -ok(res == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got >> %08x\n", r

Re: user32: DdeClientTransaction should return 0 in the last param forDDE_DATA

2008-08-05 Thread Dmitry Timoshkov
"Jeff Latimer" <[EMAIL PROTECTED]> wrote: > --- a/dlls/user32/dde_client.c > +++ b/dlls/user32/dde_client.c > @@ -520,6 +520,7 @@ static WDML_QUEUE_STATE > WDML_HandleRequestReply(WDML_CONV* pConv, MSG* msg, WDML > case WM_DDE_DATA: > UnpackDDElParam(WM_DDE_DATA, msg->lParam, &uiLo, &