usp10: fix tests to allow atomic patches for new implementation
James Hawkins
truiken at gmail.com
Fri Dec 15 01:51:46 CST 2006
On 12/15/06, Clinton Stimpson <cjstimpson at utwire.net> wrote:
> Hi,
>
> Here's a patch to make unimplemented functions return E_NOTIMPL instead
> of S_OK. Even dependent functions already return E_NOTIMPL. Also
> fixing tests to check that the return value is S_OK before testing the
> 'out' parameters.
> This fix allows piecemeal patches to follow with implementation for each
> function in a separate patch.
>
> Thanks,
> Clinton Stimpson
>
> ChangeLog
> Return E_NOTIMPL where appropriate.
> And only check out parameters when appropriate.
>
There are a couple things wrong with this patch. First, you've made
it so the tests won't be run for these functions (because they don't
return S_OK as it stands). Second, there are probably applications
that depend on these functions returning S_OK. You're really making
it a lot harder than it should be. Implement each function one at a
time. If tests start to succeed, just remove the todo_wine from them.
So the implemented function and removed/added go in one patch. You
just do that for each function you're implementing.
--
James Hawkins
More information about the wine-devel
mailing list