Re: Setupapi: add stubs [resend]

[Alexandre Julliard]

Julien  writes:
> +/***********************************************************************
> -
> -
> SetupDiDestroyClassImageList(SETUPAPI.@)
> - */
>
> +BOOL WINAPI SetupDiDestroyClassImageList(PSP_CLASSIMAGELIST_DATA ClassImageListData)
> +{
> - FIXME ("Stub %p\n", ClassImageListData);
> - SetLastError(ERROR_CALL_NOT_IMPLEMENTED);
> - return TRUE;
>
> +}
> -
>
> +/***********************************************************************
> -
> -
> SetupDiGetClassImageList(SETUPAPI.@)
> - */
>
> +BOOL WINAPI SetupDiGetClassImageList(PSP_CLASSIMAGELIST_DATA ClassImageListData)
> +{
> - FIXME ("Stub %p\n", ClassImageListData);
> - SetLastError(ERROR_CALL_NOT_IMPLEMENTED);
> - return TRUE;
>
> +}

It makes no sense to set last error if you return success. And you
really shouldn't return success unless you also store the right things
in that data structure.