January 2011 - Wine-Commits

Michael Stefaniuc : windowscodecs/tests: COM cleanup for converter.c.
by Alexandre Julliard Jan. 12, 2011

Jan. 12, 2011

Module: wine Branch: master Commit: eaf394a9ac4cd2bdf1e6484a723abf5f85026104 URL: http://source.winehq.org/git/wine.git/?a=commit;h=eaf394a9ac4cd2bdf1e6484a7… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Jan 12 00:51:41 2011 +0100 windowscodecs/tests: COM cleanup for converter.c. --- dlls/windowscodecs/tests/converter.c | 76 ++++++++++++++++++---------------- 1 files changed, 40 insertions(+), 36 deletions(-) diff --git a/dlls/windowscodecs/tests/converter.c b/dlls/windowscodecs/tests/converter.c index 408a6ca..a720b4b 100644 --- a/dlls/windowscodecs/tests/converter.c +++ b/dlls/windowscodecs/tests/converter.c @@ -20,6 +20,7 @@ #include <math.h> #define COBJMACROS +#define CONST_VTABLE #include "windef.h" #include "objbase.h" @@ -37,11 +38,16 @@ typedef struct bitmap_data { } bitmap_data; typedef struct BitmapTestSrc { - const IWICBitmapSourceVtbl *lpVtbl; + IWICBitmapSource IWICBitmapSource_iface; LONG ref; const bitmap_data *data; } BitmapTestSrc; +static inline BitmapTestSrc *impl_from_IWICBitmapSource(IWICBitmapSource *iface) +{ + return CONTAINING_RECORD(iface, BitmapTestSrc, IWICBitmapSource_iface); +} + static HRESULT WINAPI BitmapTestSrc_QueryInterface(IWICBitmapSource *iface, REFIID iid, void **ppv) { @@ -59,14 +65,14 @@ static HRESULT WINAPI BitmapTestSrc_QueryInterface(IWICBitmapSource *iface, REFI static ULONG WINAPI BitmapTestSrc_AddRef(IWICBitmapSource *iface) { - BitmapTestSrc *This = (BitmapTestSrc*)iface; + BitmapTestSrc *This = impl_from_IWICBitmapSource(iface); ULONG ref = InterlockedIncrement(&This->ref); return ref; } static ULONG WINAPI BitmapTestSrc_Release(IWICBitmapSource *iface) { - BitmapTestSrc *This = (BitmapTestSrc*)iface; + BitmapTestSrc *This = impl_from_IWICBitmapSource(iface); ULONG ref = InterlockedDecrement(&This->ref); return ref; } @@ -74,7 +80,7 @@ static ULONG WINAPI BitmapTestSrc_Release(IWICBitmapSource *iface) static HRESULT WINAPI BitmapTestSrc_GetSize(IWICBitmapSource *iface, UINT *puiWidth, UINT *puiHeight) { - BitmapTestSrc *This = (BitmapTestSrc*)iface; + BitmapTestSrc *This = impl_from_IWICBitmapSource(iface); *puiWidth = This->data->width; *puiHeight = This->data->height; return S_OK; @@ -83,7 +89,7 @@ static HRESULT WINAPI BitmapTestSrc_GetSize(IWICBitmapSource *iface, static HRESULT WINAPI BitmapTestSrc_GetPixelFormat(IWICBitmapSource *iface, WICPixelFormatGUID *pPixelFormat) { - BitmapTestSrc *This = (BitmapTestSrc*)iface; + BitmapTestSrc *This = impl_from_IWICBitmapSource(iface); memcpy(pPixelFormat, This->data->format, sizeof(GUID)); return S_OK; } @@ -91,7 +97,7 @@ static HRESULT WINAPI BitmapTestSrc_GetPixelFormat(IWICBitmapSource *iface, static HRESULT WINAPI BitmapTestSrc_GetResolution(IWICBitmapSource *iface, double *pDpiX, double *pDpiY) { - BitmapTestSrc *This = (BitmapTestSrc*)iface; + BitmapTestSrc *This = impl_from_IWICBitmapSource(iface); *pDpiX = This->data->xres; *pDpiY = This->data->yres; return S_OK; @@ -106,7 +112,7 @@ static HRESULT WINAPI BitmapTestSrc_CopyPalette(IWICBitmapSource *iface, static HRESULT WINAPI BitmapTestSrc_CopyPixels(IWICBitmapSource *iface, const WICRect *prc, UINT cbStride, UINT cbBufferSize, BYTE *pbBuffer) { - BitmapTestSrc *This = (BitmapTestSrc*)iface; + BitmapTestSrc *This = impl_from_IWICBitmapSource(iface); UINT bytesperrow; UINT srcstride; UINT row_offset; @@ -171,26 +177,22 @@ static const IWICBitmapSourceVtbl BitmapTestSrc_Vtbl = { BitmapTestSrc_CopyPixels }; -static void CreateTestBitmap(const bitmap_data *data, IWICBitmapSource **bitmap) +static void CreateTestBitmap(const bitmap_data *data, BitmapTestSrc **This) { - BitmapTestSrc *This = HeapAlloc(GetProcessHeap(), 0, sizeof(BitmapTestSrc)); + *This = HeapAlloc(GetProcessHeap(), 0, sizeof(*This)); - if (This) + if (*This) { - This->lpVtbl = &BitmapTestSrc_Vtbl; - This->ref = 1; - This->data = data; - *bitmap = (IWICBitmapSource*)This; + (*This)->IWICBitmapSource_iface.lpVtbl = &BitmapTestSrc_Vtbl; + (*This)->ref = 1; + (*This)->data = data; } - else - *bitmap = NULL; } -static void DeleteTestBitmap(IWICBitmapSource *bitmap) +static void DeleteTestBitmap(BitmapTestSrc *This) { - BitmapTestSrc *This = (BitmapTestSrc*)bitmap; - ok(This->lpVtbl == &BitmapTestSrc_Vtbl, "test bitmap %p deleted with incorrect vtable\n", bitmap); - ok(This->ref == 1, "test bitmap %p deleted with %i references instead of 1\n", bitmap, This->ref); + ok(This->IWICBitmapSource_iface.lpVtbl == &BitmapTestSrc_Vtbl, "test bitmap %p deleted with incorrect vtable\n", This); + ok(This->ref == 1, "test bitmap %p deleted with %i references instead of 1\n", This, This->ref); HeapFree(GetProcessHeap(), 0, This); } @@ -289,12 +291,13 @@ static const struct bitmap_data testdata_32bppBGRA = { static void test_conversion(const struct bitmap_data *src, const struct bitmap_data *dst, const char *name, BOOL todo) { - IWICBitmapSource *src_bitmap, *dst_bitmap; + BitmapTestSrc *src_obj; + IWICBitmapSource *dst_bitmap; HRESULT hr; - CreateTestBitmap(src, &src_bitmap); + CreateTestBitmap(src, &src_obj); - hr = WICConvertBitmapSource(dst->format, src_bitmap, &dst_bitmap); + hr = WICConvertBitmapSource(dst->format, &src_obj->IWICBitmapSource_iface, &dst_bitmap); if (todo) todo_wine ok(SUCCEEDED(hr), "WICConvertBitmapSource(%s) failed, hr=%x\n", name, hr); else @@ -307,31 +310,32 @@ static void test_conversion(const struct bitmap_data *src, const struct bitmap_d IWICBitmapSource_Release(dst_bitmap); } - DeleteTestBitmap(src_bitmap); + DeleteTestBitmap(src_obj); } static void test_invalid_conversion(void) { - IWICBitmapSource *src_bitmap, *dst_bitmap; + BitmapTestSrc *src_obj; + IWICBitmapSource *dst_bitmap; HRESULT hr; - CreateTestBitmap(&testdata_32bppBGRA, &src_bitmap); + CreateTestBitmap(&testdata_32bppBGRA, &src_obj); /* convert to a non-pixel-format GUID */ - hr = WICConvertBitmapSource(&GUID_VendorMicrosoft, src_bitmap, &dst_bitmap); + hr = WICConvertBitmapSource(&GUID_VendorMicrosoft, &src_obj->IWICBitmapSource_iface, &dst_bitmap); ok(hr == WINCODEC_ERR_COMPONENTNOTFOUND, "WICConvertBitmapSource returned %x\n", hr); - DeleteTestBitmap(src_bitmap); + DeleteTestBitmap(src_obj); } static void test_default_converter(void) { - IWICBitmapSource *src_bitmap; + BitmapTestSrc *src_obj; IWICFormatConverter *converter; BOOL can_convert=1; HRESULT hr; - CreateTestBitmap(&testdata_32bppBGRA, &src_bitmap); + CreateTestBitmap(&testdata_32bppBGRA, &src_obj); hr = CoCreateInstance(&CLSID_WICDefaultFormatConverter, NULL, CLSCTX_INPROC_SERVER, &IID_IWICFormatConverter, (void**)&converter); @@ -343,7 +347,7 @@ static void test_default_converter(void) ok(SUCCEEDED(hr), "CanConvert returned %x\n", hr); ok(can_convert, "expected TRUE, got %i\n", can_convert); - hr = IWICFormatConverter_Initialize(converter, src_bitmap, + hr = IWICFormatConverter_Initialize(converter, &src_obj->IWICBitmapSource_iface, &GUID_WICPixelFormat32bppBGR, WICBitmapDitherTypeNone, NULL, 0.0, WICBitmapPaletteTypeCustom); ok(SUCCEEDED(hr), "Initialize returned %x\n", hr); @@ -354,7 +358,7 @@ static void test_default_converter(void) IWICFormatConverter_Release(converter); } - DeleteTestBitmap(src_bitmap); + DeleteTestBitmap(src_obj); } static void test_encoder(const struct bitmap_data *src, const CLSID* clsid_encoder, @@ -362,7 +366,7 @@ static void test_encoder(const struct bitmap_data *src, const CLSID* clsid_encod { HRESULT hr; IWICBitmapEncoder *encoder; - IWICBitmapSource *src_bitmap; + BitmapTestSrc *src_obj; HGLOBAL hglobal; IStream *stream; IWICBitmapFrameEncode *frameencode; @@ -371,7 +375,7 @@ static void test_encoder(const struct bitmap_data *src, const CLSID* clsid_encod IWICBitmapFrameDecode *framedecode; WICPixelFormatGUID pixelformat; - CreateTestBitmap(src, &src_bitmap); + CreateTestBitmap(src, &src_obj); hr = CoCreateInstance(clsid_encoder, NULL, CLSCTX_INPROC_SERVER, &IID_IWICBitmapEncoder, (void**)&encoder); @@ -406,7 +410,7 @@ static void test_encoder(const struct bitmap_data *src, const CLSID* clsid_encod hr = IWICBitmapFrameEncode_SetSize(frameencode, src->width, src->height); ok(SUCCEEDED(hr), "SetSize failed, hr=%x\n", hr); - hr = IWICBitmapFrameEncode_WriteSource(frameencode, src_bitmap, NULL); + hr = IWICBitmapFrameEncode_WriteSource(frameencode, &src_obj->IWICBitmapSource_iface, NULL); ok(SUCCEEDED(hr), "WriteSource failed, hr=%x\n", hr); hr = IWICBitmapFrameEncode_Commit(frameencode); @@ -450,7 +454,7 @@ static void test_encoder(const struct bitmap_data *src, const CLSID* clsid_encod IWICBitmapEncoder_Release(encoder); } - DeleteTestBitmap(src_bitmap); + DeleteTestBitmap(src_obj); } START_TEST(converter)

1 0

Ken Thomases : winemenubuilder: On Mac, ignore non-square icon entries.
by Alexandre Julliard Jan. 12, 2011

Jan. 12, 2011

Module: wine Branch: master Commit: eb4093909273b48abfecda5df4f59d9c2d310ed0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=eb4093909273b48abfecda5df… Author: Ken Thomases <ken(a)codeweavers.com> Date: Wed Jan 12 00:17:25 2011 -0600 winemenubuilder: On Mac, ignore non-square icon entries. --- programs/winemenubuilder/winemenubuilder.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/programs/winemenubuilder/winemenubuilder.c b/programs/winemenubuilder/winemenubuilder.c index 684e130..8cafce0 100644 --- a/programs/winemenubuilder/winemenubuilder.c +++ b/programs/winemenubuilder/winemenubuilder.c @@ -867,6 +867,8 @@ static HRESULT platform_write_icon(IStream *icoStream, int exeIndex, LPCWSTR ico int height = iconDirEntries[i].bHeight ? iconDirEntries[i].bHeight : 256; WINE_TRACE("[%d]: %d x %d @ %d\n", i, width, height, iconDirEntries[i].wBitCount); + if (height != width) + continue; slot = size_to_slot(width); if (slot < 0) continue;

1 0

Michael Stefaniuc : windowscodecs: Use ifaces instead of vtbl pointers in BmpDecoder.
by Alexandre Julliard Jan. 12, 2011

Jan. 12, 2011

Module: wine Branch: master Commit: 2785e905bc568d79e56e7505eb724f8f8ed2172d URL: http://source.winehq.org/git/wine.git/?a=commit;h=2785e905bc568d79e56e7505e… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Jan 12 00:49:53 2011 +0100 windowscodecs: Use ifaces instead of vtbl pointers in BmpDecoder. --- dlls/windowscodecs/bmpdecode.c | 59 +++++++++++++++++++++------------------ 1 files changed, 32 insertions(+), 27 deletions(-) diff --git a/dlls/windowscodecs/bmpdecode.c b/dlls/windowscodecs/bmpdecode.c index bd0ace1..6da6a3c 100644 --- a/dlls/windowscodecs/bmpdecode.c +++ b/dlls/windowscodecs/bmpdecode.c @@ -62,8 +62,8 @@ typedef struct { typedef HRESULT (*ReadDataFunc)(BmpDecoder* This); struct BmpDecoder { - const IWICBitmapDecoderVtbl *lpVtbl; - const IWICBitmapFrameDecodeVtbl *lpFrameVtbl; + IWICBitmapDecoder IWICBitmapDecoder_iface; + IWICBitmapFrameDecode IWICBitmapFrameDecode_iface; LONG ref; BOOL initialized; IStream *stream; @@ -81,9 +81,14 @@ struct BmpDecoder { int icoframe; /* If TRUE, this is a frame of a .ico file. */ }; -static inline BmpDecoder *impl_from_frame(IWICBitmapFrameDecode *iface) +static inline BmpDecoder *impl_from_IWICBitmapDecoder(IWICBitmapDecoder *iface) { - return CONTAINING_RECORD(iface, BmpDecoder, lpFrameVtbl); + return CONTAINING_RECORD(iface, BmpDecoder, IWICBitmapDecoder_iface); +} + +static inline BmpDecoder *impl_from_IWICBitmapFrameDecode(IWICBitmapFrameDecode *iface) +{ + return CONTAINING_RECORD(iface, BmpDecoder, IWICBitmapFrameDecode_iface); } static HRESULT WINAPI BmpFrameDecode_QueryInterface(IWICBitmapFrameDecode *iface, REFIID iid, @@ -111,22 +116,22 @@ static HRESULT WINAPI BmpFrameDecode_QueryInterface(IWICBitmapFrameDecode *iface static ULONG WINAPI BmpFrameDecode_AddRef(IWICBitmapFrameDecode *iface) { - BmpDecoder *This = impl_from_frame(iface); + BmpDecoder *This = impl_from_IWICBitmapFrameDecode(iface); - return IUnknown_AddRef((IUnknown*)This); + return IWICBitmapDecoder_AddRef(&This->IWICBitmapDecoder_iface); } static ULONG WINAPI BmpFrameDecode_Release(IWICBitmapFrameDecode *iface) { - BmpDecoder *This = impl_from_frame(iface); + BmpDecoder *This = impl_from_IWICBitmapFrameDecode(iface); - return IUnknown_Release((IUnknown*)This); + return IWICBitmapDecoder_Release(&This->IWICBitmapDecoder_iface); } static HRESULT WINAPI BmpFrameDecode_GetSize(IWICBitmapFrameDecode *iface, UINT *puiWidth, UINT *puiHeight) { - BmpDecoder *This = impl_from_frame(iface); + BmpDecoder *This = impl_from_IWICBitmapFrameDecode(iface); TRACE("(%p,%p,%p)\n", iface, puiWidth, puiHeight); if (This->bih.bV5Size == sizeof(BITMAPCOREHEADER)) @@ -146,7 +151,7 @@ static HRESULT WINAPI BmpFrameDecode_GetSize(IWICBitmapFrameDecode *iface, static HRESULT WINAPI BmpFrameDecode_GetPixelFormat(IWICBitmapFrameDecode *iface, WICPixelFormatGUID *pPixelFormat) { - BmpDecoder *This = impl_from_frame(iface); + BmpDecoder *This = impl_from_IWICBitmapFrameDecode(iface); TRACE("(%p,%p)\n", iface, pPixelFormat); memcpy(pPixelFormat, This->pixelformat, sizeof(GUID)); @@ -177,7 +182,7 @@ static HRESULT BmpHeader_GetResolution(BITMAPV5HEADER *bih, double *pDpiX, doubl static HRESULT WINAPI BmpFrameDecode_GetResolution(IWICBitmapFrameDecode *iface, double *pDpiX, double *pDpiY) { - BmpDecoder *This = impl_from_frame(iface); + BmpDecoder *This = impl_from_IWICBitmapFrameDecode(iface); TRACE("(%p,%p,%p)\n", iface, pDpiX, pDpiY); return BmpHeader_GetResolution(&This->bih, pDpiX, pDpiY); @@ -187,7 +192,7 @@ static HRESULT WINAPI BmpFrameDecode_CopyPalette(IWICBitmapFrameDecode *iface, IWICPalette *pIPalette) { HRESULT hr; - BmpDecoder *This = impl_from_frame(iface); + BmpDecoder *This = impl_from_IWICBitmapFrameDecode(iface); int count; WICColor *wiccolors=NULL; RGBTRIPLE *bgrcolors=NULL; @@ -299,7 +304,7 @@ end: static HRESULT WINAPI BmpFrameDecode_CopyPixels(IWICBitmapFrameDecode *iface, const WICRect *prc, UINT cbStride, UINT cbBufferSize, BYTE *pbBuffer) { - BmpDecoder *This = impl_from_frame(iface); + BmpDecoder *This = impl_from_IWICBitmapFrameDecode(iface); HRESULT hr=S_OK; UINT width, height; TRACE("(%p,%p,%u,%u,%p)\n", iface, prc, cbStride, cbBufferSize, pbBuffer); @@ -403,7 +408,7 @@ static HRESULT BmpFrameDecode_ReadRGB8(BmpDecoder* This) HRESULT hr; UINT width, height; - hr = IWICBitmapFrameDecode_GetSize((IWICBitmapFrameDecode*)&This->lpFrameVtbl, &width, &height); + hr = IWICBitmapFrameDecode_GetSize(&This->IWICBitmapFrameDecode_iface, &width, &height); if (SUCCEEDED(hr)) { @@ -954,7 +959,7 @@ static HRESULT BmpDecoder_ReadHeaders(BmpDecoder* This, IStream *stream) static HRESULT WINAPI BmpDecoder_QueryInterface(IWICBitmapDecoder *iface, REFIID iid, void **ppv) { - BmpDecoder *This = (BmpDecoder*)iface; + BmpDecoder *This = impl_from_IWICBitmapDecoder(iface); TRACE("(%p,%s,%p)\n", iface, debugstr_guid(iid), ppv); if (!ppv) return E_INVALIDARG; @@ -975,7 +980,7 @@ static HRESULT WINAPI BmpDecoder_QueryInterface(IWICBitmapDecoder *iface, REFIID static ULONG WINAPI BmpDecoder_AddRef(IWICBitmapDecoder *iface) { - BmpDecoder *This = (BmpDecoder*)iface; + BmpDecoder *This = impl_from_IWICBitmapDecoder(iface); ULONG ref = InterlockedIncrement(&This->ref); TRACE("(%p) refcount=%u\n", iface, ref); @@ -985,7 +990,7 @@ static ULONG WINAPI BmpDecoder_AddRef(IWICBitmapDecoder *iface) static ULONG WINAPI BmpDecoder_Release(IWICBitmapDecoder *iface) { - BmpDecoder *This = (BmpDecoder*)iface; + BmpDecoder *This = impl_from_IWICBitmapDecoder(iface); ULONG ref = InterlockedDecrement(&This->ref); TRACE("(%p) refcount=%u\n", iface, ref); @@ -1006,7 +1011,7 @@ static HRESULT WINAPI BmpDecoder_QueryCapability(IWICBitmapDecoder *iface, IStre DWORD *pdwCapability) { HRESULT hr; - BmpDecoder *This = (BmpDecoder*)iface; + BmpDecoder *This = impl_from_IWICBitmapDecoder(iface); EnterCriticalSection(&This->lock); hr = BmpDecoder_ReadHeaders(This, pIStream); @@ -1025,7 +1030,7 @@ static HRESULT WINAPI BmpDecoder_Initialize(IWICBitmapDecoder *iface, IStream *p WICDecodeOptions cacheOptions) { HRESULT hr; - BmpDecoder *This = (BmpDecoder*)iface; + BmpDecoder *This = impl_from_IWICBitmapDecoder(iface); EnterCriticalSection(&This->lock); hr = BmpDecoder_ReadHeaders(This, pIStream); @@ -1112,13 +1117,13 @@ static HRESULT WINAPI BmpDecoder_GetFrameCount(IWICBitmapDecoder *iface, static HRESULT WINAPI BmpDecoder_GetFrame(IWICBitmapDecoder *iface, UINT index, IWICBitmapFrameDecode **ppIBitmapFrame) { - BmpDecoder *This = (BmpDecoder*)iface; + BmpDecoder *This = impl_from_IWICBitmapDecoder(iface); if (index != 0) return E_INVALIDARG; if (!This->stream) return WINCODEC_ERR_WRONGSTATE; - *ppIBitmapFrame = (IWICBitmapFrameDecode*)&This->lpFrameVtbl; + *ppIBitmapFrame = &This->IWICBitmapFrameDecode_iface; IWICBitmapDecoder_AddRef(iface); return S_OK; @@ -1148,8 +1153,8 @@ static HRESULT BmpDecoder_Create(int packed, int icoframe, BmpDecoder **ppDecode This = HeapAlloc(GetProcessHeap(), 0, sizeof(BmpDecoder)); if (!This) return E_OUTOFMEMORY; - This->lpVtbl = &BmpDecoder_Vtbl; - This->lpFrameVtbl = &BmpDecoder_FrameVtbl; + This->IWICBitmapDecoder_iface.lpVtbl = &BmpDecoder_Vtbl; + This->IWICBitmapFrameDecode_iface.lpVtbl = &BmpDecoder_FrameVtbl; This->ref = 1; This->initialized = FALSE; This->stream = NULL; @@ -1178,8 +1183,8 @@ static HRESULT BmpDecoder_Construct(int packed, int icoframe, IUnknown *pUnkOute ret = BmpDecoder_Create(packed, icoframe, &This); if (FAILED(ret)) return ret; - ret = IUnknown_QueryInterface((IUnknown*)This, iid, ppv); - IUnknown_Release((IUnknown*)This); + ret = IWICBitmapDecoder_QueryInterface(&This->IWICBitmapDecoder_iface, iid, ppv); + IWICBitmapDecoder_Release(&This->IWICBitmapDecoder_iface); return ret; } @@ -1201,7 +1206,7 @@ HRESULT IcoDibDecoder_CreateInstance(BmpDecoder **ppDecoder) void BmpDecoder_GetWICDecoder(BmpDecoder *This, IWICBitmapDecoder **ppDecoder) { - *ppDecoder = (IWICBitmapDecoder*)This; + *ppDecoder = &This->IWICBitmapDecoder_iface; } /* Return the offset where the mask of an icon might be, or 0 for failure. */ @@ -1213,7 +1218,7 @@ void BmpDecoder_FindIconMask(BmpDecoder *This, ULONG *mask_offset, int *topdown) { /* RGB or BITFIELDS data */ ULONG width, height, bytesperrow, datasize; - IWICBitmapFrameDecode_GetSize((IWICBitmapFrameDecode*)&This->lpFrameVtbl, &width, &height); + IWICBitmapFrameDecode_GetSize(&This->IWICBitmapFrameDecode_iface, &width, &height); bytesperrow = (((width * This->bitsperpixel)+31)/32)*4; datasize = bytesperrow * height; *mask_offset = This->image_offset + datasize;

1 0

Dylan Smith : richedit: Clear non-default style when changing to plain text mode.
by Alexandre Julliard Jan. 12, 2011

Jan. 12, 2011

Module: wine Branch: master Commit: d43cddcc8351215eb4f60e1a41439e29261f1268 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d43cddcc8351215eb4f60e1a4… Author: Dylan Smith <dylan.ah.smith(a)gmail.com> Date: Wed Jan 12 02:23:40 2011 -0500 richedit: Clear non-default style when changing to plain text mode. Although the text must be empty for the mode to change to plain text mode, there still may be a non-default style for the temporary insert style and style for the end of text run. --- dlls/riched20/editor.c | 12 ++++++++++++ 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index 48b64c9..aace0ea 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -4332,6 +4332,18 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, { mask |= TM_RICHTEXT | TM_PLAINTEXT; changes |= wParam & (TM_RICHTEXT | TM_PLAINTEXT); + if (wParam & TM_PLAINTEXT) { + /* Clear selection since it should be possible to select the + * end of text run for rich text */ + ME_InvalidateSelection(editor); + ME_SetCursorToStart(editor, &editor->pCursors[0]); + editor->pCursors[1] = editor->pCursors[0]; + /* plain text can only have the default style. */ + ME_ClearTempStyle(editor); + ME_AddRefStyle(editor->pBuffer->pDefaultStyle); + ME_ReleaseStyle(editor->pCursors[0].pRun->member.run.style); + editor->pCursors[0].pRun->member.run.style = editor->pBuffer->pDefaultStyle; + } } /* FIXME: Currently no support for undo level and code page options */ editor->mode = (editor->mode & ~mask) | changes;

1 0

Ken Thomases : winemenubuilder: On Mac, size is implicit in ICNS slot; don' t search for best.
by Alexandre Julliard Jan. 12, 2011

Jan. 12, 2011

Module: wine Branch: master Commit: e72a37e9604ae1a14d4928abbe7551aac8c38cdf URL: http://source.winehq.org/git/wine.git/?a=commit;h=e72a37e9604ae1a14d4928abb… Author: Ken Thomases <ken(a)codeweavers.com> Date: Wed Jan 12 00:17:30 2011 -0600 winemenubuilder: On Mac, size is implicit in ICNS slot; don't search for best. --- programs/winemenubuilder/winemenubuilder.c | 6 +----- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/programs/winemenubuilder/winemenubuilder.c b/programs/winemenubuilder/winemenubuilder.c index 8cafce0..56ff158 100644 --- a/programs/winemenubuilder/winemenubuilder.c +++ b/programs/winemenubuilder/winemenubuilder.c @@ -839,7 +839,6 @@ static HRESULT platform_write_icon(IStream *icoStream, int exeIndex, LPCWSTR ico int numEntries; struct { int index; - int maxPixels; int maxBits; } best[ICNS_SLOTS]; int indexes[ICNS_SLOTS]; @@ -857,7 +856,6 @@ static HRESULT platform_write_icon(IStream *icoStream, int exeIndex, LPCWSTR ico for (i = 0; i < ICNS_SLOTS; i++) { best[i].index = -1; - best[i].maxPixels = 0; best[i].maxBits = 0; } for (i = 0; i < numEntries; i++) @@ -872,11 +870,9 @@ static HRESULT platform_write_icon(IStream *icoStream, int exeIndex, LPCWSTR ico slot = size_to_slot(width); if (slot < 0) continue; - if (iconDirEntries[i].wBitCount >= best[slot].maxBits && - (height * width) >= best[slot].maxPixels) + if (iconDirEntries[i].wBitCount >= best[slot].maxBits) { best[slot].index = i; - best[slot].maxPixels = height * width; best[slot].maxBits = iconDirEntries[i].wBitCount; } }

1 0

Michael Stefaniuc : urlmon: COM cleanup for the IInternetProtocolEx iface.
by Alexandre Julliard Jan. 12, 2011

Jan. 12, 2011

Module: wine Branch: master Commit: c41cf7d6a5c22bd22aac842995bed46c7867ab44 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c41cf7d6a5c22bd22aac84299… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Jan 12 00:43:44 2011 +0100 urlmon: COM cleanup for the IInternetProtocolEx iface. --- dlls/urlmon/binding.c | 24 +++++++------ dlls/urlmon/bindprot.c | 87 +++++++++++++++++++++++---------------------- dlls/urlmon/session.c | 2 +- dlls/urlmon/urlmon_main.h | 4 +-- 4 files changed, 59 insertions(+), 58 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=c41cf7d6a5c22bd22aac8…

1 0

Michael Stefaniuc : urlmon: COM cleanup for the rest of the iface in BindProtocol.
by Alexandre Julliard Jan. 12, 2011

Jan. 12, 2011

Module: wine Branch: master Commit: 1515b36aaafd851dbe86aec71dee2aed296e098e URL: http://source.winehq.org/git/wine.git/?a=commit;h=1515b36aaafd851dbe86aec71… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Jan 12 00:47:19 2011 +0100 urlmon: COM cleanup for the rest of the iface in BindProtocol. --- dlls/urlmon/bindprot.c | 127 +++++++++++++++++++++++---------------------- dlls/urlmon/urlmon_main.h | 10 ++-- 2 files changed, 70 insertions(+), 67 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=1515b36aaafd851dbe86a…

1 0

Alexandre Julliard : dbghelp: Try to load the string table from file number 4 too.
by Alexandre Julliard Jan. 12, 2011

Jan. 12, 2011

Module: wine Branch: master Commit: 530e9c9b175c03f7592eaabdf8c34f45be50117a URL: http://source.winehq.org/git/wine.git/?a=commit;h=530e9c9b175c03f7592eaabdf… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Jan 11 20:34:46 2011 +0100 dbghelp: Try to load the string table from file number 4 too. --- dlls/dbghelp/msc.c | 46 +++++++++++++++++++++------------------------- 1 files changed, 21 insertions(+), 25 deletions(-) diff --git a/dlls/dbghelp/msc.c b/dlls/dbghelp/msc.c index 93eef21..0514bc9 100644 --- a/dlls/dbghelp/msc.c +++ b/dlls/dbghelp/msc.c @@ -2119,12 +2119,8 @@ static void* pdb_read_ds_file(const struct PDB_DS_HEADER* pdb, DWORD i; if (!toc || file_nr >= toc->num_files) return NULL; + if (toc->file_size[file_nr] == 0 || toc->file_size[file_nr] == 0xFFFFFFFF) return NULL; - if (toc->file_size[file_nr] == 0 || toc->file_size[file_nr] == 0xFFFFFFFF) - { - FIXME(">>> requesting NULL stream (%u)\n", file_nr); - return NULL; - } block_list = &toc->file_size[toc->num_files]; for (i = 0; i < file_nr; i++) block_list += (toc->file_size[i] + pdb->block_size - 1) / pdb->block_size; @@ -2177,6 +2173,23 @@ static void pdb_free_file(struct pdb_file_info* pdb_file) } } +static void* pdb_read_strings(const struct pdb_file_info* pdb_file) +{ + void *ret; + + /* FIXME: how to determine the correct file number? */ + /* 4 and 12 have been observed, there may be others */ + + ret = pdb_read_file( pdb_file, 4 ); + if (ret && *(const DWORD *)ret == 0xeffeeffe) return ret; + pdb_free( ret ); + ret = pdb_read_file( pdb_file, 12 ); + if (ret && *(const DWORD *)ret == 0xeffeeffe) return ret; + pdb_free( ret ); + WARN("string table not found\n"); + return NULL; +} + static void pdb_module_remove(struct process* pcsn, struct module_format* modfmt) { unsigned i; @@ -2596,20 +2609,8 @@ static BOOL pdb_process_internal(const struct process* pcs, symbols.version, symbols.version); } - files_image = pdb_read_file(pdb_file, 12); /* FIXME: really fixed ??? */ - if (files_image) - { - if (*(const DWORD*)files_image == 0xeffeeffe) - { - files_size = *(const DWORD*)(files_image + 8); - } - else - { - WARN("wrong header %x expecting 0xeffeeffe\n", *(const DWORD*)files_image); - pdb_free(files_image); - files_image = NULL; - } - } + files_image = pdb_read_strings(pdb_file); + if (files_image) files_size = *(const DWORD*)(files_image + 8); pdb_process_symbol_imports(pcs, msc_dbg, &symbols, symbols_image, image, pdb_lookup, pdb_module_info, module_index); @@ -3015,13 +3016,8 @@ BOOL pdb_virtual_unwind(struct cpu_stack_walk* csw, DWORD_PTR ip, TRACE("searching %lx => %lx\n", ip, ip - (DWORD_PTR)pair.effective->module.BaseOfImage); ip -= (DWORD_PTR)pair.effective->module.BaseOfImage; - strbase = pdb_read_file(&pdb_info->pdb_files[0], 12); /* FIXME: really fixed index ??? */ + strbase = pdb_read_strings(&pdb_info->pdb_files[0]); if (!strbase) return FALSE; - if (*(const DWORD*)strbase != 0xeffeeffe) - { - pdb_free(strbase); - return FALSE; - } strsize = *(const DWORD*)(strbase + 8); fpoext = pdb_read_file(&pdb_info->pdb_files[0], 10); size = pdb_get_file_size(&pdb_info->pdb_files[0], 10);

1 0

Alexandre Julliard : winedump: Try to load the PDB string table from file number 4 too.
by Alexandre Julliard Jan. 12, 2011

Jan. 12, 2011

Module: wine Branch: master Commit: 4f574ee942be99c7401a3bef472a81d0f2bb0b5b URL: http://source.winehq.org/git/wine.git/?a=commit;h=4f574ee942be99c7401a3bef4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Jan 12 10:45:54 2011 +0100 winedump: Try to load the PDB string table from file number 4 too. --- tools/winedump/pdb.c | 32 ++++++++++++++++++-------------- 1 files changed, 18 insertions(+), 14 deletions(-) diff --git a/tools/winedump/pdb.c b/tools/winedump/pdb.c index 5720c40..1b7beeb 100644 --- a/tools/winedump/pdb.c +++ b/tools/winedump/pdb.c @@ -155,6 +155,21 @@ static void pdb_exit(struct pdb_reader* reader) free((char*)reader->u.ds.toc); } +static void *read_string_table(struct pdb_reader* reader) +{ + void *ret; + + /* FIXME: how to determine the correct file number? */ + /* 4 and 12 have been observed, there may be others */ + ret = reader->read_file(reader, 4); + if (ret && *(const DWORD*)ret == 0xeffeeffe) return ret; + free( ret ); + ret = reader->read_file(reader, 12); + if (ret && *(const DWORD*)ret == 0xeffeeffe) return ret; + free( ret ); + return NULL; +} + static void pdb_dump_symbols(struct pdb_reader* reader) { PDB_SYMBOLS* symbols; @@ -224,20 +239,8 @@ static void pdb_dump_symbols(struct pdb_reader* reader) dump_data(src, symbols->offset_size, " "); } - filesimage = reader->read_file(reader, 12); /* FIXME: really fixed ??? */ - if (filesimage) - { - if (*(const DWORD*)filesimage == 0xeffeeffe) - { - filessize = *(const DWORD*)(filesimage + 8); - } - else - { - printf("wrong header %x expecting 0xeffeeffe\n", *(const DWORD*)filesimage); - free(filesimage); - filesimage = NULL; - } - } + if (!(filesimage = read_string_table(reader))) printf("string table not found\n"); + else filessize = *(const DWORD*)(filesimage + 8); if (symbols->srcmodule_size) { @@ -721,6 +724,7 @@ static void pdb_ds_dump(void) * 1: root structure * 2: types * 3: modules + * 4: string table (FIXME: in which case?) * 5: FPO data * 8: segments * 10: extended FPO data

1 0

Alexandre Julliard : winedump: Print the machine field from the PDB symbol table.
by Alexandre Julliard Jan. 11, 2011

Jan. 11, 2011

Module: wine Branch: master Commit: ce6779c9b1c77d5502b97023d3be4ded26b2aa5e URL: http://source.winehq.org/git/wine.git/?a=commit;h=ce6779c9b1c77d5502b97023d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Jan 11 16:17:27 2011 +0100 winedump: Print the machine field from the PDB symbol table. --- include/wine/mscvpdb.h | 5 ++++- tools/winedump/pdb.c | 40 +++++++++++++++++++++------------------- 2 files changed, 25 insertions(+), 20 deletions(-) diff --git a/include/wine/mscvpdb.h b/include/wine/mscvpdb.h index 59ce8a5..637ed0f 100644 --- a/include/wine/mscvpdb.h +++ b/include/wine/mscvpdb.h @@ -1914,7 +1914,10 @@ typedef struct _PDB_SYMBOLS DWORD hash_size; DWORD srcmodule_size; DWORD pdbimport_size; - DWORD resvd[5]; + DWORD resvd[3]; + WORD resvd3; + WORD machine; + DWORD resvd4; } PDB_SYMBOLS, *PPDB_SYMBOLS; typedef struct _PDB_FPO_DATA diff --git a/tools/winedump/pdb.c b/tools/winedump/pdb.c index 4568d29..5720c40 100644 --- a/tools/winedump/pdb.c +++ b/tools/winedump/pdb.c @@ -178,23 +178,24 @@ static void pdb_dump_symbols(struct pdb_reader* reader) printf("-Unknown symbol info version %d\n", symbols->version); } printf("Symbols:\n" - "\tsignature: %08x\n" - "\tversion: %u\n" - "\tunknown: %08x\n" - "\thash1_file: %08x\n" - "\thash2_file: %08x\n" - "\tgsym_file: %04x\n" - "\tunknown1: %04x\n" - "\tmodule_size: %08x\n" - "\toffset_size: %08x\n" - "\thash_size: %08x\n" - "\tsrc_module_size %08x\n" - "\tpdbimport_size %08x\n" - "\tresvd[0] %08x\n" - "\tresvd[1] %08x\n" - "\tresvd[2] %08x\n" - "\tresvd[3] %08x\n" - "\tresvd[4] %08x\n", + "\tsignature: %08x\n" + "\tversion: %u\n" + "\tunknown: %08x\n" + "\thash1_file: %08x\n" + "\thash2_file: %08x\n" + "\tgsym_file: %04x\n" + "\tunknown1: %04x\n" + "\tmodule_size: %08x\n" + "\toffset_size: %08x\n" + "\thash_size: %08x\n" + "\tsrc_module_size: %08x\n" + "\tpdbimport_size: %08x\n" + "\tresvd[0] %08x\n" + "\tresvd[1] %08x\n" + "\tresvd[2] %08x\n" + "\tresvd[3] %04x\n" + "\tmachine: %s\n" + "\tresvd[4] %08x\n", symbols->signature, symbols->version, symbols->unknown, @@ -210,8 +211,9 @@ static void pdb_dump_symbols(struct pdb_reader* reader) symbols->resvd[0], symbols->resvd[1], symbols->resvd[2], - symbols->resvd[3], - symbols->resvd[4]); + symbols->resvd3, + get_machine_str( symbols->machine ), + symbols->resvd4); if (symbols->offset_size) {

1 0