wine-commits May 2012

wine-commits@winehq.org

2 participants
737 discussions

Jacek Caban : mshtml: Forward BSCallback::QueryService call to document.
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: 0847563dd410916f3a1095bff3afc2fb71981064 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0847563dd410916f3a1095bff… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed May 30 15:49:41 2012 +0200 mshtml: Forward BSCallback::QueryService call to document. --- dlls/mshtml/view.c | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/mshtml/view.c b/dlls/mshtml/view.c index 6701987..08c5332 100644 --- a/dlls/mshtml/view.c +++ b/dlls/mshtml/view.c @@ -953,8 +953,11 @@ static ULONG WINAPI WindowForBindingUI_Release(IWindowForBindingUI *iface) static HRESULT WINAPI WindowForBindingUI_GetWindow(IWindowForBindingUI *iface, REFGUID rguidReason, HWND *phwnd) { HTMLDocumentObj *This = impl_from_IWindowForBindingUI(iface); - FIXME("(%p)->(%s %p)\n", This, debugstr_guid(rguidReason), phwnd); - return E_NOTIMPL; + + TRACE("(%p)->(%s %p)\n", This, debugstr_guid(rguidReason), phwnd); + + *phwnd = This->hwnd; + return S_OK; } static const IWindowForBindingUIVtbl WindowForBindingUIVtbl = {

1 0

Jacek Caban : mshtml: Added support for BSCallback::QueryService( IID_IWindowForBindingUI).
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: 3c1e2132e5e2efc1ba80fd6cfb6a2715d19c253c URL: http://source.winehq.org/git/wine.git/?a=commit;h=3c1e2132e5e2efc1ba80fd6cf… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed May 30 15:49:28 2012 +0200 mshtml: Added support for BSCallback::QueryService(IID_IWindowForBindingUI). --- dlls/mshtml/navigate.c | 11 ++++++++++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/dlls/mshtml/navigate.c b/dlls/mshtml/navigate.c index 3697070..1979af3 100644 --- a/dlls/mshtml/navigate.c +++ b/dlls/mshtml/navigate.c @@ -582,7 +582,11 @@ static HRESULT WINAPI BSCServiceProvider_QueryService(IServiceProvider *iface, REFGUID guidService, REFIID riid, void **ppv) { BSCallback *This = impl_from_IServiceProvider(iface); + TRACE("(%p)->(%s %s %p)\n", This, debugstr_guid(guidService), debugstr_guid(riid), ppv); + + if(This->doc && IsEqualGUID(guidService, &IID_IWindowForBindingUI)) + return IServiceProvider_QueryService(&This->doc->basedoc.IServiceProvider_iface, guidService, riid, ppv); return E_NOINTERFACE; } @@ -721,6 +725,8 @@ HRESULT start_binding(HTMLWindow *window, HTMLDocumentNode *doc, BSCallback *bsc TRACE("(%p %p %p %p)\n", window, doc, bscallback, bctx); bscallback->doc = doc; + if(!doc && window) + bscallback->doc = window->doc; /* NOTE: IE7 calls IsSystemMoniker here*/ @@ -992,8 +998,11 @@ static HRESULT on_start_nsrequest(nsChannelBSC *This) list_remove(&This->bsc.entry); list_init(&This->bsc.entry); update_window_doc(This->window); - if(This->window->doc != This->bsc.doc) + if(This->window->doc != This->bsc.doc) { + if(This->bsc.doc) + list_remove(&This->bsc.entry); This->bsc.doc = This->window->doc; + } list_add_head(&This->bsc.doc->bindings, &This->bsc.entry); if(This->window->readystate != READYSTATE_LOADING) set_ready_state(This->window, READYSTATE_LOADING);

1 0

Jacek Caban : mshtml: Added IWindowForBindingUI stub implementation.
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: cf9d0d320c0603018d3c463f8daa69cbfd68a755 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cf9d0d320c0603018d3c463f8… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed May 30 15:49:15 2012 +0200 mshtml: Added IWindowForBindingUI stub implementation. --- dlls/mshtml/htmldoc.c | 2 + dlls/mshtml/mshtml_private.h | 3 ++ dlls/mshtml/service.c | 5 +++ dlls/mshtml/view.c | 56 ++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 66 insertions(+), 0 deletions(-) diff --git a/dlls/mshtml/htmldoc.c b/dlls/mshtml/htmldoc.c index 05eb664..a0ab79d 100644 --- a/dlls/mshtml/htmldoc.c +++ b/dlls/mshtml/htmldoc.c @@ -2428,6 +2428,8 @@ HRESULT HTMLDocument_Create(IUnknown *pUnkOuter, REFIID riid, void** ppvObject) doc->usermode = UNKNOWN_USERMODE; + init_binding_ui(doc); + hres = create_nscontainer(doc, &doc->nscontainer); if(FAILED(hres)) { ERR("Failed to init Gecko, returning CLASS_E_CLASSNOTAVAILABLE\n"); diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index 7eaaa57..3eb81b9 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -453,6 +453,8 @@ struct HTMLDocumentObj { ICustomDoc ICustomDoc_iface; ITargetContainer ITargetContainer_iface; + IWindowForBindingUI IWindowForBindingUI_iface; + LONG ref; NSContainer *nscontainer; @@ -657,6 +659,7 @@ void HTMLDocument_Service_Init(HTMLDocument*) DECLSPEC_HIDDEN; void HTMLDocument_Hlink_Init(HTMLDocument*) DECLSPEC_HIDDEN; void TargetContainer_Init(HTMLDocumentObj*) DECLSPEC_HIDDEN; +void init_binding_ui(HTMLDocumentObj*) DECLSPEC_HIDDEN; void HTMLDocumentNode_SecMgr_Init(HTMLDocumentNode*) DECLSPEC_HIDDEN; diff --git a/dlls/mshtml/service.c b/dlls/mshtml/service.c index 9cafbf0..56744e6 100644 --- a/dlls/mshtml/service.c +++ b/dlls/mshtml/service.c @@ -254,6 +254,11 @@ static HRESULT WINAPI ServiceProvider_QueryService(IServiceProvider *iface, REFG return IHTMLDocument2_QueryInterface(&This->IHTMLDocument2_iface, riid, ppv); } + if(IsEqualGUID(&IID_IWindowForBindingUI, guidService)) { + TRACE("IID_IWindowForBindingUI\n"); + return IWindowForBindingUI_QueryInterface(&This->doc_obj->IWindowForBindingUI_iface, riid, ppv); + } + TRACE("(%p)->(%s %s %p)\n", This, debugstr_guid(guidService), debugstr_guid(riid), ppv); if(This->doc_obj->client) { diff --git a/dlls/mshtml/view.c b/dlls/mshtml/view.c index 3852046..6701987 100644 --- a/dlls/mshtml/view.c +++ b/dlls/mshtml/view.c @@ -912,3 +912,59 @@ void HTMLDocument_View_Init(HTMLDocument *This) This->IOleDocumentView_iface.lpVtbl = &OleDocumentViewVtbl; This->IViewObjectEx_iface.lpVtbl = &ViewObjectVtbl; } + +static inline HTMLDocumentObj *impl_from_IWindowForBindingUI(IWindowForBindingUI *iface) +{ + return CONTAINING_RECORD(iface, HTMLDocumentObj, IWindowForBindingUI_iface); +} + +static HRESULT WINAPI WindowForBindingUI_QueryInterface(IWindowForBindingUI *iface, REFIID riid, void **ppv) +{ + HTMLDocumentObj *This = impl_from_IWindowForBindingUI(iface); + + if(IsEqualGUID(&IID_IUnknown, riid)) { + TRACE("(%p)->(IID_IUnknown %p)\n", This, ppv); + *ppv = &This->IWindowForBindingUI_iface; + }else if(IsEqualGUID(&IID_IWindowForBindingUI, riid)) { + TRACE("(%p)->(IID_IWindowForBindingUI %p)\n", This, ppv); + *ppv = &This->IWindowForBindingUI_iface; + }else { + WARN("(%p)->(%s %p)\n", This, debugstr_guid(riid), ppv); + *ppv = NULL; + return E_NOINTERFACE; + } + + IUnknown_AddRef((IUnknown*)*ppv); + return S_OK; +} + +static ULONG WINAPI WindowForBindingUI_AddRef(IWindowForBindingUI *iface) +{ + HTMLDocumentObj *This = impl_from_IWindowForBindingUI(iface); + return htmldoc_addref(&This->basedoc); +} + +static ULONG WINAPI WindowForBindingUI_Release(IWindowForBindingUI *iface) +{ + HTMLDocumentObj *This = impl_from_IWindowForBindingUI(iface); + return htmldoc_release(&This->basedoc); +} + +static HRESULT WINAPI WindowForBindingUI_GetWindow(IWindowForBindingUI *iface, REFGUID rguidReason, HWND *phwnd) +{ + HTMLDocumentObj *This = impl_from_IWindowForBindingUI(iface); + FIXME("(%p)->(%s %p)\n", This, debugstr_guid(rguidReason), phwnd); + return E_NOTIMPL; +} + +static const IWindowForBindingUIVtbl WindowForBindingUIVtbl = { + WindowForBindingUI_QueryInterface, + WindowForBindingUI_AddRef, + WindowForBindingUI_Release, + WindowForBindingUI_GetWindow +}; + +void init_binding_ui(HTMLDocumentObj *doc) +{ + doc->IWindowForBindingUI_iface.lpVtbl = &WindowForBindingUIVtbl; +}

1 0

Rico Schüller : d3dx9: Crop the input in ID3DXBaseEffect::SetBool().
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: f658aabfd5a49506d2b916805e3df7556442853e URL: http://source.winehq.org/git/wine.git/?a=commit;h=f658aabfd5a49506d2b916805… Author: Rico Schüller <kgbricola(a)web.de> Date: Wed May 30 14:36:18 2012 +0200 d3dx9: Crop the input in ID3DXBaseEffect::SetBool(). --- dlls/d3dx9_36/effect.c | 2 ++ dlls/d3dx9_36/tests/effect.c | 3 ++- 2 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c index def73a0..5ac79d1 100644 --- a/dlls/d3dx9_36/effect.c +++ b/dlls/d3dx9_36/effect.c @@ -1725,6 +1725,8 @@ static HRESULT WINAPI ID3DXBaseEffectImpl_SetBool(ID3DXBaseEffect *iface, D3DXHA if (param && !param->element_count && param->rows == 1 && param->columns == 1) { + /* crop input */ + b = b ? TRUE : FALSE; set_number(param->data, param->type, &b, D3DXPT_BOOL); return D3D_OK; } diff --git a/dlls/d3dx9_36/tests/effect.c b/dlls/d3dx9_36/tests/effect.c index eb52f7e..c4046c7 100644 --- a/dlls/d3dx9_36/tests/effect.c +++ b/dlls/d3dx9_36/tests/effect.c @@ -1857,11 +1857,12 @@ static void test_effect_parameter_value(IDirect3DDevice9 *device) test_effect_parameter_value_GetTestGroup(&res[k], effect, &blob[res_value_offset], parameter, i); /* SetBool */ - bvalue = 1; + bvalue = 5; memcpy(expected_value, &blob[res_value_offset], res_desc->Bytes); hr = effect->lpVtbl->SetBool(effect, parameter, bvalue); if (!res_desc->Elements && res_desc->Rows == 1 && res_desc->Columns == 1) { + bvalue = bvalue ? TRUE : FALSE; set_number(expected_value, res_desc->Type, &bvalue, D3DXPT_BOOL); ok(hr == D3D_OK, "%u - %s: SetBool failed, got %#x, expected %#x\n", i, res_full_name, hr, D3D_OK); }

1 0

Jacek Caban : wininet: Use CRLF separator in INTERNET_CERTIFICATE_INFO.
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: 37e483d567327fb65c71970fe2992ddd12b7f937 URL: http://source.winehq.org/git/wine.git/?a=commit;h=37e483d567327fb65c71970fe… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed May 30 12:13:53 2012 +0200 wininet: Use CRLF separator in INTERNET_CERTIFICATE_INFO. --- dlls/wininet/http.c | 8 ++++---- dlls/wininet/tests/http.c | 41 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 45 insertions(+), 4 deletions(-) diff --git a/dlls/wininet/http.c b/dlls/wininet/http.c index 74c9006..54ac0d5 100644 --- a/dlls/wininet/http.c +++ b/dlls/wininet/http.c @@ -2130,18 +2130,18 @@ static DWORD HTTPREQ_QueryOption(object_header_t *hdr, DWORD option, void *buffe info->ftExpiry = context->pCertInfo->NotAfter; info->ftStart = context->pCertInfo->NotBefore; len = CertNameToStrA(context->dwCertEncodingType, - &context->pCertInfo->Subject, CERT_SIMPLE_NAME_STR, NULL, 0); + &context->pCertInfo->Subject, CERT_SIMPLE_NAME_STR|CERT_NAME_STR_CRLF_FLAG, NULL, 0); info->lpszSubjectInfo = LocalAlloc(0, len); if(info->lpszSubjectInfo) CertNameToStrA(context->dwCertEncodingType, - &context->pCertInfo->Subject, CERT_SIMPLE_NAME_STR, + &context->pCertInfo->Subject, CERT_SIMPLE_NAME_STR|CERT_NAME_STR_CRLF_FLAG, info->lpszSubjectInfo, len); len = CertNameToStrA(context->dwCertEncodingType, - &context->pCertInfo->Issuer, CERT_SIMPLE_NAME_STR, NULL, 0); + &context->pCertInfo->Issuer, CERT_SIMPLE_NAME_STR|CERT_NAME_STR_CRLF_FLAG, NULL, 0); info->lpszIssuerInfo = LocalAlloc(0, len); if(info->lpszIssuerInfo) CertNameToStrA(context->dwCertEncodingType, - &context->pCertInfo->Issuer, CERT_SIMPLE_NAME_STR, + &context->pCertInfo->Issuer, CERT_SIMPLE_NAME_STR|CERT_NAME_STR_CRLF_FLAG, info->lpszIssuerInfo, len); info->dwKeySize = NETCON_GetCipherStrength(req->netconn); CertFreeCertificateContext(context); diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c index 2e7b693..204c5b7 100644 --- a/dlls/wininet/tests/http.c +++ b/dlls/wininet/tests/http.c @@ -2878,6 +2878,45 @@ static void release_cert_info(INTERNET_CERTIFICATE_INFOA *info) LocalFree(info->lpszEncryptionAlgName); } +static void test_cert_struct(HINTERNET req) +{ + INTERNET_CERTIFICATE_INFOA info; + DWORD size; + BOOL res; + + static const char ex_subject[] = + "US\r\n" + "Minnesota\r\n" + "Saint Paul\r\n" + "WineHQ\r\n" + "test.winehq.org\r\n" + "webmaster(a)winehq.org"; + + static const char ex_issuer[] = + "US\r\n" + "Minnesota\r\n" + "WineHQ\r\n" + "test.winehq.org\r\n" + "webmaster(a)winehq.org"; + + memset(&info, 0x5, sizeof(&info)); + + size = sizeof(info); + res = InternetQueryOption(req, INTERNET_OPTION_SECURITY_CERTIFICATE_STRUCT, &info, &size); + ok(res, "InternetQueryOption failed: %u\n", GetLastError()); + ok(size == sizeof(info), "size = %u\n", size); + + ok(!strcmp(info.lpszSubjectInfo, ex_subject), "lpszSubjectInfo = %s\n", info.lpszSubjectInfo); + ok(!strcmp(info.lpszIssuerInfo, ex_issuer), "lpszIssuerInfo = %s\n", info.lpszIssuerInfo); + ok(!info.lpszSignatureAlgName, "lpszSignatureAlgName = %s\n", info.lpszSignatureAlgName); + ok(!info.lpszEncryptionAlgName, "lpszEncryptionAlgName = %s\n", info.lpszEncryptionAlgName); + ok(!info.lpszProtocolName, "lpszProtocolName = %s\n", info.lpszProtocolName); + todo_wine + ok(info.dwKeySize == 128, "dwKeySize = %u\n", info.dwKeySize); + + release_cert_info(&info); +} + #define test_secflags_option(a,b) _test_secflags_option(__LINE__,a,b) static void _test_secflags_option(unsigned line, HINTERNET req, DWORD ex_flags) { @@ -3058,6 +3097,8 @@ static void test_security_flags(void) test_secflags_option(req, SECURITY_FLAG_SECURE|SECURITY_FLAG_IGNORE_UNKNOWN_CA |SECURITY_FLAG_STRENGTH_STRONG|0x800000); + test_cert_struct(req); + res = InternetReadFile(req, buf, sizeof(buf), &size); ok(res, "InternetReadFile failed: %u\n", GetLastError()); ok(size, "size = 0\n");

1 0

Christian Costa : d3drm: Generate normals automatically when there are not present in the x file.
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: 8d9d97be0fc6beec7f7e8451e2f191d5ec4f5a80 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8d9d97be0fc6beec7f7e8451e… Author: Christian Costa <titan.costa(a)gmail.com> Date: Sun May 27 17:44:59 2012 +0200 d3drm: Generate normals automatically when there are not present in the x file. --- dlls/d3drm/meshbuilder.c | 55 +++++++++++++++++++++++++++++++++++++++++---- dlls/d3drm/tests/d3drm.c | 2 +- 2 files changed, 51 insertions(+), 6 deletions(-) diff --git a/dlls/d3drm/meshbuilder.c b/dlls/d3drm/meshbuilder.c index 4401a56..22aec7e 100644 --- a/dlls/d3drm/meshbuilder.c +++ b/dlls/d3drm/meshbuilder.c @@ -1118,6 +1118,8 @@ HRESULT load_mesh_data(IDirect3DRMMeshBuilder3* iface, LPDIRECTXFILEDATA pData) TRACE("Mesh name is '%s'\n", This->name ? This->name : ""); + This->nb_normals = 0; + hr = IDirectXFileData_GetData(pData, NULL, &size, (void**)&ptr); if (hr != DXFILE_OK) goto end; @@ -1211,10 +1213,19 @@ HRESULT load_mesh_data(IDirect3DRMMeshBuilder3* iface, LPDIRECTXFILEDATA pData) pData2 = NULL; } + if (!This->nb_normals) + { + /* Allocate normals, one per vertex */ + This->pNormals = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, This->nb_vertices * sizeof(D3DVECTOR)); + if (!This->pNormals) + goto end; + } + for (i = 0; i < This->nb_faces; i++) { DWORD j; DWORD nb_face_indexes; + D3DVECTOR face_normal; if (faces_vertex_idx_size < sizeof(DWORD)) WARN("Not enough data to read number of indices of face %d\n", i); @@ -1227,22 +1238,46 @@ HRESULT load_mesh_data(IDirect3DRMMeshBuilder3* iface, LPDIRECTXFILEDATA pData) if (faces_vertex_idx_size < (nb_face_indexes * sizeof(DWORD))) WARN("Not enough data to read all indices of face %d\n", i); + if (!This->nb_normals) + { + /* Compute face normal */ + if (nb_face_indexes > 2) + { + D3DVECTOR a, b; + + D3DRMVectorSubtract(&a, &This->pVertices[faces_vertex_idx_ptr[2]], &This->pVertices[faces_vertex_idx_ptr[1]]); + D3DRMVectorSubtract(&a, &This->pVertices[faces_vertex_idx_ptr[1]], &This->pVertices[faces_vertex_idx_ptr[0]]); + D3DRMVectorCrossProduct(&face_normal, &a, &b); + D3DRMVectorNormalize(&face_normal); + } + else + { + face_normal.u1.x = 0.0f; + face_normal.u2.y = 0.0f; + face_normal.u3.z = 0.0f; + } + } + for (j = 0; j < nb_face_indexes; j++) { /* Copy vertex index */ - *(faces_data_ptr + faces_data_size++) = *(faces_vertex_idx_ptr++); + *(faces_data_ptr + faces_data_size++) = *faces_vertex_idx_ptr; /* Copy normal index */ - if (faces_normal_idx_data) + if (This->nb_normals) { /* Read from x file */ *(faces_data_ptr + faces_data_size++) = *(faces_normal_idx_ptr++); } else { - FIXME("No normal available, generate a fake normal index\n"); - /* Must be generated, put 0 for now */ - *(faces_data_ptr + faces_data_size++) = 0; + DWORD vertex_idx = *faces_vertex_idx_ptr; + if (vertex_idx > This->nb_vertices) + vertex_idx = 0; + *(faces_data_ptr + faces_data_size++) = vertex_idx; + /* Add face normal to vertex normal */ + D3DRMVectorAdd(&This->pNormals[vertex_idx], &This->pNormals[vertex_idx], &face_normal); } + faces_vertex_idx_ptr++; } faces_vertex_idx_size -= nb_face_indexes; } @@ -1253,6 +1288,16 @@ HRESULT load_mesh_data(IDirect3DRMMeshBuilder3* iface, LPDIRECTXFILEDATA pData) /* Set size (in number of DWORD) of all faces data */ This->face_data_size = faces_data_size; + if (!This->nb_normals) + { + /* Normalize all normals */ + for (i = 0; i < This->nb_vertices; i++) + { + D3DRMVectorNormalize(&This->pNormals[i]); + } + This->nb_normals = This->nb_vertices; + } + /* If there is no texture coordinates, generate default texture coordinates (0.0f, 0.0f) for each vertex */ if (!This->pCoords2d) { diff --git a/dlls/d3drm/tests/d3drm.c b/dlls/d3drm/tests/d3drm.c index fdf4e90..8d35802 100644 --- a/dlls/d3drm/tests/d3drm.c +++ b/dlls/d3drm/tests/d3drm.c @@ -240,7 +240,7 @@ static void test_MeshBuilder(void) hr = IDirect3DRMMeshBuilder_GetVertices(pMeshBuilder, &val1, NULL, &val2, NULL, &val3, NULL); ok(hr == D3DRM_OK, "Cannot get vertices information (hr = %x)\n", hr); ok(val1 == 4, "Wrong number of vertices %d (must be 4)\n", val1); - todo_wine ok(val2 == 4, "Wrong number of normals %d (must be 4)\n", val2); + ok(val2 == 4, "Wrong number of normals %d (must be 4)\n", val2); ok(val3 == 22, "Wrong number of face data bytes %d (must be 22)\n", val3); /* Check that Load method generated default texture coordinates (0.0f, 0.0f) for each vertex */

1 0

Christian Costa : d3drm: Implement IDirect3DRMMeshImpl_GetGroupMaterial.
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: 3fbe4914bded91d0fb9a22e542a1a8ab4ab69b9b URL: http://source.winehq.org/git/wine.git/?a=commit;h=3fbe4914bded91d0fb9a22e54… Author: Christian Costa <titan.costa(a)gmail.com> Date: Sun May 27 17:44:51 2012 +0200 d3drm: Implement IDirect3DRMMeshImpl_GetGroupMaterial. --- dlls/d3drm/meshbuilder.c | 19 ++++++++++++++++--- dlls/d3drm/tests/d3drm.c | 6 +++--- 2 files changed, 19 insertions(+), 6 deletions(-) diff --git a/dlls/d3drm/meshbuilder.c b/dlls/d3drm/meshbuilder.c index 2074702..4401a56 100644 --- a/dlls/d3drm/meshbuilder.c +++ b/dlls/d3drm/meshbuilder.c @@ -43,6 +43,7 @@ typedef struct { unsigned vertex_per_face; DWORD face_data_size; unsigned* face_data; + IDirect3DRMMaterial2* material; IDirect3DRMTexture3* texture; } mesh_group; @@ -2452,6 +2453,7 @@ static HRESULT WINAPI IDirect3DRMMeshImpl_AddGroup(IDirect3DRMMesh* iface, memcpy(group->face_data, face_data, group->face_data_size * sizeof(unsigned)); + group->material = NULL; group->texture = NULL; *return_id = This->nb_groups++; @@ -2623,13 +2625,24 @@ static D3DRMRENDERQUALITY WINAPI IDirect3DRMMeshImpl_GetGroupQuality(IDirect3DRM } static HRESULT WINAPI IDirect3DRMMeshImpl_GetGroupMaterial(IDirect3DRMMesh* iface, - D3DRMGROUPINDEX id, LPDIRECT3DRMMATERIAL *returnPtr) + D3DRMGROUPINDEX id, LPDIRECT3DRMMATERIAL *material) { IDirect3DRMMeshImpl *This = impl_from_IDirect3DRMMesh(iface); - FIXME("(%p)->(%u,%p): stub\n", This, id, returnPtr); + TRACE("(%p)->(%u,%p)\n", This, id, material); - return E_NOTIMPL; + if (id >= This->nb_groups) + return D3DRMERR_BADVALUE; + + if (!material) + return E_POINTER; + + if (This->groups[id].material) + IDirect3DRMTexture_QueryInterface(This->groups[id].material, &IID_IDirect3DRMMaterial, (void**)material); + else + *material = NULL; + + return D3DRM_OK; } static HRESULT WINAPI IDirect3DRMMeshImpl_GetGroupTexture(IDirect3DRMMesh* iface, diff --git a/dlls/d3drm/tests/d3drm.c b/dlls/d3drm/tests/d3drm.c index f9940c1..fdf4e90 100644 --- a/dlls/d3drm/tests/d3drm.c +++ b/dlls/d3drm/tests/d3drm.c @@ -353,7 +353,7 @@ static void test_MeshBuilder(void) DWORD nb_groups; unsigned nb_vertices, nb_faces, nb_face_vertices; DWORD data_size; - LPDIRECT3DRMMATERIAL material; + LPDIRECT3DRMMATERIAL material = (LPDIRECT3DRMMATERIAL)0xdeadbeef; LPDIRECT3DRMTEXTURE texture = (LPDIRECT3DRMTEXTURE)0xdeadbeef; D3DVALUE values[3]; @@ -371,8 +371,8 @@ static void test_MeshBuilder(void) ok(hr == D3DRM_OK, "GetCroupTexture failed returning hr = %x\n", hr); ok(texture == NULL, "No texture should be present\n"); hr = IDirect3DRMMesh_GetGroupMaterial(mesh, 0, &material); - todo_wine ok(hr == D3DRM_OK, "GetCroupMaterial failed returning hr = %x\n", hr); - todo_wine ok(material != NULL, "No material should be present\n"); + ok(hr == D3DRM_OK, "GetCroupMaterial failed returning hr = %x\n", hr); + todo_wine ok(material != NULL, "No material present\n"); if ((hr == D3DRM_OK) && material) { hr = IDirect3DRMMaterial_GetEmissive(material, &values[0], &values[1], &values[2]);

1 0

Christian Costa : d3drm: Implement IDirect3DRMMesh_GetGroupTexture.
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: b340501a93478d79e223c6933a8a1646d7ad8b37 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b340501a93478d79e223c6933… Author: Christian Costa <titan.costa(a)gmail.com> Date: Sun May 27 17:44:42 2012 +0200 d3drm: Implement IDirect3DRMMesh_GetGroupTexture. --- dlls/d3drm/meshbuilder.c | 20 +++++++++++++++++--- dlls/d3drm/tests/d3drm.c | 6 +++--- 2 files changed, 20 insertions(+), 6 deletions(-) diff --git a/dlls/d3drm/meshbuilder.c b/dlls/d3drm/meshbuilder.c index 97f85c9..2074702 100644 --- a/dlls/d3drm/meshbuilder.c +++ b/dlls/d3drm/meshbuilder.c @@ -43,6 +43,7 @@ typedef struct { unsigned vertex_per_face; DWORD face_data_size; unsigned* face_data; + IDirect3DRMTexture3* texture; } mesh_group; typedef struct { @@ -2451,6 +2452,8 @@ static HRESULT WINAPI IDirect3DRMMeshImpl_AddGroup(IDirect3DRMMesh* iface, memcpy(group->face_data, face_data, group->face_data_size * sizeof(unsigned)); + group->texture = NULL; + *return_id = This->nb_groups++; return D3DRM_OK; @@ -2630,13 +2633,24 @@ static HRESULT WINAPI IDirect3DRMMeshImpl_GetGroupMaterial(IDirect3DRMMesh* ifac } static HRESULT WINAPI IDirect3DRMMeshImpl_GetGroupTexture(IDirect3DRMMesh* iface, - D3DRMGROUPINDEX id, LPDIRECT3DRMTEXTURE *returnPtr) + D3DRMGROUPINDEX id, LPDIRECT3DRMTEXTURE *texture) { IDirect3DRMMeshImpl *This = impl_from_IDirect3DRMMesh(iface); - FIXME("(%p)->(%u,%p): stub\n", This, id, returnPtr); + TRACE("(%p)->(%u,%p)\n", This, id, texture); - return E_NOTIMPL; + if (id >= This->nb_groups) + return D3DRMERR_BADVALUE; + + if (!texture) + return E_POINTER; + + if (This->groups[id].texture) + IDirect3DRMTexture_QueryInterface(This->groups[id].texture, &IID_IDirect3DRMTexture, (void**)texture); + else + *texture = NULL; + + return D3DRM_OK; } static const struct IDirect3DRMMeshVtbl Direct3DRMMesh_Vtbl = diff --git a/dlls/d3drm/tests/d3drm.c b/dlls/d3drm/tests/d3drm.c index c22804d..f9940c1 100644 --- a/dlls/d3drm/tests/d3drm.c +++ b/dlls/d3drm/tests/d3drm.c @@ -354,7 +354,7 @@ static void test_MeshBuilder(void) unsigned nb_vertices, nb_faces, nb_face_vertices; DWORD data_size; LPDIRECT3DRMMATERIAL material; - LPDIRECT3DRMTEXTURE texture; + LPDIRECT3DRMTEXTURE texture = (LPDIRECT3DRMTEXTURE)0xdeadbeef; D3DVALUE values[3]; nb_groups = IDirect3DRMMesh_GetGroupCount(mesh); @@ -368,8 +368,8 @@ static void test_MeshBuilder(void) todo_wine ok(nb_face_vertices == 3, "Wrong number of vertices per face %u (must be 3)\n", nb_face_vertices); todo_wine ok(data_size == 3, "Wrong number of face data bytes %u (must be 3)\n", data_size); hr = IDirect3DRMMesh_GetGroupTexture(mesh, 0, &texture); - todo_wine ok(hr == D3DRM_OK, "GetCroupTexture failed returning hr = %x\n", hr); - todo_wine ok(texture == NULL, "No texture should be present\n"); + ok(hr == D3DRM_OK, "GetCroupTexture failed returning hr = %x\n", hr); + ok(texture == NULL, "No texture should be present\n"); hr = IDirect3DRMMesh_GetGroupMaterial(mesh, 0, &material); todo_wine ok(hr == D3DRM_OK, "GetCroupMaterial failed returning hr = %x\n", hr); todo_wine ok(material != NULL, "No material should be present\n");

1 0

Christian Costa : d3drm: Implement IDirect3DRMFrameXImpl_GetLights and IDirect3DRMLightArray interface .
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: 6e15d91927c6b855d9c9058d18cdee9804f64795 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6e15d91927c6b855d9c9058d1… Author: Christian Costa <titan.costa(a)gmail.com> Date: Sun May 27 17:44:19 2012 +0200 d3drm: Implement IDirect3DRMFrameXImpl_GetLights and IDirect3DRMLightArray interface. --- dlls/d3drm/frame.c | 164 ++++++++++++++++++++++++++++++++++++++++++++- dlls/d3drm/tests/d3drm.c | 2 +- 2 files changed, 161 insertions(+), 5 deletions(-) diff --git a/dlls/d3drm/frame.c b/dlls/d3drm/frame.c index 574e2f8..f0e1cd0 100644 --- a/dlls/d3drm/frame.c +++ b/dlls/d3drm/frame.c @@ -71,8 +71,16 @@ typedef struct { LPDIRECT3DRMVISUAL* visuals; } IDirect3DRMVisualArrayImpl; +typedef struct { + IDirect3DRMLightArray IDirect3DRMLightArray_iface; + LONG ref; + ULONG size; + LPDIRECT3DRMLIGHT* lights; +} IDirect3DRMLightArrayImpl; + HRESULT Direct3DRMFrameArray_create(IDirect3DRMFrameArray** obj); HRESULT Direct3DRMVisualArray_create(IDirect3DRMVisualArray** ret_iface); +HRESULT Direct3DRMLightArray_create(IDirect3DRMLightArray** ret_iface); static inline IDirect3DRMFrameImpl *impl_from_IDirect3DRMFrame2(IDirect3DRMFrame2 *iface) { @@ -87,6 +95,11 @@ static inline IDirect3DRMFrameImpl *impl_from_IDirect3DRMFrame3(IDirect3DRMFrame static inline IDirect3DRMFrameImpl *unsafe_impl_from_IDirect3DRMFrame2(IDirect3DRMFrame2 *iface); static inline IDirect3DRMFrameImpl *unsafe_impl_from_IDirect3DRMFrame3(IDirect3DRMFrame3 *iface); +static inline IDirect3DRMLightArrayImpl *impl_from_IDirect3DRMLightArray(IDirect3DRMLightArray *iface) +{ + return CONTAINING_RECORD(iface, IDirect3DRMLightArrayImpl, IDirect3DRMLightArray_iface); +} + /*** IUnknown methods ***/ static HRESULT WINAPI IDirect3DRMFrame2Impl_QueryInterface(IDirect3DRMFrame2* iface, REFIID riid, void** object) @@ -348,9 +361,9 @@ static HRESULT WINAPI IDirect3DRMFrame2Impl_GetLights(IDirect3DRMFrame2* iface, { IDirect3DRMFrameImpl *This = impl_from_IDirect3DRMFrame2(iface); - FIXME("(%p/%p)->(%p): stub\n", iface, This, lights); + TRACE("(%p/%p)->(%p)\n", iface, This, lights); - return E_NOTIMPL; + return IDirect3DRMFrame3_GetLights(&This->IDirect3DRMFrame3_iface, lights); } static D3DRMMATERIALMODE WINAPI IDirect3DRMFrame2Impl_GetMaterialMode(IDirect3DRMFrame2* iface) @@ -1401,10 +1414,33 @@ static HRESULT WINAPI IDirect3DRMFrame3Impl_GetLights(IDirect3DRMFrame3* iface, LPDIRECT3DRMLIGHTARRAY *lights) { IDirect3DRMFrameImpl *This = impl_from_IDirect3DRMFrame3(iface); + IDirect3DRMLightArrayImpl* obj; + HRESULT hr; - FIXME("(%p/%p)->(%p): stub\n", iface, This, lights); + TRACE("(%p/%p)->(%p)\n", iface, This, lights); - return E_NOTIMPL; + if (!lights) + return D3DRMERR_BADVALUE; + + hr = Direct3DRMLightArray_create(lights); + + if (hr != D3DRM_OK) + return hr; + + obj = (IDirect3DRMLightArrayImpl*)*lights; + + obj->size = This->nb_lights; + if (This->nb_lights) + { + ULONG i; + obj->lights = HeapAlloc(GetProcessHeap(), 0, This->nb_lights * sizeof(LPDIRECT3DRMLIGHT)); + if (!obj->lights) + return E_OUTOFMEMORY; + for (i = 0; i < This->nb_lights; i++) + IDirect3DRMLight_QueryInterface(This->lights[i], &IID_IDirect3DRMLight, (void**)&obj->lights[i]); + } + + return D3DRM_OK; } static D3DRMMATERIALMODE WINAPI IDirect3DRMFrame3Impl_GetMaterialMode(IDirect3DRMFrame3* iface) @@ -2544,3 +2580,123 @@ HRESULT Direct3DRMVisualArray_create(IDirect3DRMVisualArray** ret_iface) return S_OK; } + +/*** IUnknown methods ***/ +static HRESULT WINAPI IDirect3DRMLightArrayImpl_QueryInterface(IDirect3DRMLightArray* iface, + REFIID riid, void** object) +{ + IDirect3DRMLightArrayImpl *This = impl_from_IDirect3DRMLightArray(iface); + + TRACE("(%p/%p)->(%s, %p)\n", iface, This, debugstr_guid(riid), object); + + *object = NULL; + + if (IsEqualGUID(riid, &IID_IUnknown) || + IsEqualGUID(riid, &IID_IDirect3DRMLightArray)) + { + *object = &This->IDirect3DRMLightArray_iface; + } + else + { + FIXME("interface %s not implemented\n", debugstr_guid(riid)); + return E_NOINTERFACE; + } + + IDirect3DRMLightArray_AddRef(iface); + return S_OK; +} + +static ULONG WINAPI IDirect3DRMLightArrayImpl_AddRef(IDirect3DRMLightArray* iface) +{ + IDirect3DRMLightArrayImpl *This = impl_from_IDirect3DRMLightArray(iface); + ULONG ref = InterlockedIncrement(&This->ref); + + TRACE("(%p)->(): new ref = %u\n", This, ref); + + return ref; +} + +static ULONG WINAPI IDirect3DRMLightArrayImpl_Release(IDirect3DRMLightArray* iface) +{ + IDirect3DRMLightArrayImpl *This = impl_from_IDirect3DRMLightArray(iface); + ULONG ref = InterlockedDecrement(&This->ref); + ULONG i; + + TRACE("(%p)->(): new ref = %u\n", This, ref); + + if (!ref) + { + for (i = 0; i < This->size; i++) + IDirect3DRMLight_Release(This->lights[i]); + HeapFree(GetProcessHeap(), 0, This->lights); + HeapFree(GetProcessHeap(), 0, This); + } + + return ref; +} + +/*** IDirect3DRMArray methods ***/ +static DWORD WINAPI IDirect3DRMLightArrayImpl_GetSize(IDirect3DRMLightArray* iface) +{ + IDirect3DRMLightArrayImpl *This = impl_from_IDirect3DRMLightArray(iface); + + TRACE("(%p)->() = %d\n", This, This->size); + + return This->size; +} + +/*** IDirect3DRMLightArray methods ***/ +static HRESULT WINAPI IDirect3DRMLightArrayImpl_GetElement(IDirect3DRMLightArray* iface, DWORD index, LPDIRECT3DRMLIGHT* light) +{ + IDirect3DRMLightArrayImpl *This = impl_from_IDirect3DRMLightArray(iface); + + TRACE("(%p)->(%u, %p)\n", This, index, light); + + if (!light) + return D3DRMERR_BADVALUE; + + *light = NULL; + + if (index >= This->size) + return D3DRMERR_BADVALUE; + + IDirect3DRMLight_AddRef(This->lights[index]); + *light = This->lights[index]; + + return D3DRM_OK; +} + +static const struct IDirect3DRMLightArrayVtbl Direct3DRMLightArray_Vtbl = +{ + /*** IUnknown methods ***/ + IDirect3DRMLightArrayImpl_QueryInterface, + IDirect3DRMLightArrayImpl_AddRef, + IDirect3DRMLightArrayImpl_Release, + /*** IDirect3DRMArray methods ***/ + IDirect3DRMLightArrayImpl_GetSize, + /*** IDirect3DRMLightArray methods ***/ + IDirect3DRMLightArrayImpl_GetElement +}; + +HRESULT Direct3DRMLightArray_create(IDirect3DRMLightArray** obj) +{ + IDirect3DRMLightArrayImpl* object; + + TRACE("(%p)\n", obj); + + *obj = NULL; + + object = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(IDirect3DRMLightArrayImpl)); + if (!object) + { + ERR("Out of memory\n"); + return E_OUTOFMEMORY; + } + + object->IDirect3DRMLightArray_iface.lpVtbl = &Direct3DRMLightArray_Vtbl; + object->ref = 1; + + *obj = &object->IDirect3DRMLightArray_iface; + + return S_OK; +} diff --git a/dlls/d3drm/tests/d3drm.c b/dlls/d3drm/tests/d3drm.c index b6ed51a..c22804d 100644 --- a/dlls/d3drm/tests/d3drm.c +++ b/dlls/d3drm/tests/d3drm.c @@ -795,7 +795,7 @@ static void test_Frame(void) pLightArray = NULL; hr = IDirect3DRMFrame_GetLights(pFrameP1, &pLightArray); - todo_wine ok(hr == D3DRM_OK, "Cannot get lights (hr = %x)\n", hr); + ok(hr == D3DRM_OK, "Cannot get lights (hr = %x)\n", hr); if (pLightArray) { count = IDirect3DRMLightArray_GetSize(pLightArray);

1 0

Piotr Caban : msvcrt: Handle rethrown exception in cxx_frame_handler.
by Alexandre Julliard 30 May '12

30 May '12

Module: wine Branch: master Commit: 5e418f0ccfa307af3a6adbebfdedf01093f99ab8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5e418f0ccfa307af3a6adbebf… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Fri May 25 19:42:51 2012 +0200 msvcrt: Handle rethrown exception in cxx_frame_handler. --- dlls/msvcrt/except_i386.c | 20 +++++++++++++++++++- 1 files changed, 19 insertions(+), 1 deletions(-) diff --git a/dlls/msvcrt/except_i386.c b/dlls/msvcrt/except_i386.c index d8b03a4..7c3265c 100644 --- a/dlls/msvcrt/except_i386.c +++ b/dlls/msvcrt/except_i386.c @@ -373,7 +373,9 @@ static DWORD catch_function_nested_handler( EXCEPTION_RECORD *rec, EXCEPTION_REG if(rec->ExceptionCode == CXX_EXCEPTION) { PEXCEPTION_RECORD prev_rec = nested_frame->rec; - if(rec->ExceptionInformation[1] == 0 && rec->ExceptionInformation[2] == 0) + if((rec->ExceptionInformation[1] == 0 && rec->ExceptionInformation[2] == 0) || + (rec->ExceptionInformation[1] == prev_rec->ExceptionInformation[1] && + rec->ExceptionInformation[2] == prev_rec->ExceptionInformation[2])) { /* exception was rethrown */ rec->ExceptionInformation[1] = prev_rec->ExceptionInformation[1]; @@ -381,6 +383,13 @@ static DWORD catch_function_nested_handler( EXCEPTION_RECORD *rec, EXCEPTION_REG TRACE("detect rethrow: re-propagate: obj: %lx, type: %lx\n", rec->ExceptionInformation[1], rec->ExceptionInformation[2]); } + else if (nested_frame->prev_rec && + nested_frame->prev_rec->ExceptionInformation[1] == prev_rec->ExceptionInformation[1] && + nested_frame->prev_rec->ExceptionInformation[2] == prev_rec->ExceptionInformation[2]) + { + TRACE("detect threw new exception in catch block - not owning old(obj: %lx type: %lx)\n", + prev_rec->ExceptionInformation[1], prev_rec->ExceptionInformation[2]); + } else { /* new exception in exception handler, destroy old */ void *object = (void*)prev_rec->ExceptionInformation[1]; @@ -506,6 +515,15 @@ DWORD CDECL cxx_frame_handler( PEXCEPTION_RECORD rec, cxx_exception_frame* frame if(rec->ExceptionCode == CXX_EXCEPTION) { + if (rec->ExceptionInformation[1] == 0 && rec->ExceptionInformation[2] == 0) + { + EXCEPTION_RECORD *exc_record = msvcrt_get_thread_data()->exc_record; + rec->ExceptionInformation[1] = exc_record->ExceptionInformation[1]; + rec->ExceptionInformation[2] = exc_record->ExceptionInformation[2]; + TRACE("detect rethrow: obj: %lx, type: %lx\n", + rec->ExceptionInformation[1], rec->ExceptionInformation[2]); + } + exc_type = (cxx_exception_type *)rec->ExceptionInformation[2]; if (rec->ExceptionInformation[0] > CXX_FRAME_MAGIC_VC8 &&

1 0

← Newer
1
2
3
4
5
6
7
8
...
74
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 May 2012