On 4/24/2010 12:53, Ilya Basin wrote:
1) removes the block of code that splits lpFile into filename and
parameters and removes '"'s from filename . Even without this block
ShellExecute not fails with quoted lpFile.
2) removes 2 corresponding todo_wine's from tests.
The real application affected by this bug is uTorrent: if you have 2
downloads containing, say,
"foo/bar.avi" and "foo 2/baz.avi", when you double click the latter
avi file, it'll open the explorer in folder "foo" instead.
try 2:
disabled todo_wine for "%s\\masked file.shlexec"
instead of disabling all todo_wine's in the loop
try 3:
try 2 didn't apply cleanly in git: ok() format string changed.
---
dlls/shell32/shlexec.c | 62 +-----------------------------------------
dlls/shell32/tests/shlexec.c | 16 +++++------
2 files changed, 8 insertions(+), 70 deletions(-)
diff --git a/dlls/shell32/shlexec.c b/dlls/shell32/shlexec.c
index a81cada..b89bb08 100644
--- a/dlls/shell32/shlexec.c
+++ b/dlls/shell32/shlexec.c
@@ -1678,67 +1678,7 @@ static BOOL SHELL_execute( LPSHELLEXECUTEINFOW sei,
SHELL_ExecuteW32 execfunc )
/* Else, try to execute the filename */
TRACE("execute:%s,%s,%s\n", debugstr_w(wszApplicationName),
debugstr_w(wszParameters), debugstr_w(wszDir));
- /* separate out command line arguments from executable file name */
Where are tests for this separation path? You know, it's suspicious to
just remove it to solve for not-clearly related problem to this code block.