[1/5] windowscodecs: Implement IWICBitmapScaler::GetSize.

Vincent Povirk madewokherd at gmail.com
Wed Jun 27 11:29:11 CDT 2012


Good point. I'll resend with parameter checking.

On Wed, Jun 27, 2012 at 11:25 AM, Dmitry Timoshkov <dmitry at baikal.ru> wrote:
> Vincent Povirk <madewokherd at gmail.com> wrote:
>
>>  static HRESULT WINAPI BitmapScaler_GetSize(IWICBitmapScaler *iface,
>>      UINT *puiWidth, UINT *puiHeight)
>>  {
>> -    FIXME("(%p,%p,%p): stub\n", iface, puiWidth, puiHeight);
>> +    BitmapScaler *This = impl_from_IWICBitmapScaler(iface);
>> +    TRACE("(%p,%p,%p)\n", iface, puiWidth, puiHeight);
>>
>> -    return E_NOTIMPL;
>> +    if (!This->source)
>> +        return WINCODEC_ERR_WRONGSTATE;
>> +
>> +    *puiWidth = This->width;
>> +    *puiHeight = This->height;
>> +
>> +    return S_OK;
>
> This patch and remaning patches in the sequence completely miss any
> parameter checking.
>
> --
> Dmitry.



More information about the wine-devel mailing list