On 04/24/2010 10:53 AM, Ilya Basin wrote:
- 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.
- 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.
Hi Ilya,
In your last email on wine-devel about this subject you mentioned adding some extra tests. There was at least some communication with you and the author of that piece you are now removing. The reason for that piece was shown in a code-snippet he gave and adding tests for that specific case would be worthwhile I think.
PV> Hi Ilya,
PV> In your last email on wine-devel about this subject you mentioned adding PV> some extra tests. There was at least some communication with you and the PV> author of that piece you are now removing. The reason for that piece was PV> shown in a code-snippet he gave and adding tests for that specific case PV> would be worthwhile I think.
Hi Paul. Yore're right, I forgot. IB> Add a new verb that has '%*' in it. I put the reply from M.Fuchs in IB> the bottom. It seems that it was related to .lnk handling. Then postponing till my next slack period, the more so I got another e-mail:
=== W7PROX64 (32 bit shlexec) === No test failures found
=== W7PROX64 (64 bit shlexec) ===
shell32: shlexec.c:1609: Test failed: ShellExecuteEx(null, "C:\Users\winetest\AppData\Local\Temp\wt1F81.tmp\test file.sde", null, null) failed: rc=29 err=2 shlexec.c:1609: Test failed: ShellExecuteEx(null, "C:\Users\winetest\AppData\Local\Temp\wt1F81.tmp\test file.sde", null, null) failed: rc=29 err=2 shlexec.c:1609: Test failed: ShellExecuteEx(null, "C:\Users\winetest\AppData\Local\Temp\wt1F81.tmp\test file.sde", null, null) failed: rc=29 err=2 shlexec.c:1609: Test failed: ShellExecuteEx(null, "C:\Users\winetest\AppData\Local\Temp\wt1F81.tmp\test file.sde", null, null) failed: rc=29 err=2 shlexec.c:1609: Test failed: ShellExecuteEx(null, "C:\Users\winetest\AppData\Local\Temp\wt1F81.tmp\test file.sde", null, null) failed: rc=29 err=2
Strange, isn't it? I wonder if the tests succeeded without my patch.
I was wrong in the following part: IB> But currently I'm more worried by IB> http://bugs.winehq.org/show_bug.cgi?id=19385 IB> ( the 'wine start' launcher does not open MS Office documents that IB> have spaces in their path )
IB> If the <verb>\command doesn't contain '%', e.g. IB> '"C:\PROGRA~1\MICROS~2\OFFICE11\WINWORD.EXE" /n /dde' IB> it's argyfied incorrectly
IB> Now there's no test verb without '%', I think I should add it IB> and add related tests.
Later I reviewed the code and learned that this was unrelated to the bug with lpFile parsing. I sent another patch, so the bug 19385 is fixed now. We can still add more DDE tests, but, I repeat, it's not related to the current topic.
-
Ilya Basin -
Paul Vriens