[5/5] windowscodecs: Implement JpegEncoder_Commit.

Vincent Povirk madewokherd at gmail.com
Mon Apr 16 11:01:48 CDT 2012


-------------- next part --------------
From f55ec3e7cd1ca1dc6580805b7229fb5b2b6c8e68 Mon Sep 17 00:00:00 2001
From: Vincent Povirk <vincent at codeweavers.com>
Date: Wed, 11 Apr 2012 17:15:48 -0500
Subject: [PATCH 5/8] windowscodecs: Implement JpegEncoder_Commit.

---
 dlls/windowscodecs/jpegformat.c |   20 ++++++++++++++++++--
 1 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/dlls/windowscodecs/jpegformat.c b/dlls/windowscodecs/jpegformat.c
index cd453cf..3a9b095 100644
--- a/dlls/windowscodecs/jpegformat.c
+++ b/dlls/windowscodecs/jpegformat.c
@@ -737,6 +737,7 @@ typedef struct JpegEncoder {
     int started_compress;
     int lines_written;
     int frame_committed;
+    int committed;
     UINT width, height;
     double xres, yres;
     const jpeg_compress_format *format;
@@ -1306,8 +1307,22 @@ static HRESULT WINAPI JpegEncoder_CreateNewFrame(IWICBitmapEncoder *iface,
 
 static HRESULT WINAPI JpegEncoder_Commit(IWICBitmapEncoder *iface)
 {
-    FIXME("(%p): stub\n", iface);
-    return E_NOTIMPL;
+    JpegEncoder *This = impl_from_IWICBitmapEncoder(iface);
+    TRACE("(%p)\n", iface);
+
+    EnterCriticalSection(&This->lock);
+
+    if (!This->frame_committed || This->committed)
+    {
+        LeaveCriticalSection(&This->lock);
+        return WINCODEC_ERR_WRONGSTATE;
+    }
+
+    This->committed = TRUE;
+
+    LeaveCriticalSection(&This->lock);
+
+    return S_OK;
 }
 
 static HRESULT WINAPI JpegEncoder_GetMetadataQueryWriter(IWICBitmapEncoder *iface,
@@ -1362,6 +1377,7 @@ HRESULT JpegEncoder_CreateInstance(IUnknown *pUnkOuter, REFIID iid, void** ppv)
     This->started_compress = 0;
     This->lines_written = 0;
     This->frame_committed = 0;
+    This->committed = 0;
     This->width = This->height = 0;
     This->xres = This->yres = 0.0;
     This->format = NULL;
-- 
1.7.9.1


More information about the wine-patches mailing list