Julien <[EMAIL PROTECTED]> 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.

-- 
Alexandre Julliard
[EMAIL PROTECTED]


Reply via email to