Dmitry Timoshkov wrote:
First of all, Eric, thank you very much for reviewing these patches, it's very important to do the right thing in such a fragile area as DDE.
as DDE protocole is a nightmare (from an implementation point of view)... but other fixes from your initial are more than welcomed too !
It's fResponse and fRelease which were changed.
Since we handle WM_DDE_REQUEST here, we must set fResponse to TRUE according to MSDN. And I retested again under both win2k and Wine, and fRelease change can be reverted. Setting it to FALSE doesn't effect my applications.
you're right here.
Is the attached patch acceptable for you?
sounds fine yes. thanks
A+