Re: [PATCH 4/7] gdiplus: Don't store stream reference in GpImage, it's done by IWICBitmapDecoder

[Dmitry Timoshkov]

Piotr Caban  wrote:
> @@ -3808,12 +3816,6 @@ GpStatus WINGDIPAPI GdipImageSelectActiveFrame(GpImage *image, GDIPCONST GUID *d
>      if (image->current_frame == frame)
>          return Ok;
> - if (!image->stream)
> - {
> -
>    TRACE("image doesn't have an associated stream\n");
> -
>    return Ok;
> - }

It doesn't make much sence to add this piece of code in previous patch.

[PATCH 4/7] gdiplus: Don't store stream reference in GpImage, it's done by IWICBitmapDecoder

[Piotr Caban]

On 03/12/15 05:08, Dmitry Timoshkov wrote:
> Piotr Caban  wrote:
> > @@ -3808,12 +3816,6 @@ GpStatus WINGDIPAPI GdipImageSelectActiveFrame(GpImage *image, GDIPCONST GUID *d
> >       if (image->current_frame == frame)
> >           return Ok;
> > - if (!image->stream)
> > - {
> > -
> >    TRACE("image doesn't have an associated stream\n");
> > -
> >    return Ok;
> > - }
>
> It doesn't make much sence to add this piece of code in previous patch.

I didn't add it, it already was there (I have added similar check for 
decoder).

Thanks,
Piotr