cabinet: Revert "cabinet: Fix for FDICopy with an empty cabinet file."

Reece Dunn msclrhd at googlemail.com
Sat Apr 26 01:03:53 CDT 2008


2008/4/26 James Hawkins <jhawkins at codeweavers.com>:
> Hi,
>
>  The test was incorrect and failed on all platforms.  Vitaly, please
>  ask someone to test your patches on a real windows system in the
>  future if you don't have access to one.

Wouldn't it be better to actually fix the test case than revert it all together?

Just simply change line 630:

-    ok(ret, "Expected FDICopy to succeed\n");
+    ok(!ret, "Expected FDICopy to fail\n");

and remove the todo_wine. Adding a GetLastError check would be good as well.

- Reece



More information about the wine-devel mailing list