[PATCH 1/5, try2] shell32/tests: Add tests for ShellExecute DDE behavior.

Misha Koshelev mk144210 at bcm.edu
Sun May 27 20:00:52 CDT 2007


Changes: Sorry I think my previous versions were not quite ready for prime time yet (specifically old patch #3
for default DDE app names).
	Patch 1. Add a wait for child thread so that GetExitCode thread does not return STILL_ACTIVE (259)
	         which is incorrectly taken to be a successful execution sometimes.
        Patch 2. No changes.
        Patch 3 (NEW). Comprehensive conformance test for default DDE application name.
        Patch 4 (old 3). Use SearchPathW return value rather than filename pointer to track success as it
                 is more reliable, and don't forget to search the whole start command in SearchPathW if we
                 don't find anything smaller to match.
        Patch 5 (NEW). Just condenses create_test_verb into a call to create_test_verb_dde with NULL for the
	         specific DDE parameters.

As a reminder, these patches fix DDE execution logic to conform. Previously, DDE was only used if there
is no command key, which is not native behavior and does not work for some native apps like Vector NTI
that actually expect the filename as a DDE string as they requested and not on the command line. And add
support for default DDE application names.
---
 dlls/shell32/tests/shlexec.c |  282 ++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 282 insertions(+), 0 deletions(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 34b60bce9e81c2bea58cdd93771e129b9efe4447.diff
Type: text/x-patch
Size: 11070 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20070527/9577d060/34b60bce9e81c2bea58cdd93771e129b9efe4447.bin


More information about the wine-patches mailing list