From: Dmitry Timoshkov dmitry@baikal.ru
Based on OLEPictureImpl_Save implementation.
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=8532 Signed-off-by: Dmitry Timoshkov dmitry@baikal.ru --- dlls/oleaut32/olepicture.c | 92 +++++++++++++++++++++++++++----- dlls/oleaut32/tests/olepicture.c | 14 +---- 2 files changed, 80 insertions(+), 26 deletions(-)
diff --git a/dlls/oleaut32/olepicture.c b/dlls/oleaut32/olepicture.c index 5b3783f1de9..0ea65cf5430 100644 --- a/dlls/oleaut32/olepicture.c +++ b/dlls/oleaut32/olepicture.c @@ -830,19 +830,6 @@ static HRESULT WINAPI OLEPictureImpl_PictureChanged(IPicture *iface) return S_OK; }
-/************************************************************************ - * OLEPictureImpl_SaveAsFile - */ -static HRESULT WINAPI OLEPictureImpl_SaveAsFile(IPicture *iface, - IStream *pstream, - BOOL SaveMemCopy, - LONG *pcbSize) -{ - OLEPictureImpl *This = impl_from_IPicture(iface); - FIXME("(%p)->(%p, %d, %p), hacked stub.\n", This, pstream, SaveMemCopy, pcbSize); - return IStream_Write(pstream,This->data,This->datalen,(ULONG*)pcbSize); -} - /************************************************************************ * OLEPictureImpl_get_Attributes */ @@ -1952,6 +1939,85 @@ static HRESULT WINAPI OLEPictureImpl_GetSizeMax(IPersistStream *iface, ULARGE_IN return hr; }
+/************************************************************************ + * OLEPictureImpl_SaveAsFile + */ +static HRESULT WINAPI OLEPictureImpl_SaveAsFile(IPicture *iface, + IStream *stream, BOOL mem_copy, LONG *size) +{ + OLEPictureImpl *This = impl_from_IPicture(iface); + void *data; + unsigned int data_size; + ULONG written; + HRESULT hr; + + FIXME("(%p)->(%p,%d,%p): semi-stub\n", This, stream, mem_copy, size); + + switch (This->desc.picType) + { + case PICTYPE_NONE: + return S_OK; + + case PICTYPE_ICON: + if (!mem_copy) return E_FAIL; + + if (This->bIsDirty || !This->data) + { + if (!serializeIcon(This->desc.icon.hicon, &data, &data_size)) + return E_FAIL; + HeapFree(GetProcessHeap(), 0, This->data); + This->data = data; + This->datalen = data_size; + } + hr = IStream_Write(stream, This->data, This->datalen, &written); + if (hr == S_OK && size) *size = written; + return hr; + + case PICTYPE_BITMAP: + if (!mem_copy) return E_FAIL; + + if (This->bIsDirty || !This->data) + { + switch (This->keepOrigFormat ? This->loadtime_format : BITMAP_FORMAT_BMP) + { + case BITMAP_FORMAT_BMP: + hr = serializeBMP(This->desc.bmp.hbitmap, &data, &data_size); + if (hr != S_OK) return hr; + break; + case BITMAP_FORMAT_JPEG: + FIXME("BITMAP_FORMAT_JPEG is not implemented\n"); + return E_NOTIMPL; + case BITMAP_FORMAT_GIF: + FIXME("BITMAP_FORMAT_GIF is not implemented\n"); + return E_NOTIMPL; + case BITMAP_FORMAT_PNG: + FIXME("BITMAP_FORMAT_PNG is not implemented\n"); + return E_NOTIMPL; + default: + FIXME("PICTYPE_BITMAP/%#x is not implemented\n", This->loadtime_format); + return E_NOTIMPL; + } + + HeapFree(GetProcessHeap(), 0, This->data); + This->data = data; + This->datalen = data_size; + } + hr = IStream_Write(stream, This->data, This->datalen, &written); + if (hr == S_OK && size) *size = written; + return hr; + + case PICTYPE_METAFILE: + FIXME("PICTYPE_METAFILE is not implemented\n"); + return E_NOTIMPL; + case PICTYPE_ENHMETAFILE: + FIXME("ENHMETAFILE is not implemented\n"); + return E_NOTIMPL; + default: + FIXME("%#x is not implemented\n", This->desc.picType); + break; + } + return E_NOTIMPL; +}
/************************************************************************ * IDispatch diff --git a/dlls/oleaut32/tests/olepicture.c b/dlls/oleaut32/tests/olepicture.c index 2b2c84abe38..2f909503462 100644 --- a/dlls/oleaut32/tests/olepicture.c +++ b/dlls/oleaut32/tests/olepicture.c @@ -1240,18 +1240,14 @@ static void test_load_save_bmp(void) size = -1; hr = IPicture_SaveAsFile(pic, dst_stream, TRUE, &size); ok(hr == S_OK, "IPicture_SaveasFile error %#lx\n", hr); - todo_wine ok(size == 66, "expected 66, got %ld\n", size); mem = GlobalLock(hmem); - todo_wine ok(!memcmp(&mem[0], "BM", 2), "got wrong bmp header %04lx\n", mem[0]); GlobalUnlock(hmem);
size = -1; hr = IPicture_SaveAsFile(pic, dst_stream, FALSE, &size); - todo_wine ok(hr == E_FAIL, "expected E_FAIL, got %#lx\n", hr); - todo_wine ok(size == -1, "expected -1, got %ld\n", size);
offset.QuadPart = 0; @@ -1329,15 +1325,12 @@ static void test_load_save_icon(void) todo_wine ok(size == 766, "expected 766, got %ld\n", size); mem = GlobalLock(hmem); - todo_wine ok(mem[0] == 0x00010000, "got wrong icon header %04lx\n", mem[0]); GlobalUnlock(hmem);
size = -1; hr = IPicture_SaveAsFile(pic, dst_stream, FALSE, &size); - todo_wine ok(hr == E_FAIL, "expected E_FAIL, got %#lx\n", hr); - todo_wine ok(size == -1, "expected -1, got %ld\n", size);
offset.QuadPart = 0; @@ -1418,13 +1411,11 @@ static void test_load_save_empty_picture(void) size = -1; hr = IPicture_SaveAsFile(pic, dst_stream, TRUE, &size); ok(hr == S_OK, "IPicture_SaveasFile error %#lx\n", hr); - todo_wine ok(size == -1, "expected -1, got %ld\n", size);
size = -1; hr = IPicture_SaveAsFile(pic, dst_stream, FALSE, &size); ok(hr == S_OK, "IPicture_SaveasFile error %#lx\n", hr); - todo_wine ok(size == -1, "expected -1, got %ld\n", size);
hr = IPicture_QueryInterface(pic, &IID_IPersistStream, (void **)&src_stream); @@ -1564,7 +1555,6 @@ static void test_load_save_dib(void) size = -1; hr = IPicture_SaveAsFile(pic, dst_stream, TRUE, &size); ok(hr == S_OK, "IPicture_SaveasFile error %#lx\n", hr); - todo_wine ok(size == expected_size, "expected %ld, got %ld\n", expected_size, size); if (size == expected_size) { mem = GlobalLock(hmem); @@ -1577,9 +1567,7 @@ static void test_load_save_dib(void)
size = -1; hr = IPicture_SaveAsFile(pic, dst_stream, FALSE, &size); - todo_wine ok(hr == E_FAIL, "expected E_FAIL, got %#lx\n", hr); - todo_wine ok(size == -1, "expected -1, got %ld\n", size);
offset.QuadPart = 0; @@ -1663,6 +1651,7 @@ static void test_load_save_emf(void)
size = -1; hr = IPicture_SaveAsFile(pic, dst_stream, TRUE, &size); + todo_wine ok(hr == S_OK, "IPicture_SaveasFile error %#lx\n", hr); todo_wine ok(size == 128, "expected 128, got %ld\n", size); @@ -1678,7 +1667,6 @@ static void test_load_save_emf(void) hr = IPicture_SaveAsFile(pic, dst_stream, FALSE, &size); todo_wine ok(hr == E_FAIL, "expected E_FAIL, got %#lx\n", hr); - todo_wine ok(size == -1, "expected -1, got %ld\n", size);
offset.QuadPart = 0;