November 2013 - Wine-Commits

Piotr Caban : qcap: Warn if additional filters should be used in ICaptureGraphBuilder2:: RenderStream.
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: f810d14149c7ca83fecb587e405e4ca5fd8defe7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f810d14149c7ca83fecb587e4… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Nov 20 12:01:59 2013 +0100 qcap: Warn if additional filters should be used in ICaptureGraphBuilder2::RenderStream. --- dlls/qcap/capturegraph.c | 24 ++++++++++++++++++++---- 1 files changed, 20 insertions(+), 4 deletions(-) diff --git a/dlls/qcap/capturegraph.c b/dlls/qcap/capturegraph.c index 5af4f3b..f5973c8 100644 --- a/dlls/qcap/capturegraph.c +++ b/dlls/qcap/capturegraph.c @@ -42,6 +42,7 @@ *#include "dshow.h" *#include "ddraw.h" */ +#include "uuids.h" #include "qcap_main.h" #include "wine/unicode.h" @@ -259,8 +260,7 @@ fnCaptureGraphBuilder2_RenderStream(ICaptureGraphBuilder2 * iface, IBaseFilter *pfRenderer) { CaptureGraphImpl *This = impl_from_ICaptureGraphBuilder2(iface); - IPin *source_out; - IPin *renderer_in; + IPin *source_out, *renderer_in, *capture, *preview; HRESULT hr; FIXME("(%p/%p)->(%s, %s, %p, %p, %p) semi-stub!\n", This, iface, @@ -281,14 +281,30 @@ fnCaptureGraphBuilder2_RenderStream(ICaptureGraphBuilder2 * iface, hr = ICaptureGraphBuilder2_FindPin(iface, pSource, PINDIR_OUTPUT, pCategory, pType, TRUE, 0, &source_out); if (FAILED(hr)) return E_INVALIDARG; - hr = ICaptureGraphBuilder2_FindPin(iface, (IUnknown*)pfRenderer, PINDIR_INPUT, pCategory, pType, TRUE, 0, &renderer_in); + + if (pCategory && IsEqualIID(pCategory, &PIN_CATEGORY_VBI)) { + FIXME("Tee/Sink-to-Sink filter not supported\n"); + IPin_Release(source_out); + return E_NOTIMPL; + } + + hr = ICaptureGraphBuilder2_FindPin(iface, pSource, PINDIR_OUTPUT, &PIN_CATEGORY_CAPTURE, NULL, TRUE, 0, &capture); + if (SUCCEEDED(hr)) { + hr = ICaptureGraphBuilder2_FindPin(iface, pSource, PINDIR_OUTPUT, &PIN_CATEGORY_PREVIEW, NULL, TRUE, 0, &preview); + if (FAILED(hr)) + FIXME("Smart Tee filter not supported - not creating preview pin\n"); + else + IPin_Release(preview); + IPin_Release(capture); + } + + hr = ICaptureGraphBuilder2_FindPin(iface, (IUnknown*)pfRenderer, PINDIR_INPUT, NULL, NULL, TRUE, 0, &renderer_in); if (FAILED(hr)) { IPin_Release(source_out); return hr; } - /* Uses 'Intelligent Connect', so Connect, not ConnectDirect here */ if (!pfCompressor) hr = IGraphBuilder_Connect(This->mygraph, source_out, renderer_in); else

1 0

Piotr Caban : qcap: Don' t ignore type and category parameters in ICaptureGraphBuilder2::FindPin.
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: 4adcca84f059ecb4a42c84b578560f11fe6f63c4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4adcca84f059ecb4a42c84b57… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Nov 20 12:01:27 2013 +0100 qcap: Don't ignore type and category parameters in ICaptureGraphBuilder2::FindPin. --- dlls/qcap/capturegraph.c | 90 +++++++++++++++++++++++++++++++++++---------- 1 files changed, 70 insertions(+), 20 deletions(-) diff --git a/dlls/qcap/capturegraph.c b/dlls/qcap/capturegraph.c index a852b61..5af4f3b 100644 --- a/dlls/qcap/capturegraph.c +++ b/dlls/qcap/capturegraph.c @@ -368,31 +368,73 @@ fnCaptureGraphBuilder2_CopyCaptureFile(ICaptureGraphBuilder2 * iface, return E_NOTIMPL; } -static BOOL pin_matches(IPin *pin, PIN_DIRECTION direction, const GUID *cat, const GUID *type, BOOL unconnected) +static HRESULT pin_matches(IPin *pin, PIN_DIRECTION direction, const GUID *cat, const GUID *type, BOOL unconnected) { IPin *partner; PIN_DIRECTION pindir; + HRESULT hr; - IPin_QueryDirection(pin, &pindir); - if (pindir != direction) - { - TRACE("No match, wrong direction\n"); - return FALSE; - } + hr = IPin_QueryDirection(pin, &pindir); - if (unconnected && IPin_ConnectedTo(pin, &partner) == S_OK) + if (unconnected && IPin_ConnectedTo(pin, &partner) == S_OK && partner!=NULL) { IPin_Release(partner); TRACE("No match, %p already connected to %p\n", pin, partner); - return FALSE; + return FAILED(hr) ? hr : S_FALSE; } - if (cat || type) - FIXME("Ignoring category/type\n"); + if (FAILED(hr)) + return hr; + if (SUCCEEDED(hr) && pindir != direction) + return S_FALSE; + + if (cat) + { + IKsPropertySet *props; + GUID category; + DWORD fetched; + + hr = IPin_QueryInterface(pin, &IID_IKsPropertySet, (void**)&props); + if (FAILED(hr)) + return S_FALSE; - TRACE("Match made in heaven\n"); + hr = IKsPropertySet_Get(props, &AMPROPSETID_Pin, 0, NULL, + 0, &category, sizeof(category), &fetched); + IKsPropertySet_Release(props); + if (FAILED(hr) || !IsEqualIID(&category, cat)) + return S_FALSE; + } + + if (type) + { + IEnumMediaTypes *types; + AM_MEDIA_TYPE *media_type; + ULONG fetched; + + hr = IPin_EnumMediaTypes(pin, &types); + if (FAILED(hr)) + return S_FALSE; + + IEnumMediaTypes_Reset(types); + while (1) { + if (IEnumMediaTypes_Next(types, 1, &media_type, &fetched) != S_OK || fetched != 1) + { + IEnumMediaTypes_Release(types); + return S_FALSE; + } - return TRUE; + if (IsEqualIID(&media_type->majortype, type)) + { + DeleteMediaType(media_type); + break; + } + DeleteMediaType(media_type); + } + IEnumMediaTypes_Release(types); + } + + TRACE("Pin matched\n"); + return S_OK; } static HRESULT WINAPI @@ -426,7 +468,7 @@ fnCaptureGraphBuilder2_FindPin(ICaptureGraphBuilder2 * iface, if (hr == E_NOINTERFACE) { WARN("Input not filter or pin?!\n"); - return E_FAIL; + return E_NOINTERFACE; } hr = IBaseFilter_EnumPins(filter, &enumpins); @@ -436,11 +478,11 @@ fnCaptureGraphBuilder2_FindPin(ICaptureGraphBuilder2 * iface, return hr; } - IEnumPins_Reset(enumpins); - while (1) { - hr = IEnumPins_Next(enumpins, 1, &pin, NULL); + ULONG fetched; + + hr = IEnumPins_Next(enumpins, 1, &pin, &fetched); if (hr == VFW_E_ENUM_OUT_OF_SYNC) { numcurrent = 0; @@ -448,14 +490,22 @@ fnCaptureGraphBuilder2_FindPin(ICaptureGraphBuilder2 * iface, pin = NULL; continue; } - if (hr != S_OK) break; + if (fetched != 1) + { + hr = E_FAIL; + break; + } + TRACE("Testing match\n"); - if (pin_matches(pin, pindir, pCategory, pType, fUnconnected) && numcurrent++ == num) + hr = pin_matches(pin, pindir, pCategory, pType, fUnconnected); + if (hr == S_OK && numcurrent++ == num) break; IPin_Release(pin); pin = NULL; + if (FAILED(hr)) + break; } IEnumPins_Release(enumpins); @@ -465,7 +515,7 @@ fnCaptureGraphBuilder2_FindPin(ICaptureGraphBuilder2 * iface, return E_FAIL; } } - else if (!pin_matches(pin, pindir, pCategory, pType, fUnconnected)) + else if (pin_matches(pin, pindir, pCategory, pType, fUnconnected) != S_OK) { IPin_Release(pin); return E_FAIL;

1 0

Piotr Caban : qcap: Support compression filter in ICaptureGraphBuilder2:: RenderStream.
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: badc77d522f64112f7e7bc6f7d5d07da058d553c URL: http://source.winehq.org/git/wine.git/?a=commit;h=badc77d522f64112f7e7bc6f7… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Nov 20 12:00:07 2013 +0100 qcap: Support compression filter in ICaptureGraphBuilder2::RenderStream. --- dlls/qcap/capturegraph.c | 59 +++++++++++++++++++++++++++++++++------------ 1 files changed, 43 insertions(+), 16 deletions(-) diff --git a/dlls/qcap/capturegraph.c b/dlls/qcap/capturegraph.c index 7cec77e..a852b61 100644 --- a/dlls/qcap/capturegraph.c +++ b/dlls/qcap/capturegraph.c @@ -259,37 +259,64 @@ fnCaptureGraphBuilder2_RenderStream(ICaptureGraphBuilder2 * iface, IBaseFilter *pfRenderer) { CaptureGraphImpl *This = impl_from_ICaptureGraphBuilder2(iface); - IPin *pin_in = NULL; - IPin *pin_out = NULL; + IPin *source_out; + IPin *renderer_in; HRESULT hr; - FIXME("(%p/%p)->(%s, %s, %p, %p, %p) Stub!\n", This, iface, + FIXME("(%p/%p)->(%s, %s, %p, %p, %p) semi-stub!\n", This, iface, debugstr_guid(pCategory), debugstr_guid(pType), pSource, pfCompressor, pfRenderer); - if (pfCompressor) - FIXME("Intermediate streams not supported yet\n"); - if (!This->mygraph) { FIXME("Need a capture graph\n"); return E_UNEXPECTED; } + if (!pfRenderer) + { + FIXME("pfRenderer == NULL not yet supported\n"); + return E_NOTIMPL; + } - ICaptureGraphBuilder2_FindPin(iface, pSource, PINDIR_OUTPUT, pCategory, pType, TRUE, 0, &pin_in); - if (!pin_in) - return E_FAIL; - ICaptureGraphBuilder2_FindPin(iface, (IUnknown*)pfRenderer, PINDIR_INPUT, pCategory, pType, TRUE, 0, &pin_out); - if (!pin_out) + hr = ICaptureGraphBuilder2_FindPin(iface, pSource, PINDIR_OUTPUT, pCategory, pType, TRUE, 0, &source_out); + if (FAILED(hr)) + return E_INVALIDARG; + hr = ICaptureGraphBuilder2_FindPin(iface, (IUnknown*)pfRenderer, PINDIR_INPUT, pCategory, pType, TRUE, 0, &renderer_in); + if (FAILED(hr)) { - IPin_Release(pin_in); - return E_FAIL; + IPin_Release(source_out); + return hr; } /* Uses 'Intelligent Connect', so Connect, not ConnectDirect here */ - hr = IGraphBuilder_Connect(This->mygraph, pin_in, pin_out); - IPin_Release(pin_in); - IPin_Release(pin_out); + if (!pfCompressor) + hr = IGraphBuilder_Connect(This->mygraph, source_out, renderer_in); + else + { + IPin *compressor_in, *compressor_out; + + hr = ICaptureGraphBuilder2_FindPin(iface, (IUnknown*)pfCompressor, + PINDIR_INPUT, NULL, NULL, TRUE, 0, &compressor_in); + if (SUCCEEDED(hr)) + { + hr = IGraphBuilder_Connect(This->mygraph, source_out, compressor_in); + IPin_Release(compressor_in); + } + + if (SUCCEEDED(hr)) + { + hr = ICaptureGraphBuilder2_FindPin(iface, (IUnknown*)pfCompressor, + PINDIR_OUTPUT, NULL, NULL, TRUE, 0, &compressor_out); + if (SUCCEEDED(hr)) + { + hr = IGraphBuilder_Connect(This->mygraph, compressor_out, renderer_in); + IPin_Release(compressor_out); + } + } + } + + IPin_Release(source_out); + IPin_Release(renderer_in); return hr; }

1 0

Hans Leidekker : netapi32: Add support for info level 502 in NetShareAdd.
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: d9723d2bedab2bdba3250cfd2e4bb74ad359bb23 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d9723d2bedab2bdba3250cfd2… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 20 10:30:32 2013 +0100 netapi32: Add support for info level 502 in NetShareAdd. --- dlls/netapi32/netapi32.c | 463 ++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 463 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d9723d2bedab2bdba3250…

1 0

Hans Leidekker : netapi32: The last parameter to Samba' s NetShareAdd is not optional.
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: 466f435a3c9840a9b30f2f589550d8d43651721c URL: http://source.winehq.org/git/wine.git/?a=commit;h=466f435a3c9840a9b30f2f589… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 20 10:32:12 2013 +0100 netapi32: The last parameter to Samba's NetShareAdd is not optional. --- dlls/netapi32/netapi32.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dlls/netapi32/netapi32.c b/dlls/netapi32/netapi32.c index d2516d8..832fdbe 100644 --- a/dlls/netapi32/netapi32.c +++ b/dlls/netapi32/netapi32.c @@ -782,8 +782,11 @@ static NET_API_STATUS share_add( LMSTR servername, DWORD level, LPBYTE buf, LPDW status = share_info_to_samba( level, buf, &info ); if (!status) { - status = pNetShareAdd( server, level, info, parm_err ); + unsigned int err; + + status = pNetShareAdd( server, level, info, &err ); HeapFree( GetProcessHeap(), 0, info ); + if (parm_err) *parm_err = err; } HeapFree( GetProcessHeap(), 0, server ); return status;

1 0

Hans Leidekker : netapi32: Add helpers to initialize and destroy a libnetapi context.
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: 499eed087ec36d0344b403ff9b5800fd2a4053fc URL: http://source.winehq.org/git/wine.git/?a=commit;h=499eed087ec36d0344b403ff9… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 20 10:30:01 2013 +0100 netapi32: Add helpers to initialize and destroy a libnetapi context. --- dlls/netapi32/netapi32.c | 68 +++++++++++++++++++++++++++------------------- 1 files changed, 40 insertions(+), 28 deletions(-) diff --git a/dlls/netapi32/netapi32.c b/dlls/netapi32/netapi32.c index 74635dc..e6f3459 100644 --- a/dlls/netapi32/netapi32.c +++ b/dlls/netapi32/netapi32.c @@ -85,10 +85,48 @@ static NET_API_STATUS (*pNetShareAdd)(const char *, unsigned int, unsigned char static NET_API_STATUS (*pNetShareDel)(const char *, const char *, unsigned int); static NET_API_STATUS (*pNetWkstaGetInfo)(const char *, unsigned int, unsigned char **); +static void destroy_context(void) +{ + TRACE( "destroying %p\n", libnetapi_ctx ); + plibnetapi_free( libnetapi_ctx ); + libnetapi_ctx = NULL; +} + +static BOOL init_context(void) +{ + DWORD status; + + if ((status = plibnetapi_init( &libnetapi_ctx ))) + { + ERR( "Failed to initialize context %u\n", status ); + return FALSE; + } + if (TRACE_ON( netapi32 ) && (status = plibnetapi_set_debuglevel( libnetapi_ctx, "10" ))) + { + ERR( "Failed to set debug level %u\n", status ); + destroy_context(); + return FALSE; + } + /* perform an anonymous login by default (avoids a password prompt) */ + if ((status = plibnetapi_set_username( libnetapi_ctx, "Guest" ))) + { + ERR( "Failed to set username %u\n", status ); + destroy_context(); + return FALSE; + } + if ((status = plibnetapi_set_password( libnetapi_ctx, "" ))) + { + ERR( "Failed to set password %u\n", status ); + destroy_context(); + return FALSE; + } + TRACE( "using %p\n", libnetapi_ctx ); + return TRUE; +} + static BOOL libnetapi_init(void) { char buf[200]; - DWORD status; if (libnetapi_handle) return TRUE; if (!(libnetapi_handle = wine_dlopen( SONAME_LIBNETAPI, RTLD_NOW, buf, sizeof(buf) ))) @@ -118,35 +156,9 @@ static BOOL libnetapi_init(void) LOAD_FUNCPTR(NetWkstaGetInfo) #undef LOAD_FUNCPTR - if ((status = plibnetapi_init( &libnetapi_ctx ))) - { - ERR( "Failed to initialize context %u\n", status ); - goto error; - } - if (TRACE_ON( netapi32 ) && (status = plibnetapi_set_debuglevel( libnetapi_ctx, "10" ))) - { - ERR( "Failed to set debug level %u\n", status ); - goto error; - } - /* perform an anonymous login by default (avoids a password prompt) */ - if ((status = plibnetapi_set_username( libnetapi_ctx, "Guest" ))) - { - ERR( "Failed to set username %u\n", status ); - goto error; - } - if ((status = plibnetapi_set_password( libnetapi_ctx, "" ))) - { - ERR( "Failed to set password %u\n", status ); - goto error; - } - return TRUE; + if (init_context()) return TRUE; error: - if (libnetapi_ctx) - { - plibnetapi_free( libnetapi_ctx ); - libnetapi_ctx = NULL; - } wine_dlclose( libnetapi_handle, NULL, 0 ); libnetapi_handle = NULL; return FALSE;

1 0

Henri Verbeet : wined3d: Get rid of the redundant texture_name and texture_name_srgb fields in struct wined3d_surface .
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: 7e8897b7b84af07b7722911d2269daca79e3dd49 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7e8897b7b84af07b7722911d2… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Nov 20 10:04:32 2013 +0100 wined3d: Get rid of the redundant texture_name and texture_name_srgb fields in struct wined3d_surface. --- dlls/wined3d/context.c | 6 ++-- dlls/wined3d/surface.c | 54 +++++++++---------------------------- dlls/wined3d/swapchain.c | 2 +- dlls/wined3d/texture.c | 57 ++++++++++++--------------------------- dlls/wined3d/wined3d_private.h | 5 +--- 5 files changed, 36 insertions(+), 88 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=7e8897b7b84af07b77229…

1 0

Henri Verbeet : wined3d: Surfaces always have a container in arbfp_blit_set ().
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: e64b19387b2fa37fe2eeabbd76eeaf34af78d7cc URL: http://source.winehq.org/git/wine.git/?a=commit;h=e64b19387b2fa37fe2eeabbd7… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Nov 20 10:04:31 2013 +0100 wined3d: Surfaces always have a container in arbfp_blit_set(). --- dlls/wined3d/arb_program_shader.c | 7 +------ 1 files changed, 1 insertions(+), 6 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index 5efb562..aa62bfe 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -7302,12 +7302,7 @@ static HRESULT arbfp_blit_set(void *blit_priv, struct wined3d_context *context, struct arbfp_blit_priv *priv = blit_priv; enum complex_fixup fixup; const struct wined3d_gl_info *gl_info = context->gl_info; - GLenum textype; - - if (surface->container) - textype = surface->container->target; - else - textype = surface->texture_target; + GLenum textype = surface->container->target; if (surface->flags & SFLAG_CONVERTED) {

1 0

Henri Verbeet : wined3d: Surfaces always have a container in ffp_blit_set() .
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: ad6a194aa00bcdf1518b1c31748c4229b2b7adb0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ad6a194aa00bcdf1518b1c317… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Nov 20 10:04:30 2013 +0100 wined3d: Surfaces always have a container in ffp_blit_set(). --- dlls/wined3d/surface.c | 8 +------- 1 files changed, 1 insertions(+), 7 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 320a79b..c1714c1 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -5594,14 +5594,8 @@ static void ffp_blit_free(struct wined3d_device *device) { } static HRESULT ffp_blit_set(void *blit_priv, struct wined3d_context *context, const struct wined3d_surface *surface) { const struct wined3d_gl_info *gl_info = context->gl_info; - GLenum target; - if (surface->container) - target = surface->container->target; - else - target = surface->texture_target; - - gl_info->gl_ops.gl.p_glEnable(target); + gl_info->gl_ops.gl.p_glEnable(surface->container->target); checkGLcall("glEnable(target)"); return WINED3D_OK;

1 0

Henri Verbeet : wined3d: Surfaces always have a container in surface_invalidate_location().
by Alexandre Julliard Nov. 20, 2013

Nov. 20, 2013

Module: wine Branch: master Commit: 51f05a1f141a524dd66d4c1126fc203f828c68f3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=51f05a1f141a524dd66d4c112… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Nov 20 10:04:29 2013 +0100 wined3d: Surfaces always have a container in surface_invalidate_location(). --- dlls/wined3d/surface.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 60348ab..320a79b 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -5215,7 +5215,7 @@ void surface_invalidate_location(struct wined3d_surface *surface, DWORD location { TRACE("surface %p, location %s.\n", surface, debug_surflocation(location & SFLAG_LOCATIONS)); - if ((location & (SFLAG_INTEXTURE | SFLAG_INSRGBTEX)) && surface->container) + if (location & (SFLAG_INTEXTURE | SFLAG_INSRGBTEX)) wined3d_texture_set_dirty(surface->container); surface->flags &= ~(location & SFLAG_LOCATIONS);

1 0