wine-commits April 2012

wine-commits@winehq.org

2 participants
718 discussions

Henri Verbeet : include: Fix a couple of ID3DXSaveUserData prototypes.
by Alexandre Julliard 13 Apr '12

13 Apr '12

Module: wine Branch: master Commit: bdcc868ec473739076204e1fb6f3dec68af7a7bd URL: http://source.winehq.org/git/wine.git/?a=commit;h=bdcc868ec473739076204e1fb… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Apr 12 22:49:05 2012 +0200 include: Fix a couple of ID3DXSaveUserData prototypes. --- include/d3dx9anim.h | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/include/d3dx9anim.h b/include/d3dx9anim.h index 142acba..1e31299 100644 --- a/include/d3dx9anim.h +++ b/include/d3dx9anim.h @@ -192,10 +192,10 @@ DECLARE_INTERFACE(ID3DXLoadUserData) #define INTERFACE ID3DXSaveUserData DECLARE_INTERFACE(ID3DXSaveUserData) { - STDMETHOD(AddFrameChildData)(CONST LPD3DXFRAME *frame, LPD3DXFILESAVEOBJECT save_obj, - LPD3DXFILESAVEDATA frame_data) PURE; - STDMETHOD(AddMeshChildData)(CONST LPD3DXMESHCONTAINER *mesh_container, - LPD3DXFILESAVEOBJECT save_obj, LPD3DXFILESAVEDATA mesh_data) PURE; + STDMETHOD(AddFrameChildData)(const D3DXFRAME *frame, + ID3DXFileSaveObject *save_obj, ID3DXFileSaveData *frame_data) PURE; + STDMETHOD(AddMeshChildData)(const D3DXMESHCONTAINER *mesh_container, + ID3DXFileSaveObject *save_obj, ID3DXFileSaveData *mesh_data) PURE; STDMETHOD(AddTopLevelDataObjectsPre)(LPD3DXFILESAVEOBJECT save_obj) PURE; STDMETHOD(AddTopLevelDataObjectsPost)(LPD3DXFILESAVEOBJECT save_obj) PURE; STDMETHOD(RegisterTemplates)(LPD3DXFILE xfile) PURE;

1 0

Henri Verbeet : wined3d: Unify the wined3d_mapped_rect and wined3d_mapped_box structures.
by Alexandre Julliard 13 Apr '12

13 Apr '12

Module: wine Branch: master Commit: 8cef8b358760b17de780f0093f653c0c7e6afebc URL: http://source.winehq.org/git/wine.git/?a=commit;h=8cef8b358760b17de780f0093… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Apr 12 22:49:04 2012 +0200 wined3d: Unify the wined3d_mapped_rect and wined3d_mapped_box structures. --- dlls/d3d10core/texture.c | 2 +- dlls/d3d8/surface.c | 27 +++++++++++++++++---------- dlls/d3d8/volume.c | 12 ++++++++---- dlls/d3d9/surface.c | 11 ++++++++--- dlls/d3d9/volume.c | 12 ++++++++---- dlls/ddraw/surface.c | 16 ++++++++-------- dlls/wined3d/device.c | 20 +++++++++----------- dlls/wined3d/surface.c | 29 +++++++++++++++-------------- dlls/wined3d/volume.c | 20 ++++++++++---------- include/wine/wined3d.h | 12 +++--------- 10 files changed, 87 insertions(+), 74 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=8cef8b358760b17de780f…

1 0

Henri Verbeet : d3d9: COM cleanup for the IDirect3DVertexDeclaration9 interface.
by Alexandre Julliard 13 Apr '12

13 Apr '12

Module: wine Branch: master Commit: 949f7b45244d77bfd839307ee002fa1bc6ca52d2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=949f7b45244d77bfd839307ee… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Apr 12 22:49:03 2012 +0200 d3d9: COM cleanup for the IDirect3DVertexDeclaration9 interface. --- dlls/d3d9/buffer.c | 1 - dlls/d3d9/d3d9_private.h | 5 +++- dlls/d3d9/device.c | 11 ++++--- dlls/d3d9/shader.c | 1 - dlls/d3d9/surface.c | 1 - dlls/d3d9/vertexdeclaration.c | 61 +++++++++++++++++++++++++--------------- 6 files changed, 48 insertions(+), 32 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=949f7b45244d77bfd8393…

1 0

Henri Verbeet : d3d9: Use the wined3d declaration to retrieve the FVF in IDirect3DDevice9Impl_GetFVF ().
by Alexandre Julliard 13 Apr '12

13 Apr '12

Module: wine Branch: master Commit: 81ed6e0c86108a640f741d0a6bc6e3622a4ce90e URL: http://source.winehq.org/git/wine.git/?a=commit;h=81ed6e0c86108a640f741d0a6… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Apr 12 22:49:02 2012 +0200 d3d9: Use the wined3d declaration to retrieve the FVF in IDirect3DDevice9Impl_GetFVF(). --- dlls/d3d9/device.c | 29 +++++++++++++++++------------ 1 files changed, 17 insertions(+), 12 deletions(-) diff --git a/dlls/d3d9/device.c b/dlls/d3d9/device.c index 440533c..b265d82 100644 --- a/dlls/d3d9/device.c +++ b/dlls/d3d9/device.c @@ -2236,32 +2236,37 @@ static HRESULT WINAPI IDirect3DDevice9Impl_SetFVF(IDirect3DDevice9Ex *iface, DWO return hr; } -static HRESULT WINAPI IDirect3DDevice9Impl_GetFVF(IDirect3DDevice9Ex *iface, DWORD *pFVF) +static HRESULT WINAPI IDirect3DDevice9Impl_GetFVF(IDirect3DDevice9Ex *iface, DWORD *fvf) { - IDirect3DVertexDeclaration9 *decl; + IDirect3DDevice9Impl *device = impl_from_IDirect3DDevice9Ex(iface); + struct wined3d_vertex_declaration *wined3d_declaration; + IDirect3DVertexDeclaration9Impl *d3d9_declaration; HRESULT hr; - TRACE("iface %p, fvf %p.\n", iface, pFVF); + TRACE("iface %p, fvf %p.\n", iface, fvf); - hr = IDirect3DDevice9_GetVertexDeclaration(iface, &decl); - if (FAILED(hr)) + wined3d_mutex_lock(); + if (FAILED(hr = wined3d_device_get_vertex_declaration(device->wined3d_device, &wined3d_declaration))) { - WARN("Failed to get vertex declaration, %#x\n", hr); - *pFVF = 0; + wined3d_mutex_unlock(); + WARN("Failed to get vertex declaration, hr %#x.\n", hr); + *fvf = 0; return hr; } - if (decl) + if (wined3d_declaration) { - *pFVF = ((IDirect3DVertexDeclaration9Impl *)decl)->convFVF; - IDirect3DVertexDeclaration9_Release(decl); + d3d9_declaration = wined3d_vertex_declaration_get_parent(wined3d_declaration); + *fvf = d3d9_declaration->convFVF; + wined3d_vertex_declaration_decref(wined3d_declaration); } else { - *pFVF = 0; + *fvf = 0; } + wined3d_mutex_unlock(); - TRACE("Returning FVF %#x\n", *pFVF); + TRACE("Returning FVF %#x.\n", *fvf); return hr; }

1 0

Akihiro Sagawa : comctl32: Add support for retrieving lpszText in TOOLINFO structure.
by Alexandre Julliard 12 Apr '12

12 Apr '12

Module: wine Branch: master Commit: b48f394e4f3ee73eb2f974d19770d411d13f3e62 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b48f394e4f3ee73eb2f974d19… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Thu Apr 12 00:42:55 2012 +0900 comctl32: Add support for retrieving lpszText in TOOLINFO structure. --- dlls/comctl32/tests/tooltips.c | 16 ++++++++++++++++ dlls/comctl32/tooltips.c | 27 +++++++++++++++++++-------- 2 files changed, 35 insertions(+), 8 deletions(-) diff --git a/dlls/comctl32/tests/tooltips.c b/dlls/comctl32/tests/tooltips.c index 596598f..31504d1 100644 --- a/dlls/comctl32/tests/tooltips.c +++ b/dlls/comctl32/tests/tooltips.c @@ -322,6 +322,11 @@ static void test_gettext(void) toolinfoA.lpszText = bufA; SendMessageA(hwnd, TTM_GETTEXTA, 0, (LPARAM)&toolinfoA); ok(strcmp(toolinfoA.lpszText, "") == 0, "lpszText should be an empty string\n"); + + toolinfoA.lpszText = bufA; + SendMessageA(hwnd, TTM_GETTOOLINFOA, 0, (LPARAM)&toolinfoA); + ok(toolinfoA.lpszText == NULL, + "expected NULL, got %p", toolinfoA.lpszText); } else { @@ -355,6 +360,12 @@ static void test_gettext(void) SendMessageA(hwnd, TTM_GETTEXTA, 0, (LPARAM)&toolinfoA); ok(strcmp(toolinfoA.lpszText, testtipA) == 0, "lpszText should be an empty string\n"); + memset(bufA, 0x1f, sizeof(bufA)); + toolinfoA.lpszText = bufA; + SendMessageA(hwnd, TTM_GETTOOLINFOA, 0, (LPARAM)&toolinfoA); + ok(strcmp(toolinfoA.lpszText, testtipA) == 0, + "expected %s, got %p\n", testtipA, toolinfoA.lpszText); + length = SendMessage(hwnd, WM_GETTEXTLENGTH, 0, 0); ok(length == 0, "Expected 0, got %d\n", length); } @@ -378,6 +389,11 @@ static void test_gettext(void) SendMessageA(hwnd, TTM_GETTEXTA, 0, (LPARAM)&toolinfoA); ok(strcmp(toolinfoA.lpszText, testcallbackA) == 0, "lpszText should be an (%s) string\n", testcallbackA); + + toolinfoA.lpszText = bufA; + SendMessageA(hwnd, TTM_GETTOOLINFOA, 0, (LPARAM)&toolinfoA); + ok(toolinfoA.lpszText == LPSTR_TEXTCALLBACKA, + "expected LPSTR_TEXTCALLBACKA, got %p\n", toolinfoA.lpszText); } DestroyWindow(hwnd); diff --git a/dlls/comctl32/tooltips.c b/dlls/comctl32/tooltips.c index 162867d..1d758b4 100644 --- a/dlls/comctl32/tooltips.c +++ b/dlls/comctl32/tooltips.c @@ -943,6 +943,21 @@ TOOLTIPS_GetToolFromPoint (const TOOLTIPS_INFO *infoPtr, HWND hwnd, const POINT return -1; } +static inline void +TOOLTIPS_CopyInfoT (const TTTOOL_INFO *toolPtr, TTTOOLINFOW *ti, BOOL isW) +{ + if (ti->lpszText) { + if (toolPtr->lpszText == NULL || + IS_INTRESOURCE(toolPtr->lpszText) || + toolPtr->lpszText == LPSTR_TEXTCALLBACKW) + ti->lpszText = toolPtr->lpszText; + else if (isW) + strcpyW (ti->lpszText, toolPtr->lpszText); + else + WideCharToMultiByte(CP_ACP, 0, toolPtr->lpszText, -1, + (LPSTR)ti->lpszText, INFOTIPSIZE, NULL, NULL); + } +} static BOOL TOOLTIPS_IsWindowActive (HWND hwnd) @@ -1199,8 +1214,7 @@ TOOLTIPS_EnumToolsT (const TOOLTIPS_INFO *infoPtr, UINT uIndex, TTTOOLINFOW *ti, ti->uId = toolPtr->uId; ti->rect = toolPtr->rect; ti->hinst = toolPtr->hinst; -/* ti->lpszText = toolPtr->lpszText; */ - ti->lpszText = NULL; /* FIXME */ + TOOLTIPS_CopyInfoT (toolPtr, ti, isW); if (ti->cbSize >= TTTOOLINFOA_V2_SIZE) ti->lParam = toolPtr->lParam; @@ -1246,8 +1260,7 @@ TOOLTIPS_GetCurrentToolT (const TOOLTIPS_INFO *infoPtr, TTTOOLINFOW *ti, BOOL is ti->uFlags = toolPtr->uFlags; ti->rect = toolPtr->rect; ti->hinst = toolPtr->hinst; -/* ti->lpszText = toolPtr->lpszText; */ - ti->lpszText = NULL; /* FIXME */ + TOOLTIPS_CopyInfoT (toolPtr, ti, isW); if (ti->cbSize >= TTTOOLINFOW_V2_SIZE) ti->lParam = toolPtr->lParam; @@ -1385,8 +1398,7 @@ TOOLTIPS_GetToolInfoT (const TOOLTIPS_INFO *infoPtr, TTTOOLINFOW *ti, BOOL isW) ti->uFlags = toolPtr->uFlags; ti->rect = toolPtr->rect; ti->hinst = toolPtr->hinst; -/* lpToolInfo->lpszText = toolPtr->lpszText; */ - ti->lpszText = NULL; /* FIXME */ + TOOLTIPS_CopyInfoT (toolPtr, ti, isW); if (ti->cbSize >= TTTOOLINFOW_V2_SIZE) ti->lParam = toolPtr->lParam; @@ -1420,8 +1432,7 @@ TOOLTIPS_HitTestT (const TOOLTIPS_INFO *infoPtr, LPTTHITTESTINFOW lptthit, lptthit->ti.uId = toolPtr->uId; lptthit->ti.rect = toolPtr->rect; lptthit->ti.hinst = toolPtr->hinst; -/* lptthit->ti.lpszText = toolPtr->lpszText; */ - lptthit->ti.lpszText = NULL; /* FIXME */ + TOOLTIPS_CopyInfoT (toolPtr, &lptthit->ti, isW); if (lptthit->ti.cbSize >= TTTOOLINFOW_V2_SIZE) lptthit->ti.lParam = toolPtr->lParam; }

1 0

Aric Stewart : wineqtdecoder: Call OutputQueue_Destroy during pin disconnect.
by Alexandre Julliard 12 Apr '12

12 Apr '12

Module: wine Branch: master Commit: 40456f4bb4089f4c77131bf99efc1cd0175f85e9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=40456f4bb4089f4c77131bf99… Author: Aric Stewart <aric(a)codeweavers.com> Date: Wed Apr 11 14:07:18 2012 -0500 wineqtdecoder: Call OutputQueue_Destroy during pin disconnect. --- dlls/wineqtdecoder/qtsplitter.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/wineqtdecoder/qtsplitter.c b/dlls/wineqtdecoder/qtsplitter.c index ba2d05d..b21f198 100644 --- a/dlls/wineqtdecoder/qtsplitter.c +++ b/dlls/wineqtdecoder/qtsplitter.c @@ -745,6 +745,7 @@ static HRESULT QT_RemoveOutputPins(QTSplitter *This) if (This->pVideo_Pin) { + OutputQueue_Destroy(This->pVideo_Pin->queue); hr = BaseOutputPinImpl_BreakConnect(&This->pVideo_Pin->pin); TRACE("Disconnect: %08x\n", hr); IPin_Release(&This->pVideo_Pin->pin.pin.IPin_iface); @@ -752,6 +753,7 @@ static HRESULT QT_RemoveOutputPins(QTSplitter *This) } if (This->pAudio_Pin) { + OutputQueue_Destroy(This->pAudio_Pin->queue); hr = BaseOutputPinImpl_BreakConnect(&This->pAudio_Pin->pin); TRACE("Disconnect: %08x\n", hr); IPin_Release(&This->pAudio_Pin->pin.pin.IPin_iface); @@ -1269,7 +1271,6 @@ static ULONG WINAPI QTOutPin_Release(IPin *iface) { DeleteMediaType(This->pmt); FreeMediaType(&This->pin.pin.mtCurrent); - OutputQueue_Destroy(This->queue); CoTaskMemFree(This); return 0; }

1 0

Piotr Caban : msvcrt: Added support for _TRUNCATE flag in wcsncpy_s.
by Alexandre Julliard 12 Apr '12

12 Apr '12

Module: wine Branch: master Commit: a80aec4c56bfc9b845e89bf55af4e638f153b4cd URL: http://source.winehq.org/git/wine.git/?a=commit;h=a80aec4c56bfc9b845e89bf55… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Apr 12 16:19:31 2012 +0200 msvcrt: Added support for _TRUNCATE flag in wcsncpy_s. --- dlls/msvcrt/tests/string.c | 6 ++++++ dlls/msvcrt/wcs.c | 11 ++++++++--- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/dlls/msvcrt/tests/string.c b/dlls/msvcrt/tests/string.c index 9f0d8be..afa9954 100644 --- a/dlls/msvcrt/tests/string.c +++ b/dlls/msvcrt/tests/string.c @@ -682,6 +682,12 @@ static void test_wcscpy_s(void) ret = p_wcsncpy_s(szDestShort, 8, szLongText, sizeof(szLongText)/sizeof(WCHAR)); ok(ret == ERANGE || ret == EINVAL, "expected ERANGE/EINVAL got %d\n", ret); ok(szDestShort[0] == 0, "szDestShort[0] not 0\n"); + + szDest[0] = 'A'; + ret = p_wcsncpy_s(szDest, 5, szLongText, -1); + ok(ret == STRUNCATE, "expected STRUNCATE got %d\n", ret); + ok(szDest[4] == 0, "szDest[4] not 0\n"); + ok(!memcmp(szDest, szLongText, 4*sizeof(WCHAR)), "szDest = %s\n", wine_dbgstr_w(szDest)); } static void test__wcsupr_s(void) diff --git a/dlls/msvcrt/wcs.c b/dlls/msvcrt/wcs.c index b6786ce..21cee47 100644 --- a/dlls/msvcrt/wcs.c +++ b/dlls/msvcrt/wcs.c @@ -1230,6 +1230,7 @@ INT CDECL MSVCRT_wcsncpy_s( MSVCRT_wchar_t* wcDest, MSVCRT_size_t numElement, co MSVCRT_size_t count ) { MSVCRT_size_t size = 0; + INT ret = 0; if (!wcDest || !numElement) return MSVCRT_EINVAL; @@ -1242,8 +1243,12 @@ INT CDECL MSVCRT_wcsncpy_s( MSVCRT_wchar_t* wcDest, MSVCRT_size_t numElement, co } size = min(strlenW(wcSrc), count); - - if (size >= numElement) + if (count==MSVCRT__TRUNCATE && size>=numElement) + { + ret = MSVCRT_STRUNCATE; + size = numElement-1; + } + else if (size >= numElement) { return MSVCRT_ERANGE; } @@ -1251,7 +1256,7 @@ INT CDECL MSVCRT_wcsncpy_s( MSVCRT_wchar_t* wcDest, MSVCRT_size_t numElement, co memcpy( wcDest, wcSrc, size*sizeof(WCHAR) ); wcDest[size] = '\0'; - return 0; + return ret; } /******************************************************************

1 0

Ilya Basin : regedit: Fix buffer size when exporting binary values.
by Alexandre Julliard 12 Apr '12

12 Apr '12

Module: wine Branch: master Commit: f900ed1474b0fd05dbf0a11907d0bf0682fff979 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f900ed1474b0fd05dbf0a1190… Author: Ilya Basin <basinilya(a)gmail.com> Date: Thu Apr 12 18:10:59 2012 +0400 regedit: Fix buffer size when exporting binary values. --- programs/regedit/regproc.c | 9 +++++---- 1 files changed, 5 insertions(+), 4 deletions(-) diff --git a/programs/regedit/regproc.c b/programs/regedit/regproc.c index 1a87442..f82d793 100644 --- a/programs/regedit/regproc.c +++ b/programs/regedit/regproc.c @@ -1018,12 +1018,13 @@ static void REGPROC_export_binary(WCHAR **line_buf, DWORD *line_buf_size, DWORD /* - The 2 spaces that concat places at the start of the * line effectively reduce the space available for data. * - If the value name and hex prefix are very long - * ( > REG_FILE_HEX_LINE_LEN) then we may overestimate + * ( > REG_FILE_HEX_LINE_LEN) or *line_len divides + * without a remainder then we may overestimate * the needed number of lines by one. But that's ok. - * - The trailing linefeed takes the place of a comma so - * it's accounted for already. + * - The trailing '\r' takes the place of a comma so + * we only need to add 1 for the trailing '\n' */ - *line_len += *line_len / (REG_FILE_HEX_LINE_LEN - concat_prefix) * concat_len; + *line_len += *line_len / (REG_FILE_HEX_LINE_LEN - concat_prefix) * concat_len + 1; REGPROC_resize_char_buffer(line_buf, line_buf_size, *line_len); lstrcpyW(*line_buf + hex_pos, hex_prefix); if (value_size)

1 0

Huw Davies : wineps: Move the default resolution fallback to the ppd parser .
by Alexandre Julliard 12 Apr '12

12 Apr '12

Module: wine Branch: master Commit: 1d2cd06592da89b0a8401cc8a1d331181a8a8f7a URL: http://source.winehq.org/git/wine.git/?a=commit;h=1d2cd06592da89b0a8401cc8a… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 12 13:03:13 2012 +0100 wineps: Move the default resolution fallback to the ppd parser. --- dlls/wineps.drv/init.c | 5 ----- dlls/wineps.drv/ppd.c | 4 ++++ 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/dlls/wineps.drv/init.c b/dlls/wineps.drv/init.c index 0134bd6..584254c 100644 --- a/dlls/wineps.drv/init.c +++ b/dlls/wineps.drv/init.c @@ -707,11 +707,6 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) goto fail; } - /* Some gimp-print ppd files don't contain a DefaultResolution line - set it to 300 if it's not specified */ - if(pi->ppd->DefaultResolution == 0) - pi->ppd->DefaultResolution = 300; - if(using_default_devmode) { DWORD papersize; diff --git a/dlls/wineps.drv/ppd.c b/dlls/wineps.drv/ppd.c index c4dff21..456195d 100644 --- a/dlls/wineps.drv/ppd.c +++ b/dlls/wineps.drv/ppd.c @@ -634,6 +634,10 @@ PPD *PSDRV_ParsePPD(char *fname) list_init( &ppd->InputSlots ); list_init( &ppd->Duplexes ); + /* Some gimp-print ppd files don't contain a DefaultResolution line + so default to 300 */ + ppd->DefaultResolution = 300; + /* * The Windows PostScript drivers create the following "virtual bin" for * every PostScript printer

1 0

Huw Davies : wineps: Simplify the failure path.
by Alexandre Julliard 12 Apr '12

12 Apr '12

Module: wine Branch: master Commit: 92bce60f37b5e870d92a04cbe178d04d3e8f55dd URL: http://source.winehq.org/git/wine.git/?a=commit;h=92bce60f37b5e870d92a04cbe… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 12 13:03:12 2012 +0100 wineps: Simplify the failure path. --- dlls/wineps.drv/init.c | 23 +++++++++-------------- 1 files changed, 9 insertions(+), 14 deletions(-) diff --git a/dlls/wineps.drv/init.c b/dlls/wineps.drv/init.c index 2e8087b..0134bd6 100644 --- a/dlls/wineps.drv/init.c +++ b/dlls/wineps.drv/init.c @@ -608,7 +608,7 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) if (OpenPrinterW( pi->friendly_name, &hPrinter, NULL ) == 0) { ERR ("OpenPrinter failed with code %i\n", GetLastError ()); - goto cleanup; + goto fail; } len = WideCharToMultiByte( CP_ACP, 0, name, -1, NULL, 0, NULL, NULL ); @@ -616,7 +616,7 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) WideCharToMultiByte( CP_ACP, 0, name, -1, nameA, len, NULL, NULL ); pi->Devmode = get_devmode( hPrinter, name, &using_default_devmode ); - if (!pi->Devmode) goto closeprinter; + if (!pi->Devmode) goto fail; #ifdef SONAME_LIBCUPS if (cupshandle != (void*)-1) { @@ -681,7 +681,7 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) { res = ERROR_FILE_NOT_FOUND; ERR ("Error %i getting PPD file name for printer '%s'\n", res, debugstr_w(name)); - goto closeprinter; + goto fail; } ppdFileName = HeapAlloc( PSDRV_Heap, 0, strlen(data_dir) + strlen(filename) + 1 ); strcpy( ppdFileName, data_dir ); @@ -704,7 +704,7 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) if(!pi->ppd) { MESSAGE("Couldn't find PPD file '%s', expect a crash now!\n", ppdFileName); - goto closeprinter; + goto fail; } /* Some gimp-print ppd files don't contain a DefaultResolution line @@ -748,7 +748,7 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) TRACE ("No 'Paper Size' for printer '%s'\n", debugstr_w(name)); else { ERR ("GetPrinterDataA returned %i\n", res); - goto closeprinter; + goto fail; } /* Duplex is indicated by the setting of the DM_DUPLEX bit in dmFields. @@ -767,10 +767,6 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) pi->FontSubTable = load_font_sub_table( hPrinter, &pi->FontSubTableSize ); - ClosePrinter( hPrinter ); - - pi->Fonts = NULL; - LIST_FOR_EACH_ENTRY( font, &pi->ppd->InstalledFonts, FONTNAME, entry ) { afm = PSDRV_FindAFMinList(PSDRV_AFMFontList, font->Name); @@ -782,24 +778,23 @@ PRINTERINFO *PSDRV_FindPrinterInfo(LPCWSTR name) BOOL added; if (PSDRV_AddAFMtoList(&pi->Fonts, afm, &added) == FALSE) { PSDRV_FreeAFMList(pi->Fonts); - goto cleanup; + goto fail; } } } + ClosePrinter( hPrinter ); HeapFree( GetProcessHeap(), 0, nameA ); if (ppd) unlink(ppd); list_add_head( &printer_list, &pi->entry ); return pi; -closeprinter: - ClosePrinter(hPrinter); -cleanup: +fail: + if (hPrinter) ClosePrinter( hPrinter ); HeapFree(PSDRV_Heap, 0, ppdFileName); HeapFree(PSDRV_Heap, 0, pi->FontSubTable); HeapFree(PSDRV_Heap, 0, pi->friendly_name); HeapFree(PSDRV_Heap, 0, pi->Devmode); -fail: HeapFree(PSDRV_Heap, 0, pi); HeapFree( GetProcessHeap(), 0, nameA ); if (ppd) unlink(ppd);

1 0

← Newer
1
...
38
39
40
41
42
43
44
...
72
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits April 2012