Dmitry Timoshkov wrote:
"Jeff Latimer" lats@yless4u.com.au 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 doesn't say anything, and using meaningful symbol names is always preferred.
Ok, it will be consistent.
Also IMO a more appropriate place to set ack to DDE_FNOTPROCESSED is WDML_HandleReply, so that all types of replies get covered.
I thought this as well when I was hunting for a place for setting ack. It seems that handle routines all do something different. Some don't set the ack/res and this causes tests to fail (ie. deadbeef passes through some of them). I opted to place the setting of ack in the routine that applies to this test.
Although there is no tests for all DdeClientTransaction cases, it should be pretty safe to assume that all request types behave similar in returning handling state.
That is not how it looked when I set ack to DMLERR_NO_ERROR at the start of the WDML_HandleReply. Tests started fail != 0xdeadbeef in Windows. I would prefer to leave the test where I placed it for that reason.