January 2008 - Wine-commits

Stefan Dösinger : wined3d: Implement IWineD3DVolume:: UnLoad.
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: 0ca65c7bc045f29ae07fce5fac8474d87afa623e URL: http://source.winehq.org/git/wine.git/?a=commit;h=0ca65c7bc045f29ae07fce5fa… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Tue Jan 8 15:28:57 2008 +0100 wined3d: Implement IWineD3DVolume::UnLoad. --- dlls/wined3d/volume.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/volume.c b/dlls/wined3d/volume.c index 7d739d1..2b8b246 100644 --- a/dlls/wined3d/volume.c +++ b/dlls/wined3d/volume.c @@ -98,7 +98,10 @@ static void WINAPI IWineD3DVolumeImpl_PreLoad(IWineD3DVolume *iface) { } static void WINAPI IWineD3DVolumeImpl_UnLoad(IWineD3DVolume *iface) { - IWineD3DResourceImpl_UnLoad((IWineD3DResource *)iface); + /* The whole content is shadowed on This->resource.allocatedMemory, and the + * texture name is managed by the VolumeTexture container + */ + TRACE("(%p): Nothing to do\n", iface); } static WINED3DRESOURCETYPE WINAPI IWineD3DVolumeImpl_GetType(IWineD3DVolume *iface) {

1 0

Stefan Dösinger : wined3d: Implement IWineD3DSurface:: UnLoad.
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: c372654b2644ce18d9e0ed8790139c96e2766191 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c372654b2644ce18d9e0ed879… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sat Jan 12 22:33:04 2008 +0100 wined3d: Implement IWineD3DSurface::UnLoad. --- dlls/wined3d/surface.c | 54 ++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 52 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 27e32aa..c9da469 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -489,8 +489,58 @@ void WINAPI IWineD3DSurfaceImpl_PreLoad(IWineD3DSurface *iface) { return; } -void WINAPI IWineD3DSurfaceImpl_UnLoad(IWineD3DSurface *iface) { - FIXME("(%p): Stub!\n", iface); +static void surface_remove_pbo(IWineD3DSurfaceImpl *This) { + This->resource.heapMemory = HeapAlloc(GetProcessHeap() ,0 , This->resource.size + RESOURCE_ALIGNMENT); + This->resource.allocatedMemory = + (BYTE *)(((ULONG_PTR) This->resource.heapMemory + (RESOURCE_ALIGNMENT - 1)) & ~(RESOURCE_ALIGNMENT - 1)); + + ENTER_GL(); + GL_EXTCALL(glBindBufferARB(GL_PIXEL_UNPACK_BUFFER_ARB, This->pbo)); + checkGLcall("glBindBuffer(GL_PIXEL_UNPACK_BUFFER, This->pbo)"); + GL_EXTCALL(glGetBufferSubDataARB(GL_PIXEL_UNPACK_BUFFER_ARB, 0, This->resource.size, This->resource.allocatedMemory)); + checkGLcall("glGetBufferSubData"); + GL_EXTCALL(glDeleteBuffersARB(1, &This->pbo)); + checkGLcall("glDeleteBuffers"); + LEAVE_GL(); + + This->pbo = 0; + This->Flags &= ~SFLAG_PBO; +} + +static void WINAPI IWineD3DSurfaceImpl_UnLoad(IWineD3DSurface *iface) { + IWineD3DBaseTexture *texture = NULL; + IWineD3DSurfaceImpl *This = (IWineD3DSurfaceImpl *) iface; + TRACE("(%p)\n", iface); + + /* Default pool resources are supposed to be destroyed before Reset is called. + * Implicit resources stay however. So this means we have an implicit render target + * or depth stencil, and the content isn't supposed to survive the reset anyway + */ + if(This->resource.pool == WINED3DPOOL_DEFAULT) { + TRACE("Default pool - nothing to do\n"); + return; + } + + /* Load the surface into system memory */ + IWineD3DSurface_LoadLocation(iface, SFLAG_INSYSMEM, NULL); + + /* Destroy PBOs, but load them into real sysmem before */ + if(This->Flags & SFLAG_PBO) { + surface_remove_pbo(This); + } + + /* If we're in a texture, the texture name belongs to the texture. Otherwise, + * destroy it + */ + IWineD3DSurface_GetContainer(iface, &IID_IWineD3DBaseTexture, (void **) &texture); + if(!texture) { + ENTER_GL(); + glDeleteTextures(1, &This->glDescription.textureName); + This->glDescription.textureName = 0; + LEAVE_GL(); + } else { + IWineD3DBaseTexture_Release(texture); + } return; }

1 0

Stefan Dösinger : wined3d: Implement IWineD3DVertexBuffer::UnLoad.
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: 15245a74b646543a641077b57b056108f69d23a7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=15245a74b646543a641077b57… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Tue Jan 8 15:23:35 2008 +0100 wined3d: Implement IWineD3DVertexBuffer::UnLoad. --- dlls/wined3d/vertexbuffer.c | 17 ++++++++++++++++- 1 files changed, 16 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/vertexbuffer.c b/dlls/wined3d/vertexbuffer.c index b8687e0..07ca2a0 100644 --- a/dlls/wined3d/vertexbuffer.c +++ b/dlls/wined3d/vertexbuffer.c @@ -597,7 +597,22 @@ static void WINAPI IWineD3DVertexBufferImpl_PreLoad(IWineD3DVertexBuffer *if } static void WINAPI IWineD3DVertexBufferImpl_UnLoad(IWineD3DVertexBuffer *iface) { - IWineD3DResourceImpl_UnLoad((IWineD3DResource *)iface); + IWineD3DVertexBufferImpl *This = (IWineD3DVertexBufferImpl *) iface; + IWineD3DDeviceImpl *device = This->resource.wineD3DDevice; + TRACE("(%p)\n", This); + + /* This is easy: The whole content is shadowed in This->resource.allocatedMemory, + * so we only have to destroy the vbo. Only do it if we have a vbo, which implies + * that vbos are supported + */ + if(This->vbo) { + ActivateContext(device, device->lastActiveRenderTarget, CTXUSAGE_RESOURCELOAD); + ENTER_GL(); + GL_EXTCALL(glDeleteBuffersARB(1, &This->vbo)); + checkGLcall("glDeleteBuffersARB"); + LEAVE_GL(); + This->vbo = 0; + } } static WINED3DRESOURCETYPE WINAPI IWineD3DVertexBufferImpl_GetType(IWineD3DVertexBuffer *iface) {

1 0

Rob Shearman : rpcrt4: Add traces to the start of TowerConstruct and TowerExplode.
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: 516008ff5fa6ddffa8f9230ac9d7faaa8b71c4f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=516008ff5fa6ddffa8f9230ac… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Jan 13 20:53:46 2008 +0000 rpcrt4: Add traces to the start of TowerConstruct and TowerExplode. --- dlls/rpcrt4/rpc_epmap.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/rpcrt4/rpc_epmap.c b/dlls/rpcrt4/rpc_epmap.c index cac3f62..46244ec 100644 --- a/dlls/rpcrt4/rpc_epmap.c +++ b/dlls/rpcrt4/rpc_epmap.c @@ -267,6 +267,9 @@ RPC_STATUS WINAPI TowerExplode( const twr_uuid_floor_t *object_floor; const twr_uuid_floor_t *syntax_floor; + TRACE("(%p, %p, %p, %p, %p, %p)\n", tower, object, syntax, protseq, + endpoint, address); + if (protseq) *protseq = NULL; if (endpoint) @@ -336,6 +339,9 @@ RPC_STATUS WINAPI TowerConstruct( twr_uuid_floor_t *object_floor; twr_uuid_floor_t *syntax_floor; + TRACE("(%p, %p, %s, %s, %s, %p)\n", object, syntax, debugstr_a(protseq), + debugstr_a(endpoint), debugstr_a(address), tower); + *tower = NULL; status = RpcTransport_GetTopOfTower(NULL, &tower_size, protseq, address, endpoint);

1 0

Stefan Dösinger : wined3d: Add a new IWineD3DResource: :UnLoad method.
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: e59ddb0ab18d33f472029a6cb1584e9dc740fef9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e59ddb0ab18d33f472029a6cb… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sat Jan 12 22:56:30 2008 +0100 wined3d: Add a new IWineD3DResource::UnLoad method. --- dlls/wined3d/basetexture.c | 5 +++++ dlls/wined3d/cubetexture.c | 5 +++++ dlls/wined3d/indexbuffer.c | 5 +++++ dlls/wined3d/resource.c | 6 ++++++ dlls/wined3d/surface.c | 6 ++++++ dlls/wined3d/surface_gdi.c | 22 +++++++++++++++++++--- dlls/wined3d/texture.c | 5 +++++ dlls/wined3d/vertexbuffer.c | 5 +++++ dlls/wined3d/volume.c | 5 +++++ dlls/wined3d/volumetexture.c | 5 +++++ dlls/wined3d/wined3d_private.h | 1 + include/wine/wined3d_interface.h | 20 +++++++++++++++++++- 12 files changed, 86 insertions(+), 4 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=e59ddb0ab18d33f472029…

1 0

Eric Pouech : ole32: Fixed datacache object so that we can retrieve meaningful information.
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: 528ad9ebbdbd4f29898a285d494009cf87ef3cc6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=528ad9ebbdbd4f29898a285d4… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sun Jan 13 17:03:44 2008 +0100 ole32: Fixed datacache object so that we can retrieve meaningful information. --- dlls/ole32/datacache.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/datacache.c b/dlls/ole32/datacache.c index 18cd5c2..beb6d93 100644 --- a/dlls/ole32/datacache.c +++ b/dlls/ole32/datacache.c @@ -831,7 +831,7 @@ static HRESULT DataCacheEntry_GetData(DataCacheEntry *This, if (FAILED(hr)) return hr; } - if (stgmedium->tymed == TYMED_NULL) + if (This->stgmedium.tymed == TYMED_NULL) return OLE_E_BLANK; return copy_stg_medium(This->data_cf, stgmedium, &This->stgmedium); }

1 0

Rob Shearman : rpcrt4: Add support for calculating the memory size of complex types with embedded non-conformant strings .
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: 97b842c817c9f603bfa45cb61441ee0398bcc402 URL: http://source.winehq.org/git/wine.git/?a=commit;h=97b842c817c9f603bfa45cb61… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Jan 13 20:52:54 2008 +0000 rpcrt4: Add support for calculating the memory size of complex types with embedded non-conformant strings. --- dlls/rpcrt4/ndr_marshall.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/rpcrt4/ndr_marshall.c b/dlls/rpcrt4/ndr_marshall.c index 8d358cf..d99fbf9 100644 --- a/dlls/rpcrt4/ndr_marshall.c +++ b/dlls/rpcrt4/ndr_marshall.c @@ -2126,6 +2126,7 @@ static unsigned long EmbeddedComplexSize(const MIDL_STUB_MESSAGE *pStubMsg, case RPC_FC_BOGUS_STRUCT: case RPC_FC_SMFARRAY: case RPC_FC_SMVARRAY: + case RPC_FC_CSTRING: return *(const WORD*)&pFormat[2]; case RPC_FC_USER_MARSHAL: return *(const WORD*)&pFormat[4]; @@ -2140,6 +2141,8 @@ static unsigned long EmbeddedComplexSize(const MIDL_STUB_MESSAGE *pStubMsg, return *(const SHORT*)pFormat; case RPC_FC_IP: return sizeof(void *); + case RPC_FC_WSTRING: + return *(const WORD*)&pFormat[2] * 2; default: FIXME("unhandled embedded type %02x\n", *pFormat); }

1 0

Rob Shearman : rpcrt4: Allocate a context handle if a NULL GUID is being unmarshalled.
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: 576c9dc6fa227ba9536117f062703cf6e4c15bc0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=576c9dc6fa227ba9536117f06… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Jan 13 20:52:25 2008 +0000 rpcrt4: Allocate a context handle if a NULL GUID is being unmarshalled. --- dlls/rpcrt4/ndr_contexthandle.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/rpcrt4/ndr_contexthandle.c b/dlls/rpcrt4/ndr_contexthandle.c index 7334bf6..8ed8f66 100644 --- a/dlls/rpcrt4/ndr_contexthandle.c +++ b/dlls/rpcrt4/ndr_contexthandle.c @@ -320,6 +320,7 @@ NDR_SCONTEXT WINAPI NDRSContextUnmarshall2(RPC_BINDING_HANDLE hBinding, RpcBinding *binding = hBinding; NDR_SCONTEXT SContext; RPC_STATUS status; + const ndr_context_handle *context_ndr = pBuff; TRACE("(%p %p %08x %p %u)\n", hBinding, pBuff, DataRepresentation, CtxGuard, Flags); @@ -330,12 +331,12 @@ NDR_SCONTEXT WINAPI NDRSContextUnmarshall2(RPC_BINDING_HANDLE hBinding, if (Flags & RPC_CONTEXT_HANDLE_FLAGS) FIXME("unimplemented flags: 0x%x\n", Flags & RPC_CONTEXT_HANDLE_FLAGS); - if (!pBuff) + if (!pBuff || (!context_ndr->attributes && + UuidIsNil((UUID *)&context_ndr->uuid, &status))) status = RpcServerAssoc_AllocateContextHandle(binding->Assoc, CtxGuard, &SContext); else { - const ndr_context_handle *context_ndr = pBuff; if (context_ndr->attributes) { ERR("non-null attributes 0x%x\n", context_ndr->attributes);

1 0

Rob Shearman : rpcrt4: Don' t print fixmes for NDR types without a freeing function.
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: 4402daa00b29832cf24ea0847a2c49200f3d3fb6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4402daa00b29832cf24ea0847… Author: Rob Shearman <rob(a)codeweavers.com> Date: Sun Jan 13 20:53:25 2008 +0000 rpcrt4: Don't print fixmes for NDR types without a freeing function. Not all types have a freeing function. --- dlls/rpcrt4/ndr_marshall.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/dlls/rpcrt4/ndr_marshall.c b/dlls/rpcrt4/ndr_marshall.c index d99fbf9..7dcfa51 100644 --- a/dlls/rpcrt4/ndr_marshall.c +++ b/dlls/rpcrt4/ndr_marshall.c @@ -2568,7 +2568,6 @@ static unsigned char * ComplexFree(PMIDL_STUB_MESSAGE pStubMsg, else m(pStubMsg, pMemory, desc); } - else FIXME("no freer for embedded type %02x\n", *desc); pMemory += size; pFormat += 2; continue; @@ -5245,7 +5244,6 @@ static void union_arm_free(PMIDL_STUB_MESSAGE pStubMsg, m(pStubMsg, pMemory, desc); } } - else FIXME("no freer for embedded type %02x\n", *desc); } }

1 0

Eric Pouech : winedbg: Fixed bt all so that current process/thread isn' t lost.
by Alexandre Julliard 14 Jan '08

14 Jan '08

Module: wine Branch: master Commit: f7908160eb898d4d6814fe886d4877f80a19c2c2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f7908160eb898d4d6814fe886… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sun Jan 13 17:02:29 2008 +0100 winedbg: Fixed bt all so that current process/thread isn't lost. --- programs/winedbg/stack.c | 20 ++++++++++++++++---- 1 files changed, 16 insertions(+), 4 deletions(-) diff --git a/programs/winedbg/stack.c b/programs/winedbg/stack.c index a416c8b..63103c1 100644 --- a/programs/winedbg/stack.c +++ b/programs/winedbg/stack.c @@ -351,6 +351,9 @@ static void backtrace_tid(struct dbg_process* pcs, DWORD tid) static void backtrace_all(void) { struct dbg_process* process = dbg_curr_process; + struct dbg_thread* thread = dbg_curr_thread; + CONTEXT ctx = dbg_context; + DWORD cpid = dbg_curr_pid; THREADENTRY32 entry; HANDLE snapshot = CreateToolhelp32Snapshot(TH32CS_SNAPTHREAD, 0); @@ -366,10 +369,16 @@ static void backtrace_all(void) do { if (entry.th32OwnerProcessID == GetCurrentProcessId()) continue; - if (dbg_curr_process && dbg_curr_pid != entry.th32OwnerProcessID) + if (dbg_curr_process && dbg_curr_pid != entry.th32OwnerProcessID && + cpid != dbg_curr_pid) dbg_curr_process->process_io->close_process(dbg_curr_process, FALSE); - if (entry.th32OwnerProcessID != dbg_curr_pid) + if (entry.th32OwnerProcessID == cpid) + { + dbg_curr_process = process; + dbg_curr_pid = cpid; + } + else if (entry.th32OwnerProcessID != dbg_curr_pid) { if (!dbg_attach_debuggee(entry.th32OwnerProcessID, FALSE)) { @@ -387,12 +396,15 @@ static void backtrace_all(void) } while (Thread32Next(snapshot, &entry)); - if (dbg_curr_process) + if (dbg_curr_process && cpid != dbg_curr_pid) dbg_curr_process->process_io->close_process(dbg_curr_process, FALSE); } CloseHandle(snapshot); dbg_curr_process = process; - dbg_curr_pid = process ? process->pid : 0; + dbg_curr_pid = cpid; + dbg_curr_thread = thread; + dbg_curr_tid = thread ? thread->tid : 0; + dbg_context = ctx; } void stack_backtrace(DWORD tid)

1 0