Re: shell32: remove unneeded parsing of SHELLEXECUTEINFO.lpFile

[Paul Vriens]

On 03/17/2010 04:21 PM, Ilya Basin wrote:
> @@ -1013,7 +1011,7 @@ static void test_filename(void)
>                  "%s failed: rc=%d err=%d\n", shell_call,
>                  rc, GetLastError());
>           }
> -
>    else todo_wine
> -
>    else
>     {
>         ok(rc==test->rc, "%s failed: rc=%d err=%d\n", shell_call,
>            rc, GetLastError());

Hi Ilya,

If this is no longer a todo_wine you should change the filename_tests 
struct:

/* Test file masked due to space */
     {NULL,           "%s\masked file.shlexec",   0x1, 33},

to

/* Test file masked due to space */
     {NULL,           "%s\masked file.shlexec",   0x0, 33},

You can even consider removing the "if ((test->todo & 0x1)==0)" as now 
there are no more of those tests.

On the one hand it's nice to have these todo constructs, removing them 
however is a greater incentive to get the implementation right ;)