May 2012 - Wine-commits - list.winehq.org

Lucas Fialho Zawacki : Added portuguese translation for release 1.5.5
by Jeremy Newman 29 May '12

29 May '12

Module: website Branch: master Commit: a2000bb15017609b5157315db496b07c63954f0c URL: http://source.winehq.org/git/website.git/?a=commit;h=a2000bb15017609b515731… Author: Lucas Fialho Zawacki <lfzawacki(a)gmail.com> Date: Tue May 29 10:58:29 2012 -0300 Added portuguese translation for release 1.5.5 --- news/pt/2012052501.xml | 17 +++++++++++++++++ 1 files changed, 17 insertions(+), 0 deletions(-) diff --git a/news/pt/2012052501.xml b/news/pt/2012052501.xml new file mode 100644 index 0000000..439ed70 --- /dev/null +++ b/news/pt/2012052501.xml @@ -0,0 +1,17 @@ +<news> +<date>25 de Maio de 2012</date> +<title>Wine 1.5.5 lançado</title> +<body> + A versão de desenvolvimento 1.5.5 já está disponível. + <a href="{$root}/announce/1.5.5">Novidades</a> desta versão: +<ul> + <li>Suporte à instalação do Mono como um pacote adicional do Wine.</li> + <li>Dithering de pincéis na engine DIB.</li> + <li>Suporte à instalação do ambiente de execução do .NET 4.0.</li> + <li>Suporte a arquivos DDS no D3DX9.</li> + <li>Várias correções de bugs.</li> +</ul> +O código fonte está <a href="http://prdownloads.sourceforge.net/wine/wine-1.5.5.tar.bz2">disponível agora</a>. +Pacotes binários estão em processo de construção e em breve estarão em suas respectivas <a href="{$root}/download">sessões da área de downloads.</a>. +</body></news> +

1 0

Alexandre Julliard : Wine release 1.5.5
by Alexandre Julliard 25 May '12

25 May '12

Module: website Branch: master Commit: d1d9cc5691a5b4be738619e2c580513443e1108c URL: http://source.winehq.org/git/website.git/?a=commit;h=d1d9cc5691a5b4be738619… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri May 25 20:31:17 2012 +0200 Wine release 1.5.5 --- news/en/2012052501.xml | 16 ++++++++++++++++ 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/news/en/2012052501.xml b/news/en/2012052501.xml new file mode 100644 index 0000000..6e84d18 --- /dev/null +++ b/news/en/2012052501.xml @@ -0,0 +1,16 @@ +<news> +<date>May 25, 2012</date> +<title>Wine 1.5.5 Released</title> +<body> + The Wine development release 1.5.5 is now available. + <a href="{$root}/announce/1.5.5">What's new</a> in this release: +<ul> + <li>Support for installing Mono as a Wine add-on package.</li> + <li>Dithering of pattern brushes in the DIB engine.</li> + <li>Support for installing the .NET 4.0 runtime.</li> + <li>DDS files supported in D3DX9.</li> + <li>Various bug fixes.</li> +</ul> +The source is <a href="http://prdownloads.sourceforge.net/wine/wine-1.5.5.tar.bz2">available now</a>. +Binary packages are in the process of being built, and will appear soon at their respective <a href="{$root}/download">download locations</a>. +</body></news>

1 0

Alexandre Julliard : Release 1.5.5.
by Alexandre Julliard 25 May '12

25 May '12

Module: wine Branch: master Commit: d6bc74fc5f0727f12788aa76d286d74f28208589 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d6bc74fc5f0727f12788aa76d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri May 25 20:21:15 2012 +0200 Release 1.5.5. --- ANNOUNCE | 906 ++++++++++++++++++++++++++++++++++--------------------------- VERSION | 2 +- configure | 18 +- 3 files changed, 513 insertions(+), 413 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d6bc74fc5f0727f12788a…

1 0

Tag wine-1.5.5 : Alexandre Julliard : Release 1.5.5
by Alexandre Julliard 25 May '12

25 May '12

Module: wine Branch: refs/tags/wine-1.5.5 Tag: 351d36d9209d7d17bb7bf3c406a9c63c4662a53a URL: http://source.winehq.org/git/wine.git/?a=tag;h=351d36d9209d7d17bb7bf3c406a9… Tagger: Alexandre Julliard <julliard(a)winehq.org> Date: Fri May 25 20:21:16 2012 +0200 Release 1.5.5

1 0

Dmitry Timoshkov : windowscodecs: Add some tests for IFD metadata reader.
by Alexandre Julliard 25 May '12

25 May '12

Module: wine Branch: master Commit: 30fc8a553dd6aff5c1315af29d490860029f39b1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=30fc8a553dd6aff5c1315af29… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Fri May 25 19:14:05 2012 +0900 windowscodecs: Add some tests for IFD metadata reader. --- dlls/windowscodecs/tests/metadata.c | 154 +++++++++++++++++++++++++++++++++++ 1 files changed, 154 insertions(+), 0 deletions(-) diff --git a/dlls/windowscodecs/tests/metadata.c b/dlls/windowscodecs/tests/metadata.c index 09e26ca..d4249f42 100644 --- a/dlls/windowscodecs/tests/metadata.c +++ b/dlls/windowscodecs/tests/metadata.c @@ -1,5 +1,6 @@ /* * Copyright 2011 Vincent Povirk for CodeWeavers + * Copyright 2012 Dmitry Timoshkov * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -38,6 +39,47 @@ } \ } while (0) +#define IFD_SHORT 3 +#define IFD_LONG 4 +#define IFD_RATIONAL 5 + +#include "pshpack2.h" +struct IFD_entry +{ + SHORT id; + SHORT type; + ULONG length; + LONG value; +}; + +struct IFD_rational +{ + ULONG numerator; + ULONG denominator; +}; + +static const struct +{ + USHORT number_of_entries; + struct IFD_entry entry[6]; + ULONG next_IFD; + struct IFD_rational rational; +} IFD_data = +{ + 6, + { + { 0xfe, IFD_SHORT, 1, 1 }, + { 0x100, IFD_LONG, 1, 222 }, + { 0x101, IFD_LONG, 1, 333 }, + { 0x102, IFD_SHORT, 1, 24 }, + { 0x103, IFD_LONG, 1, 32773 }, + { 0x11a, IFD_RATIONAL, 1, sizeof(USHORT) + sizeof(struct IFD_entry) * 6 + sizeof(ULONG) } + }, + 0, + { 300, 1 } +}; +#include "poppack.h" + static const char metadata_unknown[] = "lalala"; static const char metadata_tEXt[] = { @@ -275,6 +317,117 @@ static void test_metadata_tEXt(void) IWICMetadataReader_Release(reader); } +static void test_metadata_IFD(void) +{ + static const struct test_data + { + ULONG type, id, value; + } td[6] = + { + { VT_UI2, 0xfe, 1 }, + { VT_UI4, 0x100, 222 }, + { VT_UI4, 0x101, 333 }, + { VT_UI2, 0x102, 24 }, + { VT_UI4, 0x103, 32773 }, + { VT_UI8, 0x11a, 300 } + }; + HRESULT hr; + IWICMetadataReader *reader; + IWICEnumMetadataItem *enumerator; + PROPVARIANT schema, id, value; + ULONG items_returned, count, i; + GUID format; + + PropVariantInit(&schema); + PropVariantInit(&id); + PropVariantInit(&value); + + hr = CoCreateInstance(&CLSID_WICIfdMetadataReader, NULL, CLSCTX_INPROC_SERVER, + &IID_IWICMetadataReader, (void**)&reader); + todo_wine ok(hr == S_OK, "CoCreateInstance error %#x\n", hr); + if (FAILED(hr)) return; + + hr = IWICMetadataReader_GetCount(reader, NULL); + ok(hr == E_INVALIDARG, "GetCount error %#x\n", hr); + + hr = IWICMetadataReader_GetCount(reader, &count); + ok(hr == S_OK, "GetCount error %#x\n", hr); + ok(count == 0, "unexpected count %u\n", count); + + load_stream((IUnknown*)reader, (const char *)&IFD_data, sizeof(IFD_data)); + + hr = IWICMetadataReader_GetCount(reader, &count); + ok(hr == S_OK, "GetCount error %#x\n", hr); + ok(count == 6, "unexpected count %u\n", count); + + hr = IWICMetadataReader_GetEnumerator(reader, NULL); + ok(hr == E_INVALIDARG, "GetEnumerator error %#x\n", hr); + + hr = IWICMetadataReader_GetEnumerator(reader, &enumerator); + ok(hr == S_OK, "GetEnumerator error %#x\n", hr); + + for (i = 0; i < count; i++) + { + hr = IWICEnumMetadataItem_Next(enumerator, 1, &schema, &id, &value, &items_returned); + ok(hr == S_OK, "Next error %#x\n", hr); + ok(items_returned == 1, "unexpected item count %u\n", items_returned); + + ok(schema.vt == VT_EMPTY, "%u: unexpected vt: %u\n", i, schema.vt); + ok(id.vt == VT_UI2, "%u: unexpected vt: %u\n", i, id.vt); + ok(U(id).uiVal == td[i].id, "%u: unexpected id: %#x\n", i, U(id).uiVal); + ok(value.vt == td[i].type, "%u: unexpected vt: %u\n", i, value.vt); + ok(U(value).ulVal == td[i].value, "%u: unexpected id: %u\n", i, U(value).ulVal); + + PropVariantClear(&schema); + PropVariantClear(&id); + PropVariantClear(&value); + } + + hr = IWICEnumMetadataItem_Next(enumerator, 1, &schema, &id, &value, &items_returned); + ok(hr == S_FALSE, "Next should fail\n"); + ok(items_returned == 0, "unexpected item count %u\n", items_returned); + + IWICEnumMetadataItem_Release(enumerator); + + hr = IWICMetadataReader_GetMetadataFormat(reader, &format); + ok(hr == S_OK, "GetMetadataFormat error %#x\n", hr); + ok(IsEqualGUID(&format, &GUID_MetadataFormatIfd), "unexpected format %s\n", debugstr_guid(&format)); + + hr = IWICMetadataReader_GetMetadataFormat(reader, NULL); + ok(hr == E_INVALIDARG, "GetMetadataFormat should fail\n"); + + hr = IWICMetadataReader_GetValueByIndex(reader, 0, NULL, NULL, NULL); + ok(hr == S_OK, "GetValueByIndex error %#x\n", hr); + + hr = IWICMetadataReader_GetValueByIndex(reader, count - 1, NULL, NULL, NULL); + ok(hr == S_OK, "GetValueByIndex error %#x\n", hr); + + hr = IWICMetadataReader_GetValueByIndex(reader, 0, &schema, NULL, NULL); + ok(hr == S_OK, "GetValueByIndex error %#x\n", hr); + ok(schema.vt == VT_EMPTY, "unexpected vt: %u\n", schema.vt); + + hr = IWICMetadataReader_GetValueByIndex(reader, count - 1, &schema, NULL, NULL); + ok(hr == S_OK, "GetValueByIndex error %#x\n", hr); + ok(schema.vt == VT_EMPTY, "unexpected vt: %u\n", schema.vt); + + hr = IWICMetadataReader_GetValueByIndex(reader, 0, NULL, &id, NULL); + ok(hr == S_OK, "GetValueByIndex error %#x\n", hr); + ok(id.vt == VT_UI2, "unexpected vt: %u\n", id.vt); + ok(U(id).uiVal == 0xfe, "unexpected id: %#x\n", U(id).uiVal); + PropVariantClear(&id); + + hr = IWICMetadataReader_GetValueByIndex(reader, 0, NULL, NULL, &value); + ok(hr == S_OK, "GetValueByIndex error %#x\n", hr); + ok(value.vt == VT_UI2, "unexpected vt: %u\n", value.vt); + ok(U(value).ulVal == 1, "unexpected id: %u\n", U(value).ulVal); + PropVariantClear(&value); + + hr = IWICMetadataReader_GetValueByIndex(reader, count, &schema, NULL, NULL); + ok(hr == E_INVALIDARG, "GetValueByIndex should fail\n"); + + IWICMetadataReader_Release(reader); +} + static void test_create_reader(void) { HRESULT hr; @@ -338,6 +491,7 @@ START_TEST(metadata) test_metadata_unknown(); test_metadata_tEXt(); + test_metadata_IFD(); test_create_reader(); CoUninitialize();

1 0

Aric Stewart : usp10: Check bounds before checking value in array.
by Alexandre Julliard 25 May '12

25 May '12

Module: wine Branch: master Commit: 1f4554d141774e948a29731db72ce5758ddad2f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1f4554d141774e948a29731db… Author: Aric Stewart <aric(a)codeweavers.com> Date: Fri May 25 07:55:19 2012 -0500 usp10: Check bounds before checking value in array. --- dlls/usp10/shape.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/usp10/shape.c b/dlls/usp10/shape.c index a039504..d582fc7 100644 --- a/dlls/usp10/shape.c +++ b/dlls/usp10/shape.c @@ -628,9 +628,9 @@ static void UpdateClustersFromGlyphProp(const int cGlyphs, const int cChars, WOR if (pwLogClust[j] == i) { int k = j; - while (!pGlyphProp[pwLogClust[k]].sva.fClusterStart && k >= 0 && k <cChars) + while (k >= 0 && k <cChars && !pGlyphProp[pwLogClust[k]].sva.fClusterStart) k-=1; - if (pGlyphProp[pwLogClust[k]].sva.fClusterStart) + if (k >= 0 && k <cChars && pGlyphProp[pwLogClust[k]].sva.fClusterStart) pwLogClust[j] = pwLogClust[k]; } }

1 0

Henri Verbeet : d3d9: Get rid of the d3dType field from d3d_dtype_lookup.
by Alexandre Julliard 25 May '12

25 May '12

Module: wine Branch: master Commit: 51024a078f4747f05527310d034f0f47c4e3cdd3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=51024a078f4747f05527310d0… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri May 25 16:10:27 2012 +0200 d3d9: Get rid of the d3dType field from d3d_dtype_lookup. --- dlls/d3d9/vertexdeclaration.c | 36 ++++++++++++++++++------------------ 1 files changed, 18 insertions(+), 18 deletions(-) diff --git a/dlls/d3d9/vertexdeclaration.c b/dlls/d3d9/vertexdeclaration.c index 421f099..679f010 100644 --- a/dlls/d3d9/vertexdeclaration.c +++ b/dlls/d3d9/vertexdeclaration.c @@ -26,30 +26,30 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3d9); static const struct { - D3DDECLTYPE d3dType; enum wined3d_format_id format; int size; int typesize; } d3d_dtype_lookup[] = { - {D3DDECLTYPE_FLOAT1, WINED3DFMT_R32_FLOAT, 1, sizeof(float)}, - {D3DDECLTYPE_FLOAT2, WINED3DFMT_R32G32_FLOAT, 2, sizeof(float)}, - {D3DDECLTYPE_FLOAT3, WINED3DFMT_R32G32B32_FLOAT, 3, sizeof(float)}, - {D3DDECLTYPE_FLOAT4, WINED3DFMT_R32G32B32A32_FLOAT, 4, sizeof(float)}, - {D3DDECLTYPE_D3DCOLOR, WINED3DFMT_B8G8R8A8_UNORM, 4, sizeof(BYTE)}, - {D3DDECLTYPE_UBYTE4, WINED3DFMT_R8G8B8A8_UINT, 4, sizeof(BYTE)}, - {D3DDECLTYPE_SHORT2, WINED3DFMT_R16G16_SINT, 2, sizeof(short int)}, - {D3DDECLTYPE_SHORT4, WINED3DFMT_R16G16B16A16_SINT, 4, sizeof(short int)}, - {D3DDECLTYPE_UBYTE4N, WINED3DFMT_R8G8B8A8_UNORM, 4, sizeof(BYTE)}, - {D3DDECLTYPE_SHORT2N, WINED3DFMT_R16G16_SNORM, 2, sizeof(short int)}, - {D3DDECLTYPE_SHORT4N, WINED3DFMT_R16G16B16A16_SNORM, 4, sizeof(short int)}, - {D3DDECLTYPE_USHORT2N, WINED3DFMT_R16G16_UNORM, 2, sizeof(short int)}, - {D3DDECLTYPE_USHORT4N, WINED3DFMT_R16G16B16A16_UNORM, 4, sizeof(short int)}, - {D3DDECLTYPE_UDEC3, WINED3DFMT_R10G10B10A2_UINT, 3, sizeof(short int)}, - {D3DDECLTYPE_DEC3N, WINED3DFMT_R10G10B10A2_SNORM, 3, sizeof(short int)}, - {D3DDECLTYPE_FLOAT16_2, WINED3DFMT_R16G16_FLOAT, 2, sizeof(short int)}, - {D3DDECLTYPE_FLOAT16_4, WINED3DFMT_R16G16B16A16_FLOAT, 4, sizeof(short int)}}; + /* D3DDECLTYPE_FLOAT1 */ {WINED3DFMT_R32_FLOAT, 1, sizeof(float)}, + /* D3DDECLTYPE_FLOAT2 */ {WINED3DFMT_R32G32_FLOAT, 2, sizeof(float)}, + /* D3DDECLTYPE_FLOAT3 */ {WINED3DFMT_R32G32B32_FLOAT, 3, sizeof(float)}, + /* D3DDECLTYPE_FLOAT4 */ {WINED3DFMT_R32G32B32A32_FLOAT, 4, sizeof(float)}, + /* D3DDECLTYPE_D3DCOLOR */ {WINED3DFMT_B8G8R8A8_UNORM, 4, sizeof(BYTE)}, + /* D3DDECLTYPE_UBYTE4 */ {WINED3DFMT_R8G8B8A8_UINT, 4, sizeof(BYTE)}, + /* D3DDECLTYPE_SHORT2 */ {WINED3DFMT_R16G16_SINT, 2, sizeof(short int)}, + /* D3DDECLTYPE_SHORT4 */ {WINED3DFMT_R16G16B16A16_SINT, 4, sizeof(short int)}, + /* D3DDECLTYPE_UBYTE4N */ {WINED3DFMT_R8G8B8A8_UNORM, 4, sizeof(BYTE)}, + /* D3DDECLTYPE_SHORT2N */ {WINED3DFMT_R16G16_SNORM, 2, sizeof(short int)}, + /* D3DDECLTYPE_SHORT4N */ {WINED3DFMT_R16G16B16A16_SNORM, 4, sizeof(short int)}, + /* D3DDECLTYPE_USHORT2N */ {WINED3DFMT_R16G16_UNORM, 2, sizeof(short int)}, + /* D3DDECLTYPE_USHORT4N */ {WINED3DFMT_R16G16B16A16_UNORM, 4, sizeof(short int)}, + /* D3DDECLTYPE_UDEC3 */ {WINED3DFMT_R10G10B10A2_UINT, 3, sizeof(short int)}, + /* D3DDECLTYPE_DEC3N */ {WINED3DFMT_R10G10B10A2_SNORM, 3, sizeof(short int)}, + /* D3DDECLTYPE_FLOAT16_2 */ {WINED3DFMT_R16G16_FLOAT, 2, sizeof(short int)}, + /* D3DDECLTYPE_FLOAT16_4 */ {WINED3DFMT_R16G16B16A16_FLOAT, 4, sizeof(short int)} +}; #define D3D_DECL_SIZE(type) d3d_dtype_lookup[type].size #define D3D_DECL_TYPESIZE(type) d3d_dtype_lookup[type].typesize

1 0

Henri Verbeet : d3d9: Get rid of the D3D_DECL_* macros.
by Alexandre Julliard 25 May '12

25 May '12

Module: wine Branch: master Commit: b5d64260e00ef98de12cecbfcbd5a2910538875b URL: http://source.winehq.org/git/wine.git/?a=commit;h=b5d64260e00ef98de12cecbfc… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri May 25 16:10:28 2012 +0200 d3d9: Get rid of the D3D_DECL_* macros. --- dlls/d3d9/vertexdeclaration.c | 10 ++++------ 1 files changed, 4 insertions(+), 6 deletions(-) diff --git a/dlls/d3d9/vertexdeclaration.c b/dlls/d3d9/vertexdeclaration.c index 679f010..2d31dd7 100644 --- a/dlls/d3d9/vertexdeclaration.c +++ b/dlls/d3d9/vertexdeclaration.c @@ -27,8 +27,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3d9); static const struct { enum wined3d_format_id format; - int size; - int typesize; + unsigned int component_count; + unsigned int component_size; } d3d_dtype_lookup[] = { @@ -51,9 +51,6 @@ d3d_dtype_lookup[] = /* D3DDECLTYPE_FLOAT16_4 */ {WINED3DFMT_R16G16B16A16_FLOAT, 4, sizeof(short int)} }; -#define D3D_DECL_SIZE(type) d3d_dtype_lookup[type].size -#define D3D_DECL_TYPESIZE(type) d3d_dtype_lookup[type].typesize - static inline IDirect3DVertexDeclaration9Impl *impl_from_IDirect3DVertexDeclaration9(IDirect3DVertexDeclaration9 *iface) { return CONTAINING_RECORD(iface, IDirect3DVertexDeclaration9Impl, IDirect3DVertexDeclaration9_iface); @@ -192,7 +189,8 @@ HRESULT vdecl_convert_fvf( elements[idx].Stream = 0; elements[idx].Method = D3DDECLMETHOD_DEFAULT; elements[idx].Offset = offset; - offset += D3D_DECL_SIZE(elements[idx].Type) * D3D_DECL_TYPESIZE(elements[idx].Type); + offset += d3d_dtype_lookup[elements[idx].Type].component_count + * d3d_dtype_lookup[elements[idx].Type].component_size; } *ppVertexElements = elements;

1 0

Henri Verbeet : d3dx9: Don't use HEAP_ZERO_MEMORY if we' re going to overwrite the memory anyway.
by Alexandre Julliard 25 May '12

25 May '12

Module: wine Branch: master Commit: 372984e053b0fc8a8e56d5f2209540b61fb2836b URL: http://source.winehq.org/git/wine.git/?a=commit;h=372984e053b0fc8a8e56d5f22… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri May 25 16:10:29 2012 +0200 d3dx9: Don't use HEAP_ZERO_MEMORY if we're going to overwrite the memory anyway. --- dlls/d3dx9_36/surface.c | 4 ++-- dlls/d3dx9_36/texture.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/d3dx9_36/surface.c b/dlls/d3dx9_36/surface.c index 821d51e..d490cef 100644 --- a/dlls/d3dx9_36/surface.c +++ b/dlls/d3dx9_36/surface.c @@ -656,7 +656,7 @@ HRESULT WINAPI D3DXGetImageInfoFromFileA(LPCSTR file, D3DXIMAGE_INFO *info) if( !file ) return D3DERR_INVALIDCALL; strlength = MultiByteToWideChar(CP_ACP, 0, file, -1, NULL, 0); - widename = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, strlength * sizeof(WCHAR)); + widename = HeapAlloc(GetProcessHeap(), 0, strlength * sizeof(*widename)); MultiByteToWideChar(CP_ACP, 0, file, -1, widename, strlength); hr = D3DXGetImageInfoFromFileW(widename, info); @@ -918,7 +918,7 @@ HRESULT WINAPI D3DXLoadSurfaceFromFileA(LPDIRECT3DSURFACE9 pDestSurface, if( !pSrcFile || !pDestSurface ) return D3DERR_INVALIDCALL; strlength = MultiByteToWideChar(CP_ACP, 0, pSrcFile, -1, NULL, 0); - pWidename = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, strlength * sizeof(WCHAR)); + pWidename = HeapAlloc(GetProcessHeap(), 0, strlength * sizeof(*pWidename)); MultiByteToWideChar(CP_ACP, 0, pSrcFile, -1, pWidename, strlength); hr = D3DXLoadSurfaceFromFileW(pDestSurface, pDestPalette, pDestRect, pWidename, pSrcRect, dwFilter, Colorkey, pSrcInfo); diff --git a/dlls/d3dx9_36/texture.c b/dlls/d3dx9_36/texture.c index 750025c..c17a327 100644 --- a/dlls/d3dx9_36/texture.c +++ b/dlls/d3dx9_36/texture.c @@ -746,7 +746,7 @@ HRESULT WINAPI D3DXCreateTextureFromFileExA(LPDIRECT3DDEVICE9 device, return D3DERR_INVALIDCALL; len = MultiByteToWideChar(CP_ACP, 0, srcfile, -1, NULL, 0); - widename = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, len * sizeof(WCHAR)); + widename = HeapAlloc(GetProcessHeap(), 0, len * sizeof(*widename)); MultiByteToWideChar(CP_ACP, 0, srcfile, -1, widename, len); hr = D3DXCreateTextureFromFileExW(device, widename, width, height, miplevels,

1 0

Henri Verbeet : d3d9: Add a separate function for d3d9 initialization.
by Alexandre Julliard 25 May '12

25 May '12

Module: wine Branch: master Commit: 5b2d1710434aa71fc9bd421f1836db710b21bf9e URL: http://source.winehq.org/git/wine.git/?a=commit;h=5b2d1710434aa71fc9bd421f1… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri May 25 16:10:30 2012 +0200 d3d9: Add a separate function for d3d9 initialization. --- dlls/d3d9/d3d9_main.c | 58 +++++++++++++++++++++++++++------------------ dlls/d3d9/d3d9_private.h | 14 +---------- dlls/d3d9/directx.c | 18 ++++++++++++- 3 files changed, 52 insertions(+), 38 deletions(-) diff --git a/dlls/d3d9/d3d9_main.c b/dlls/d3d9/d3d9_main.c index 6eed9b7..e3bb9f8 100644 --- a/dlls/d3d9/d3d9_main.c +++ b/dlls/d3d9/d3d9_main.c @@ -33,40 +33,52 @@ void WINAPI DebugSetMute(void) { /* nothing to do */ } -IDirect3D9* WINAPI DECLSPEC_HOTPATCH Direct3DCreate9(UINT SDKVersion) { - IDirect3D9Impl* object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(IDirect3D9Impl)); - - object->IDirect3D9Ex_iface.lpVtbl = &Direct3D9_Vtbl; - object->ref = 1; +IDirect3D9 * WINAPI DECLSPEC_HOTPATCH Direct3DCreate9(UINT sdk_version) +{ + IDirect3D9Impl *object; - wined3d_mutex_lock(); - object->WineD3D = wined3d_create(9, 0); - wined3d_mutex_unlock(); + TRACE("sdk_version %#x.\n", sdk_version); - TRACE("SDKVersion = %x, Created Direct3D object @ %p, WineObj @ %p\n", SDKVersion, object, object->WineD3D); + if (!(object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)))) + { + ERR("Failed to allocate d3d9 object memory.\n"); + return NULL; + } - if (!object->WineD3D) + if (!d3d9_init(object, FALSE)) { - HeapFree( GetProcessHeap(), 0, object ); - object = NULL; + WARN("Failed to initialize d3d9.\n"); + HeapFree(GetProcessHeap(), 0, object); + return NULL; } - return (IDirect3D9*) object; + + TRACE("Created d3d9 object %p.\n", object); + + return (IDirect3D9 *)&object->IDirect3D9Ex_iface; } -HRESULT WINAPI DECLSPEC_HOTPATCH Direct3DCreate9Ex(UINT SDKVersion, IDirect3D9Ex **direct3d9ex) { - IDirect3D9 *ret; - IDirect3D9Impl* object; +HRESULT WINAPI DECLSPEC_HOTPATCH Direct3DCreate9Ex(UINT sdk_version, IDirect3D9Ex **d3d9ex) +{ + IDirect3D9Impl *object; - TRACE("Calling Direct3DCreate9\n"); - ret = Direct3DCreate9(SDKVersion); - if(!ret) { - *direct3d9ex = NULL; + TRACE("sdk_version %#x, d3d9ex %p.\n", sdk_version, d3d9ex); + + if (!(object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*object)))) + { + ERR("Failed to allocate d3d9 object memory.\n"); + return E_OUTOFMEMORY; + } + + if (!d3d9_init(object, TRUE)) + { + WARN("Failed to initialize d3d9.\n"); + HeapFree(GetProcessHeap(), 0, object); return D3DERR_NOTAVAILABLE; } - object = (IDirect3D9Impl *) ret; - object->extended = TRUE; /* Enables QI for extended interfaces */ - *direct3d9ex = &object->IDirect3D9Ex_iface; + TRACE("Created d3d9 object %p.\n", object); + *d3d9ex = &object->IDirect3D9Ex_iface; + return D3D_OK; } diff --git a/dlls/d3d9/d3d9_private.h b/dlls/d3d9/d3d9_private.h index d814dd0..793c948 100644 --- a/dlls/d3d9/d3d9_private.h +++ b/dlls/d3d9/d3d9_private.h @@ -127,19 +127,6 @@ enum wined3d_format_id wined3dformat_from_d3dformat(D3DFORMAT format) DECLSPEC_H _pD3D9Caps->MaxVertexShader30InstructionSlots = _pWineCaps->MaxVertexShader30InstructionSlots; \ _pD3D9Caps->MaxPixelShader30InstructionSlots = _pWineCaps->MaxPixelShader30InstructionSlots; -/* =========================================================================== - D3D9 interfaces - =========================================================================== */ - -/* ---------- */ -/* IDirect3D9 */ -/* ---------- */ - -/***************************************************************************** - * Predeclare the interface implementation structures - */ -extern const IDirect3D9ExVtbl Direct3D9_Vtbl DECLSPEC_HIDDEN; - /***************************************************************************** * IDirect3D implementation structure */ @@ -155,6 +142,7 @@ typedef struct IDirect3D9Impl BOOL extended; } IDirect3D9Impl; +BOOL d3d9_init(IDirect3D9Impl *d3d9, BOOL extended) DECLSPEC_HIDDEN; void filter_caps(D3DCAPS9* pCaps) DECLSPEC_HIDDEN; struct fvf_declaration diff --git a/dlls/d3d9/directx.c b/dlls/d3d9/directx.c index d8a5d78..80b862d 100644 --- a/dlls/d3d9/directx.c +++ b/dlls/d3d9/directx.c @@ -551,8 +551,7 @@ static HRESULT WINAPI IDirect3D9ExImpl_GetAdapterLUID(IDirect3D9Ex *iface, UINT return hr; } - -const IDirect3D9ExVtbl Direct3D9_Vtbl = +static const struct IDirect3D9ExVtbl Direct3D9_Vtbl = { /* IUnknown */ IDirect3D9Impl_QueryInterface, @@ -581,3 +580,18 @@ const IDirect3D9ExVtbl Direct3D9_Vtbl = IDirect3D9ExImpl_GetAdapterLUID }; + +BOOL d3d9_init(IDirect3D9Impl *d3d9, BOOL extended) +{ + d3d9->IDirect3D9Ex_iface.lpVtbl = &Direct3D9_Vtbl; + d3d9->ref = 1; + + wined3d_mutex_lock(); + d3d9->WineD3D = wined3d_create(9, 0); + wined3d_mutex_unlock(); + if (!d3d9->WineD3D) + return FALSE; + d3d9->extended = extended; + + return TRUE; +}

1 0