wine-commits February 2011

wine-commits@winehq.org

2 participants
674 discussions

Austin English : shell32/tests: Don't check return values inside of if(0) ( LLVM/Clang).
by Alexandre Julliard 14 Feb '11

14 Feb '11

Module: wine Branch: master Commit: 8a3294e8b67b5313f4c408f96f2d14735ecdc038 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8a3294e8b67b5313f4c408f96… Author: Austin English <austinenglish(a)gmail.com> Date: Thu Feb 10 11:57:20 2011 -0800 shell32/tests: Don't check return values inside of if(0) (LLVM/Clang). --- dlls/shell32/tests/shlfolder.c | 43 +++++++++++++++++++-------------------- 1 files changed, 21 insertions(+), 22 deletions(-) diff --git a/dlls/shell32/tests/shlfolder.c b/dlls/shell32/tests/shlfolder.c index ea675b9..9dd6296 100644 --- a/dlls/shell32/tests/shlfolder.c +++ b/dlls/shell32/tests/shlfolder.c @@ -2369,7 +2369,7 @@ static void test_SHCreateShellItem(void) if(0) { /* Crashes under windows 7 */ - ret = pSHCreateItemFromParsingName(NULL, NULL, &IID_IShellItem, NULL); + pSHCreateItemFromParsingName(NULL, NULL, &IID_IShellItem, NULL); } shellitem = (void*)0xdeadbeef; @@ -2411,7 +2411,7 @@ static void test_SHCreateShellItem(void) if(0) { /* Crashes under win7 */ - ret = pSHCreateItemFromIDList(NULL, &IID_IShellItem, NULL); + pSHCreateItemFromIDList(NULL, &IID_IShellItem, NULL); } ret = pSHCreateItemFromIDList(NULL, &IID_IShellItem, (void**)&shellitem); @@ -2497,7 +2497,7 @@ static void test_SHGetNameFromIDList(void) if(0) { /* Crashes under win7 */ - hres = pSHGetNameFromIDList(NULL, 0, NULL); + pSHGetNameFromIDList(NULL, 0, NULL); } hres = pSHGetNameFromIDList(NULL, 0, &name_string); @@ -2639,7 +2639,7 @@ static void test_SHGetItemFromDataObject(void) if(0) { /* Crashes under win7 */ - hres = pSHGetItemFromDataObject(NULL, 0, &IID_IShellItem, NULL); + pSHGetItemFromDataObject(NULL, 0, &IID_IShellItem, NULL); } hres = pSHGetItemFromDataObject(NULL, 0, &IID_IShellItem, (void**)&psv); @@ -2830,9 +2830,9 @@ static void test_ShellItemCompare(void) if(0) { /* Crashes on native (win7, winxp) */ - hr = IShellItem_Compare(psi_a, NULL, 0, NULL); - hr = IShellItem_Compare(psi_a, psi_b, 0, NULL); - hr = IShellItem_Compare(psi_a, NULL, 0, &order); + IShellItem_Compare(psi_a, NULL, 0, NULL); + IShellItem_Compare(psi_a, psi_b, 0, NULL); + IShellItem_Compare(psi_a, NULL, 0, &order); } /* Basics */ @@ -3204,9 +3204,9 @@ static void test_SHGetItemFromObject(void) if(0) { /* Crashes with Windows 7 */ - hres = pSHGetItemFromObject((IUnknown*)psfdesktop, &IID_IUnknown, NULL); - hres = pSHGetItemFromObject(NULL, &IID_IUnknown, NULL); - hres = pSHGetItemFromObject((IUnknown*)psfdesktop, NULL, (void**)&punk); + pSHGetItemFromObject((IUnknown*)psfdesktop, &IID_IUnknown, NULL); + pSHGetItemFromObject(NULL, &IID_IUnknown, NULL); + pSHGetItemFromObject((IUnknown*)psfdesktop, NULL, (void**)&punk); } hres = pSHGetItemFromObject(NULL, &IID_IUnknown, (void**)&punk); @@ -3340,7 +3340,7 @@ static void test_SHCreateShellItemArray(void) if(0) { /* Crashes in Windows 7 */ - hr = IShellItemArray_GetCount(psia, NULL); + IShellItemArray_GetCount(psia, NULL); } IShellItemArray_GetCount(psia, &numitems); @@ -3384,9 +3384,9 @@ static void test_SHCreateShellItemArray(void) if(0) { /* Crashes under Windows 7 */ - hr = pSHCreateShellItemArrayFromShellItem(NULL, &IID_IShellItemArray, NULL); - hr = pSHCreateShellItemArrayFromShellItem(NULL, &IID_IShellItemArray, (void**)&psia); - hr = pSHCreateShellItemArrayFromShellItem(psi, &IID_IShellItemArray, NULL); + pSHCreateShellItemArrayFromShellItem(NULL, &IID_IShellItemArray, NULL); + pSHCreateShellItemArrayFromShellItem(NULL, &IID_IShellItemArray, (void**)&psia); + pSHCreateShellItemArrayFromShellItem(psi, &IID_IShellItemArray, NULL); } hr = pSHCreateItemFromIDList(pidl_testdir, &IID_IShellItem, (void**)&psi); @@ -3437,7 +3437,7 @@ static void test_SHCreateShellItemArray(void) if(0) { /* Crashes under Windows 7 */ - hr = pSHCreateShellItemArrayFromDataObject(NULL, &IID_IShellItemArray, NULL); + pSHCreateShellItemArrayFromDataObject(NULL, &IID_IShellItemArray, NULL); } hr = pSHCreateShellItemArrayFromDataObject(NULL, &IID_IShellItemArray, (void**)&psia); ok(hr == E_INVALIDARG, "Got 0x%08x\n", hr); @@ -3548,8 +3548,8 @@ static void test_ShellItemBindToHandler(void) if(0) { /* Crashes under Windows 7 */ - hr = IShellItem_BindToHandler(psi, NULL, NULL, NULL, NULL); - hr = IShellItem_BindToHandler(psi, NULL, &IID_IUnknown, &IID_IUnknown, NULL); + IShellItem_BindToHandler(psi, NULL, NULL, NULL, NULL); + IShellItem_BindToHandler(psi, NULL, &IID_IUnknown, &IID_IUnknown, NULL); } hr = IShellItem_BindToHandler(psi, NULL, &IID_IUnknown, &IID_IUnknown, (void**)&punk); ok(hr == MK_E_NOOBJECT, "Got 0x%08x\n", hr); @@ -3702,8 +3702,7 @@ void test_ShellItemGetAttributes(void) if(0) { /* Crashes on native (Win 7) */ - hr = IShellItem_GetAttributes(psi, 0, NULL); - ok(hr == S_OK, "Got 0x%08x\n", hr); + IShellItem_GetAttributes(psi, 0, NULL); } /* Test GetAttributes on the desktop folder. */ @@ -3733,9 +3732,9 @@ static void test_SHParseDisplayName(void) if (0) { /* crashes on native */ - hr = pSHParseDisplayName(NULL, NULL, NULL, 0, NULL); + pSHParseDisplayName(NULL, NULL, NULL, 0, NULL); nameW[0] = 0; - hr = pSHParseDisplayName(nameW, NULL, NULL, 0, NULL); + pSHParseDisplayName(nameW, NULL, NULL, 0, NULL); } pidl1 = (LPITEMIDLIST)0xdeadbeef; @@ -3792,7 +3791,7 @@ static void test_desktop_IPersist(void) if (0) { /* crashes on native */ - hr = IPersist_GetClassID(persist, NULL); + IPersist_GetClassID(persist, NULL); } memset(&clsid, 0, sizeof(clsid)); hr = IPersist_GetClassID(persist, &clsid);

1 0

Austin English : shell32/tests: Make sure return values are used (LLVM/ Clang).
by Alexandre Julliard 14 Feb '11

14 Feb '11

Module: wine Branch: master Commit: c2671133dd9505e28453ec9f7909698da0bee5a7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c2671133dd9505e28453ec9f7… Author: Austin English <austinenglish(a)gmail.com> Date: Thu Feb 10 13:28:24 2011 -0800 shell32/tests: Make sure return values are used (LLVM/Clang). --- dlls/shell32/tests/shlfileop.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/shell32/tests/shlfileop.c b/dlls/shell32/tests/shlfileop.c index 810a745..b2d778c 100644 --- a/dlls/shell32/tests/shlfileop.c +++ b/dlls/shell32/tests/shlfileop.c @@ -2058,6 +2058,7 @@ static void test_sh_create_dir(void) ok(ERROR_ALREADY_EXISTS == ret, "SHCreateDirectoryEx should fail to create existing directory, ret = %d\n", ret); ret = pSHCreateDirectoryExA(NULL, "c:\\testdir3", NULL); + ok(ERROR_SUCCESS == ret, "SHCreateDirectoryEx failed to create directory, ret = %d\n", ret); ok(file_exists("c:\\testdir3"), "The directory is not created\n"); }

1 0

Austin English : shell32/tests: Don't check return values inside of if(0) ( LLVM/Clang).
by Alexandre Julliard 14 Feb '11

14 Feb '11

Module: wine Branch: master Commit: df846e89ebb2c5d7b863b760e7d16d41ee0e21a0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=df846e89ebb2c5d7b863b760e… Author: Austin English <austinenglish(a)gmail.com> Date: Thu Feb 10 13:28:20 2011 -0800 shell32/tests: Don't check return values inside of if(0) (LLVM/Clang). --- dlls/shell32/tests/shfldr_special.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/shell32/tests/shfldr_special.c b/dlls/shell32/tests/shfldr_special.c index 2374d7c..cfb31be 100644 --- a/dlls/shell32/tests/shfldr_special.c +++ b/dlls/shell32/tests/shfldr_special.c @@ -150,9 +150,9 @@ static void test_printers_folder(void) if (0) { /* crashes on XP */ - hr = IShellFolder2_GetDetailsOf(folder, NULL, 0, NULL); - hr = IShellFolder2_GetDefaultColumnState(folder, 0, NULL); - hr = IPersistFolder2_GetCurFolder(pf, NULL); + IShellFolder2_GetDetailsOf(folder, NULL, 0, NULL); + IShellFolder2_GetDefaultColumnState(folder, 0, NULL); + IPersistFolder2_GetCurFolder(pf, NULL); } /* 5 columns defined */

1 0

Austin English : shell32/tests: Don't check return values inside of if(0) ( LLVM/Clang).
by Alexandre Julliard 14 Feb '11

14 Feb '11

Module: wine Branch: master Commit: d6cd0e999e6cdf54b8d45d408be1cae11a01a940 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d6cd0e999e6cdf54b8d45d408… Author: Austin English <austinenglish(a)gmail.com> Date: Thu Feb 10 13:28:17 2011 -0800 shell32/tests: Don't check return values inside of if(0) (LLVM/Clang). --- dlls/shell32/tests/shellpath.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/shell32/tests/shellpath.c b/dlls/shell32/tests/shellpath.c index 2a86eac..d81ce80 100644 --- a/dlls/shell32/tests/shellpath.c +++ b/dlls/shell32/tests/shellpath.c @@ -358,7 +358,7 @@ static void test_parameters(void) BOOL ret; if (0) - ret = pSHGetSpecialFolderPathA(NULL, NULL, CSIDL_BITBUCKET, FALSE); + pSHGetSpecialFolderPathA(NULL, NULL, CSIDL_BITBUCKET, FALSE); /* odd but true: calling with a NULL path still succeeds if it's a real * dir (on some windows platform). on winME it generates exception.

1 0

Austin English : shell32/tests: Make sure return values are used (LLVM/ Clang).
by Alexandre Julliard 14 Feb '11

14 Feb '11

Module: wine Branch: master Commit: 75f8de0d4f7fbd0b7d99e96bcd2730cb2bec83a8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=75f8de0d4f7fbd0b7d99e96bc… Author: Austin English <austinenglish(a)gmail.com> Date: Thu Feb 10 13:28:13 2011 -0800 shell32/tests: Make sure return values are used (LLVM/Clang). --- dlls/shell32/tests/shelllink.c | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/shell32/tests/shelllink.c b/dlls/shell32/tests/shelllink.c index bdcffde..d28cff0 100644 --- a/dlls/shell32/tests/shelllink.c +++ b/dlls/shell32/tests/shelllink.c @@ -431,7 +431,7 @@ void create_lnk_(int line, const WCHAR* path, lnk_desc_t* desc, int save_fails) if (0) { /* crashes on XP */ - r = IPersistFile_GetCurFile(pf, NULL); + IPersistFile_GetCurFile(pf, NULL); } /* test GetCurFile before ::Save */ @@ -760,6 +760,8 @@ static void test_load_save(void) check_lnk(lnkfile, &desc, 0x0); r=pGetShortPathNameA(mydir, mypath, sizeof(mypath)); + ok(r<sizeof(mypath), "GetShortPathName failed (%d), err %d\n", r, GetLastError()); + strcpy(realpath, mypath); strcat(realpath, "\\test.txt"); strcat(mypath, "\\\\test.txt"); @@ -809,6 +811,7 @@ static void test_load_save(void) check_lnk(lnkfile, &desc, 0x4); r = DeleteFileA(realpath); + ok(r, "failed to delete file %s (%d)\n", realpath, GetLastError()); /* FIXME: Also test saving a .lnk pointing to a pidl that cannot be * represented as a path. @@ -885,7 +888,7 @@ static void test_datalink(void) if (0) { /* the following crashes */ - r = IShellLinkDataList_GetFlags( dl, NULL ); + IShellLinkDataList_GetFlags( dl, NULL ); } flags = 0;

1 0

Rico Schüller : d3dcompiler: Implement ID3DShaderReflection::GetResourceBindingDescByName().
by Alexandre Julliard 14 Feb '11

14 Feb '11

Module: wine Branch: master Commit: cacff4b3a0e49d2e47f87ae8ec5fb88096d288cb URL: http://source.winehq.org/git/wine.git/?a=commit;h=cacff4b3a0e49d2e47f87ae8e… Author: Rico Schüller <kgbricola(a)web.de> Date: Thu Feb 10 20:06:12 2011 +0100 d3dcompiler: Implement ID3DShaderReflection::GetResourceBindingDescByName(). --- dlls/d3dcompiler_43/reflection.c | 27 +++++++++++++++++++++++++-- 1 files changed, 25 insertions(+), 2 deletions(-) diff --git a/dlls/d3dcompiler_43/reflection.c b/dlls/d3dcompiler_43/reflection.c index f3646e8..e1ba543 100644 --- a/dlls/d3dcompiler_43/reflection.c +++ b/dlls/d3dcompiler_43/reflection.c @@ -296,9 +296,32 @@ static struct ID3D11ShaderReflectionVariable * STDMETHODCALLTYPE d3dcompiler_sha static HRESULT STDMETHODCALLTYPE d3dcompiler_shader_reflection_GetResourceBindingDescByName( ID3D11ShaderReflection *iface, LPCSTR name, D3D11_SHADER_INPUT_BIND_DESC *desc) { - FIXME("iface %p, name %s, desc %p stub!\n", iface, name, desc); + struct d3dcompiler_shader_reflection *This = impl_from_ID3D11ShaderReflection(iface); + unsigned int i; - return E_NOTIMPL; + TRACE("iface %p, name %s, desc %p\n", iface, debugstr_a(name), desc); + + if (!desc || !name) + { + WARN("Invalid argument specified\n"); + return E_INVALIDARG; + } + + for (i = 0; i < This->bound_resource_count; ++i) + { + D3D11_SHADER_INPUT_BIND_DESC *d = &This->bound_resources[i]; + + if (!strcmp(d->Name, name)) + { + TRACE("Returning D3D11_SHADER_INPUT_BIND_DESC %p.\n", d); + *desc = *d; + return S_OK; + } + } + + WARN("Invalid name specified\n"); + + return E_INVALIDARG; } static UINT STDMETHODCALLTYPE d3dcompiler_shader_reflection_GetMovInstructionCount(

1 0

Rico Schüller : d3dcompiler: Implement ID3DShaderReflection::GetResourceBindingDesc().
by Alexandre Julliard 14 Feb '11

14 Feb '11

Module: wine Branch: master Commit: 162233eb6384b4c553585217ec620c4ba424a666 URL: http://source.winehq.org/git/wine.git/?a=commit;h=162233eb6384b4c553585217e… Author: Rico Schüller <kgbricola(a)web.de> Date: Thu Feb 10 20:06:07 2011 +0100 d3dcompiler: Implement ID3DShaderReflection::GetResourceBindingDesc(). --- dlls/d3dcompiler_43/reflection.c | 14 ++++++++++++-- 1 files changed, 12 insertions(+), 2 deletions(-) diff --git a/dlls/d3dcompiler_43/reflection.c b/dlls/d3dcompiler_43/reflection.c index 9677075..f3646e8 100644 --- a/dlls/d3dcompiler_43/reflection.c +++ b/dlls/d3dcompiler_43/reflection.c @@ -216,9 +216,19 @@ static struct ID3D11ShaderReflectionConstantBuffer * STDMETHODCALLTYPE d3dcompil static HRESULT STDMETHODCALLTYPE d3dcompiler_shader_reflection_GetResourceBindingDesc( ID3D11ShaderReflection *iface, UINT index, D3D11_SHADER_INPUT_BIND_DESC *desc) { - FIXME("iface %p, index %u, desc %p stub!\n", iface, index, desc); + struct d3dcompiler_shader_reflection *This = impl_from_ID3D11ShaderReflection(iface); - return E_NOTIMPL; + TRACE("iface %p, index %u, desc %p\n", iface, index, desc); + + if (!desc || index >= This->bound_resource_count) + { + WARN("Invalid argument specified\n"); + return E_INVALIDARG; + } + + *desc = This->bound_resources[index]; + + return S_OK; } static HRESULT STDMETHODCALLTYPE d3dcompiler_shader_reflection_GetInputParameterDesc(

1 0

Rico Schüller : d3dcompiler: Parse bound resources.
by Alexandre Julliard 14 Feb '11

14 Feb '11

Module: wine Branch: master Commit: 5ad824a1780a0d746250e21209dad6ac8268d3f0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5ad824a1780a0d746250e2120… Author: Rico Schüller <kgbricola(a)web.de> Date: Thu Feb 10 20:05:59 2011 +0100 d3dcompiler: Parse bound resources. --- dlls/d3dcompiler_43/d3dcompiler_private.h | 3 + dlls/d3dcompiler_43/reflection.c | 101 +++++++++++++++++++++++++--- 2 files changed, 93 insertions(+), 11 deletions(-) diff --git a/dlls/d3dcompiler_43/d3dcompiler_private.h b/dlls/d3dcompiler_43/d3dcompiler_private.h index d69b88c..bc99322 100644 --- a/dlls/d3dcompiler_43/d3dcompiler_private.h +++ b/dlls/d3dcompiler_43/d3dcompiler_private.h @@ -70,6 +70,7 @@ struct d3dcompiler_shader_reflection ID3D11ShaderReflection ID3D11ShaderReflection_iface; LONG refcount; + DWORD target; char *creator; UINT flags; UINT version; @@ -106,6 +107,8 @@ struct d3dcompiler_shader_reflection struct d3dcompiler_shader_signature *isgn; struct d3dcompiler_shader_signature *osgn; struct d3dcompiler_shader_signature *pcsg; + char *resource_string; + D3D11_SHADER_INPUT_BIND_DESC *bound_resources; }; /* reflection handling */ diff --git a/dlls/d3dcompiler_43/reflection.c b/dlls/d3dcompiler_43/reflection.c index 96adb04..9677075 100644 --- a/dlls/d3dcompiler_43/reflection.c +++ b/dlls/d3dcompiler_43/reflection.c @@ -85,6 +85,8 @@ static void reflection_cleanup(struct d3dcompiler_shader_reflection *ref) HeapFree(GetProcessHeap(), 0, ref->pcsg); } + HeapFree(GetProcessHeap(), 0, ref->bound_resources); + HeapFree(GetProcessHeap(), 0, ref->resource_string); HeapFree(GetProcessHeap(), 0, ref->creator); } @@ -506,42 +508,119 @@ static HRESULT d3dcompiler_parse_rdef(struct d3dcompiler_shader_reflection *r, c { const char *ptr = data; DWORD size = data_size >> 2; - DWORD offset; + DWORD offset, cbuffer_offset, resource_offset, creator_offset; + unsigned int i; + unsigned int string_data_offset; + unsigned int string_data_size; + char *string_data = NULL, *creator = NULL; + D3D11_SHADER_INPUT_BIND_DESC *bound_resources = NULL; + HRESULT hr; TRACE("Size %u\n", size); read_dword(&ptr, &r->constant_buffer_count); TRACE("Constant buffer count: %u\n", r->constant_buffer_count); - read_dword(&ptr, &offset); - TRACE("Constant buffer offset: %x\n", offset); + read_dword(&ptr, &cbuffer_offset); + TRACE("Constant buffer offset: %#x\n", cbuffer_offset); read_dword(&ptr, &r->bound_resource_count); TRACE("Bound resource count: %u\n", r->bound_resource_count); - read_dword(&ptr, &offset); - TRACE("Bound resource offset: %x\n", offset); + read_dword(&ptr, &resource_offset); + TRACE("Bound resource offset: %#x\n", resource_offset); - skip_dword_unknown(&ptr, 1); + read_dword(&ptr, &r->target); + TRACE("Target: %#x\n", r->target); read_dword(&ptr, &r->flags); TRACE("Flags: %u\n", r->flags); - read_dword(&ptr, &offset); - TRACE("Creator at offset %#x.\n", offset); + read_dword(&ptr, &creator_offset); + TRACE("Creator at offset %#x.\n", creator_offset); - if (!copy_name(data + offset, &r->creator)) + if (!copy_name(data + creator_offset, &creator)) { ERR("Failed to copy name.\n"); return E_OUTOFMEMORY; } - TRACE("Creator: %s.\n", debugstr_a(r->creator)); + TRACE("Creator: %s.\n", debugstr_a(creator)); + + /* todo: Parse RD11 */ + if ((r->target & 0x0000ffff) >= 0x500) + { + skip_dword_unknown(&ptr, 8); + } + + if (r->bound_resource_count) + { + /* 8 for each bind desc */ + string_data_offset = resource_offset + r->bound_resource_count * 8 * sizeof(DWORD); + string_data_size = (cbuffer_offset ? cbuffer_offset : creator_offset) - string_data_offset; + + string_data = HeapAlloc(GetProcessHeap(), 0, string_data_size); + if (!string_data) + { + ERR("Failed to allocate string data memory.\n"); + hr = E_OUTOFMEMORY; + goto err_out; + } + memcpy(string_data, data + string_data_offset, string_data_size); + + bound_resources = HeapAlloc(GetProcessHeap(), 0, r->bound_resource_count * sizeof(*bound_resources)); + if (!bound_resources) + { + ERR("Failed to allocate resources memory.\n"); + hr = E_OUTOFMEMORY; + goto err_out; + } - /* todo: Parse D3D11_SHADER_INPUT_BIND_DESC Structure */ + ptr = data + resource_offset; + for (i = 0; i < r->bound_resource_count; i++) + { + D3D11_SHADER_INPUT_BIND_DESC *desc = &bound_resources[i]; + + read_dword(&ptr, &offset); + desc->Name = string_data + (offset - string_data_offset); + TRACE("Input bind Name: %s\n", debugstr_a(desc->Name)); + + read_dword(&ptr, &desc->Type); + TRACE("Input bind Type: %#x\n", desc->Type); + + read_dword(&ptr, &desc->ReturnType); + TRACE("Input bind ReturnType: %#x\n", desc->ReturnType); + + read_dword(&ptr, &desc->Dimension); + TRACE("Input bind Dimension: %#x\n", desc->Dimension); + + read_dword(&ptr, &desc->NumSamples); + TRACE("Input bind NumSamples: %u\n", desc->NumSamples); + + read_dword(&ptr, &desc->BindPoint); + TRACE("Input bind BindPoint: %u\n", desc->BindPoint); + + read_dword(&ptr, &desc->BindCount); + TRACE("Input bind BindCount: %u\n", desc->BindCount); + + read_dword(&ptr, &desc->uFlags); + TRACE("Input bind uFlags: %u\n", desc->uFlags); + } + } /* todo: Parse Constant buffers */ + r->creator = creator; + r->resource_string = string_data; + r->bound_resources = bound_resources; + return S_OK; + +err_out: + HeapFree(GetProcessHeap(), 0, bound_resources); + HeapFree(GetProcessHeap(), 0, string_data); + HeapFree(GetProcessHeap(), 0, creator); + + return hr; } HRESULT d3dcompiler_parse_signature(struct d3dcompiler_shader_signature *s, struct dxbc_section *section)

1 0

Max TenEyck Woodbury : c2man: Add header' s subdirectory and a link to its source file in the 'IMPLEMENTATION' notes.
by Alexandre Julliard 14 Feb '11

14 Feb '11

Module: wine Branch: master Commit: de6a2528494a5db0a3e37bab7a334244c429732d URL: http://source.winehq.org/git/wine.git/?a=commit;h=de6a2528494a5db0a3e37bab7… Author: Max TenEyck Woodbury <max(a)mtew.isa-geek.net> Date: Wed Feb 9 20:07:23 2011 -0500 c2man: Add header's subdirectory and a link to its source file in the 'IMPLEMENTATION' notes. --- tools/c2man.pl | 11 +++++++---- 1 files changed, 7 insertions(+), 4 deletions(-) diff --git a/tools/c2man.pl b/tools/c2man.pl index 984563d..db9f25f 100755 --- a/tools/c2man.pl +++ b/tools/c2man.pl @@ -860,7 +860,8 @@ sub process_comment($) $tmp =~ s/\n.*//g; if ($tmp ne "") { - $h_file = `basename $tmp`; + $h_file = "$tmp"; + $h_file =~ s|^.*/\./||; } } } @@ -874,7 +875,8 @@ sub process_comment($) $tmp =~ s/\n.*//g; if ($tmp ne "") { - $h_file = `basename $tmp`; + $h_file = "$tmp"; + $h_file =~ s|^.*/\./||; } } } @@ -886,7 +888,7 @@ sub process_comment($) } else { - $h_file = "Defined in \"".$h_file."\"."; + $h_file = "Declared in \"".$h_file."\"."; } } @@ -1611,6 +1613,7 @@ sub output_api_comment($) { # Link to the file in WineHQ cvs s/^(Implemented in \")(.+?)(\"\.)/$1$2$3 http:\/\/source.winehq.org\/source\/$2/g; + s/^(Declared in \")(.+?)(\"\.)/$1$2$3 http:\/\/source.winehq.org\/source\/include\/$2/g; } # Highlight strings s/(\".+?\")/$fmt[2]$1$fmt[3]/g; @@ -2265,7 +2268,7 @@ while(defined($_ = shift @ARGV)) /^L$/ && do { last; }; /^w$/ && do { @opt_spec_file_list = (@opt_spec_file_list, shift @ARGV); last; }; s/^I// && do { if ($_ ne ".") { - foreach my $include (`find $_ -type d ! -name tests`) { + foreach my $include (`find $_/./ -type d ! -name tests`) { $include =~ s/\n//; $include = $include."/*.h"; $include =~ s/\/\//\//g;

1 0

Juan Lang : crypt32: Don't attempt to find the length of a NULL string ( valgrind).
by Alexandre Julliard 11 Feb '11

11 Feb '11

Module: wine Branch: master Commit: 9e6de30f8feb8eb0a5fbbfd88f34c7358f7d6e6b URL: http://source.winehq.org/git/wine.git/?a=commit;h=9e6de30f8feb8eb0a5fbbfd88… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Feb 10 08:52:21 2011 -0800 crypt32: Don't attempt to find the length of a NULL string (valgrind). --- dlls/crypt32/encode.c | 48 ++++++++++++++++++++++++++++++++++++------------ 1 files changed, 36 insertions(+), 12 deletions(-) diff --git a/dlls/crypt32/encode.c b/dlls/crypt32/encode.c index e14ff19..825eb44 100644 --- a/dlls/crypt32/encode.c +++ b/dlls/crypt32/encode.c @@ -1012,8 +1012,12 @@ static BOOL CRYPT_AsnEncodeUTF8String(const CERT_NAME_VALUE *value, LPCWSTR str = (LPCWSTR)value->Value.pbData; DWORD bytesNeeded, lenBytes, encodedLen, strLen; - strLen = value->Value.cbData ? value->Value.cbData / sizeof(WCHAR) : - strlenW(str); + if (value->Value.cbData) + strLen = value->Value.cbData / sizeof(WCHAR); + else if (str) + strLen = strlenW(str); + else + strLen = 0; encodedLen = WideCharToMultiByte(CP_UTF8, 0, str, strLen, NULL, 0, NULL, NULL); CRYPT_EncodeLen(encodedLen, NULL, &lenBytes); @@ -1992,8 +1996,12 @@ static BOOL CRYPT_AsnEncodeUnicodeStringCoerce(const CERT_NAME_VALUE *value, LPCWSTR str = (LPCWSTR)value->Value.pbData; DWORD bytesNeeded, lenBytes, encodedLen; - encodedLen = value->Value.cbData ? value->Value.cbData / sizeof(WCHAR) : - strlenW(str); + if (value->Value.cbData) + encodedLen = value->Value.cbData / sizeof(WCHAR); + else if (str) + encodedLen = strlenW(str); + else + encodedLen = 0; CRYPT_EncodeLen(encodedLen, NULL, &lenBytes); bytesNeeded = 1 + lenBytes + encodedLen; if (!pbEncoded) @@ -2025,8 +2033,12 @@ static BOOL CRYPT_AsnEncodeNumericString(const CERT_NAME_VALUE *value, LPCWSTR str = (LPCWSTR)value->Value.pbData; DWORD bytesNeeded, lenBytes, encodedLen; - encodedLen = value->Value.cbData ? value->Value.cbData / sizeof(WCHAR) : - strlenW(str); + if (value->Value.cbData) + encodedLen = value->Value.cbData / sizeof(WCHAR); + else if (str) + encodedLen = strlenW(str); + else + encodedLen = 0; CRYPT_EncodeLen(encodedLen, NULL, &lenBytes); bytesNeeded = 1 + lenBytes + encodedLen; if (!pbEncoded) @@ -2079,8 +2091,12 @@ static BOOL CRYPT_AsnEncodePrintableString(const CERT_NAME_VALUE *value, LPCWSTR str = (LPCWSTR)value->Value.pbData; DWORD bytesNeeded, lenBytes, encodedLen; - encodedLen = value->Value.cbData ? value->Value.cbData / sizeof(WCHAR) : - strlenW(str); + if (value->Value.cbData) + encodedLen = value->Value.cbData / sizeof(WCHAR); + else if (str) + encodedLen = strlenW(str); + else + encodedLen = 0; CRYPT_EncodeLen(encodedLen, NULL, &lenBytes); bytesNeeded = 1 + lenBytes + encodedLen; if (!pbEncoded) @@ -2126,8 +2142,12 @@ static BOOL CRYPT_AsnEncodeIA5String(const CERT_NAME_VALUE *value, LPCWSTR str = (LPCWSTR)value->Value.pbData; DWORD bytesNeeded, lenBytes, encodedLen; - encodedLen = value->Value.cbData ? value->Value.cbData / sizeof(WCHAR) : - strlenW(str); + if (value->Value.cbData) + encodedLen = value->Value.cbData / sizeof(WCHAR); + else if (str) + encodedLen = strlenW(str); + else + encodedLen = 0; CRYPT_EncodeLen(encodedLen, NULL, &lenBytes); bytesNeeded = 1 + lenBytes + encodedLen; if (!pbEncoded) @@ -2174,8 +2194,12 @@ static BOOL CRYPT_AsnEncodeUniversalString(const CERT_NAME_VALUE *value, DWORD bytesNeeded, lenBytes, strLen; /* FIXME: doesn't handle composite characters */ - strLen = value->Value.cbData ? value->Value.cbData / sizeof(WCHAR) : - strlenW(str); + if (value->Value.cbData) + strLen = value->Value.cbData / sizeof(WCHAR); + else if (str) + strLen = strlenW(str); + else + strLen = 0; CRYPT_EncodeLen(strLen * 4, NULL, &lenBytes); bytesNeeded = 1 + lenBytes + strLen * 4; if (!pbEncoded)

1 0

← Newer
1
...
36
37
38
39
40
41
42
...
68
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 February 2011