gdiplus: Reuse an existing image stream in GdipImageSelectActiveFrame.
Vincent Povirk
madewokherd at gmail.com
Thu Sep 27 09:52:07 CDT 2012
> @@ -3898,7 +3894,7 @@ GpStatus WINGDIPAPI GdipImageSelectActiveFrame(GpImage *image, GDIPCONST GUID *d
> return Ok;
> }
>
> - IStream_Release(stream);
> + IStream_Release(image->stream);
> return stat;
> }
I don't think it's necessary to AddRef/Release the stream anymore in
GdipImageSelectActiveFrame. Creating the new image and freeing the old
one will update the refcount as necessary. And it's a bit confusing
since technically the current image->stream is released by
free_image_data then overwritten (with the same value, fortunately) by
copying new_image to image.
Looks good otherwise.
More information about the wine-devel
mailing list