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

[Dmitry Timoshkov]

Piotr Caban <piotr(a)codeweavers.com> 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.

-- 
Dmitry.

[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 <piotr(a)codeweavers.com> 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