June 2017 - Wine-commits - list.winehq.org

Dmitry Timoshkov : windowscodecs: Implement IWICBitmapDecoder:: GetMetadataQueryReader in the GIF decoder.
by Alexandre Julliard 26 Jun '17

26 Jun '17

Module: wine Branch: master Commit: d3280c867bc7b7e36657aa0f50db26d7258a15b9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d3280c867bc7b7e36657aa0f5… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Thu Jun 22 11:38:59 2017 +0800 windowscodecs: Implement IWICBitmapDecoder::GetMetadataQueryReader in the GIF decoder. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/windowscodecs/gifformat.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/dlls/windowscodecs/gifformat.c b/dlls/windowscodecs/gifformat.c index 01ef089..f63a7f3 100644 --- a/dlls/windowscodecs/gifformat.c +++ b/dlls/windowscodecs/gifformat.c @@ -1247,8 +1247,13 @@ static HRESULT WINAPI GifDecoder_CopyPalette(IWICBitmapDecoder *iface, IWICPalet static HRESULT WINAPI GifDecoder_GetMetadataQueryReader(IWICBitmapDecoder *iface, IWICMetadataQueryReader **ppIMetadataQueryReader) { + GifDecoder *This = impl_from_IWICBitmapDecoder(iface); + TRACE("(%p,%p)\n", iface, ppIMetadataQueryReader); - return WINCODEC_ERR_UNSUPPORTEDOPERATION; + + if (!ppIMetadataQueryReader) return E_INVALIDARG; + + return MetadataQueryReader_CreateInstance(&This->IWICMetadataBlockReader_iface, NULL, ppIMetadataQueryReader); } static HRESULT WINAPI GifDecoder_GetPreview(IWICBitmapDecoder *iface,

1 0

Clemens Tamme : gdiplus: Write API for GdipAddPathLine and GdipAddPathLineI.
by Alexandre Julliard 26 Jun '17

26 Jun '17

Module: wine Branch: master Commit: f96e319ab5b890e0d174d617ee93c7b3ecc9dec9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f96e319ab5b890e0d174d617e… Author: Clemens Tamme <clemens.tamme(a)gmail.com> Date: Wed Jun 21 19:50:41 2017 +0200 gdiplus: Write API for GdipAddPathLine and GdipAddPathLineI. Signed-off-by: Clemens Tamme <clemens.tamme(a)gmail.com> Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/graphicspath.c | 29 +++++++++++++++++++++++++++++ 1 file changed, 29 insertions(+) diff --git a/dlls/gdiplus/graphicspath.c b/dlls/gdiplus/graphicspath.c index e34f4ed..90d15bb 100644 --- a/dlls/gdiplus/graphicspath.c +++ b/dlls/gdiplus/graphicspath.c @@ -646,6 +646,30 @@ GpStatus WINGDIPAPI GdipAddPathLine2I(GpPath *path, GDIPCONST GpPoint *points, I return stat; } +/************************************************************************* + * GdipAddPathLine [GDIPLUS.21] + * + * Add two points to the given path. + * + * PARAMS + * path [I/O] Path that the line is appended to + * x1 [I] X coordinate of the first point of the line + * y1 [I] Y coordinate of the first point of the line + * x2 [I] X coordinate of the second point of the line + * y2 [I] Y coordinate of the second point of the line + * + * RETURNS + * InvalidParameter If the first parameter is not a valid path + * OutOfMemory If the path cannot be lengthened, i.e. memory allocation fails + * Ok If everything works out as expected + * + * NOTES + * This functions takes the newfigure value of the given path into account, + * i.e. the two new points are connected to the end of the given path if it + * was set to FALSE, otherwise the first point is given the PathPointTypeStart + * value. In both cases, the value of newfigure of the given path is FALSE + * afterwards. + */ GpStatus WINGDIPAPI GdipAddPathLine(GpPath *path, REAL x1, REAL y1, REAL x2, REAL y2) { INT old_count; @@ -675,6 +699,11 @@ GpStatus WINGDIPAPI GdipAddPathLine(GpPath *path, REAL x1, REAL y1, REAL x2, REA return Ok; } +/************************************************************************* + * GdipAddPathLineI [GDIPLUS.21] + * + * See GdipAddPathLine + */ GpStatus WINGDIPAPI GdipAddPathLineI(GpPath *path, INT x1, INT y1, INT x2, INT y2) { TRACE("(%p, %d, %d, %d, %d)\n", path, x1, y1, x2, y2);

1 0

Clemens Tamme : gdiplus: Write API for GdipClonePath.
by Alexandre Julliard 26 Jun '17

26 Jun '17

Module: wine Branch: master Commit: f1123f35788effc9ceea462210189600191d9966 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f1123f35788effc9ceea46221… Author: Clemens Tamme <clemens.tamme(a)gmail.com> Date: Wed Jun 21 21:40:35 2017 +0200 gdiplus: Write API for GdipClonePath. Signed-off-by: Clemens Tamme <clemens.tamme(a)gmail.com> Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/graphicspath.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/dlls/gdiplus/graphicspath.c b/dlls/gdiplus/graphicspath.c index 90d15bb..815ce34 100644 --- a/dlls/gdiplus/graphicspath.c +++ b/dlls/gdiplus/graphicspath.c @@ -1087,6 +1087,20 @@ GpStatus WINGDIPAPI GdipAddPathStringI(GpPath* path, GDIPCONST WCHAR* string, IN return InvalidParameter; } +/************************************************************************* + * GdipClonePath [GDIPLUS.53] + * + * Duplicate the given path in memory. + * + * PARAMS + * path [I] The path to be duplicated + * clone [O] Pointer to the new path + * + * RETURNS + * InvalidParameter If the input path is invalid + * OutOfMemory If allocation of needed memory fails + * Ok If everything works out as expected + */ GpStatus WINGDIPAPI GdipClonePath(GpPath* path, GpPath **clone) { TRACE("(%p, %p)\n", path, clone);

1 0

Clemens Tamme : gdiplus: Write API documentation for GdipAddPathArc and GdipAddPathArcI.
by Alexandre Julliard 26 Jun '17

26 Jun '17

Module: wine Branch: master Commit: c95877da42c2eb03085edf54a114cd0c766ad184 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c95877da42c2eb03085edf54a… Author: Clemens Tamme <clemens.tamme(a)gmail.com> Date: Thu Jun 22 18:47:42 2017 +0200 gdiplus: Write API documentation for GdipAddPathArc and GdipAddPathArcI. Signed-off-by: Clemens Tamme <clemens.tamme(a)gmail.com> Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/graphicspath.c | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/dlls/gdiplus/graphicspath.c b/dlls/gdiplus/graphicspath.c index 815ce34..5fedd07 100644 --- a/dlls/gdiplus/graphicspath.c +++ b/dlls/gdiplus/graphicspath.c @@ -166,6 +166,32 @@ static BOOL flatten_bezier(path_list_node_t *start, REAL x2, REAL y2, REAL x3, R return TRUE; } +/******************************************************************************* + * GdipAddPathArc [GDIPLUS.1] + * + * Add an elliptical arc to the given path. + * + * PARAMS + * path [I/O] Path that the arc is appended to + * x1 [I] X coordinate of the boundary box + * y1 [I] Y coordinate of the boundary box + * x2 [I] Width of the boundary box + * y2 [I] Height of the boundary box + * startAngle [I] Starting angle of the arc, clockwise + * sweepAngle [I] Angle of the arc, clockwise + * + * RETURNS + * InvalidParameter If the given path is invalid + * OutOfMemory If memory allocation fails, i.e. the path cannot be lengthened + * Ok If everything works out as expected + * + * NOTES + * This functions takes the newfigure value of the given path into account, + * i.e. the arc is connected to the end of the given path if it was set to + * FALSE, otherwise the arc's first point gets the the PathPointTypeStart + * value. In both cases, the value of newfigure of the given path is FALSE + * afterwards. + */ GpStatus WINGDIPAPI GdipAddPathArc(GpPath *path, REAL x1, REAL y1, REAL x2, REAL y2, REAL startAngle, REAL sweepAngle) { @@ -200,6 +226,11 @@ GpStatus WINGDIPAPI GdipAddPathArc(GpPath *path, REAL x1, REAL y1, REAL x2, return Ok; } +/******************************************************************************* + * GdipAddPathArcI [GDUPLUS.2] + * + * See GdipAddPathArc + */ GpStatus WINGDIPAPI GdipAddPathArcI(GpPath *path, INT x1, INT y1, INT x2, INT y2, REAL startAngle, REAL sweepAngle) {

1 0

Owen Rudge : wsdapi: Add stub implementation of IWSDUdpMessageParameters and WSDCreateUdpMessageParameters.
by Alexandre Julliard 26 Jun '17

26 Jun '17

Module: wine Branch: master Commit: 240c703896dfcf2fba4267ca8d8e779195bc4caa URL: http://source.winehq.org/git/wine.git/?a=commit;h=240c703896dfcf2fba4267ca8… Author: Owen Rudge <orudge(a)codeweavers.com> Date: Tue Jun 20 22:25:50 2017 +0100 wsdapi: Add stub implementation of IWSDUdpMessageParameters and WSDCreateUdpMessageParameters. Signed-off-by: Owen Rudge <orudge(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wsdapi/Makefile.in | 1 + dlls/wsdapi/msgparams.c | 228 ++++++++++++++++++++++++++++++++++++++++++++++++ dlls/wsdapi/wsdapi.spec | 2 +- 3 files changed, 230 insertions(+), 1 deletion(-) diff --git a/dlls/wsdapi/Makefile.in b/dlls/wsdapi/Makefile.in index 30862cc..0f28ce5 100644 --- a/dlls/wsdapi/Makefile.in +++ b/dlls/wsdapi/Makefile.in @@ -7,4 +7,5 @@ C_SRCS = \ discovery.c \ main.c \ memory.c \ + msgparams.c \ xml.c diff --git a/dlls/wsdapi/msgparams.c b/dlls/wsdapi/msgparams.c new file mode 100644 index 0000000..a7a2f0a --- /dev/null +++ b/dlls/wsdapi/msgparams.c @@ -0,0 +1,228 @@ +/* + * Web Services on Devices + * + * Copyright 2017 Owen Rudge for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <stdarg.h> + +#define COBJMACROS + +#include "windef.h" +#include "winbase.h" +#include "wine/debug.h" +#include "wsdapi.h" + +WINE_DEFAULT_DEBUG_CHANNEL(wsdapi); + +typedef struct IWSDMessageParametersImpl { + IWSDMessageParameters IWSDMessageParameters_iface; + LONG ref; +} IWSDMessageParametersImpl; + +typedef struct IWSDUdpMessageParametersImpl { + IWSDMessageParametersImpl base; +} IWSDUdpMessageParametersImpl; + +static inline IWSDMessageParametersImpl *impl_from_IWSDMessageParameters(IWSDMessageParameters *iface) +{ + return CONTAINING_RECORD(iface, IWSDMessageParametersImpl, IWSDMessageParameters_iface); +} + +static inline IWSDUdpMessageParametersImpl *impl_from_IWSDUdpMessageParameters(IWSDUdpMessageParameters *iface) +{ + return CONTAINING_RECORD(iface, IWSDUdpMessageParametersImpl, base.IWSDMessageParameters_iface); +} + +/* IWSDMessageParameters implementation */ + +static ULONG WINAPI IWSDMessageParametersImpl_AddRef(IWSDMessageParameters *iface) +{ + IWSDMessageParametersImpl *This = impl_from_IWSDMessageParameters(iface); + ULONG ref = InterlockedIncrement(&This->ref); + + TRACE("(%p) ref=%d\n", This, ref); + return ref; +} + +static ULONG WINAPI IWSDMessageParametersImpl_Release(IWSDMessageParameters *iface) +{ + IWSDMessageParametersImpl *This = impl_from_IWSDMessageParameters(iface); + ULONG ref = InterlockedDecrement(&This->ref); + + TRACE("(%p) ref=%d\n", This, ref); + + if (ref == 0) + { + HeapFree(GetProcessHeap(), 0, This); + } + + return ref; +} + +static HRESULT WINAPI IWSDMessageParametersImpl_GetLocalAddress(IWSDMessageParameters *This, IWSDAddress **ppAddress) +{ + FIXME("(%p, %p)\n", This, ppAddress); + return E_NOTIMPL; +} + +static HRESULT WINAPI IWSDMessageParametersImpl_SetLocalAddress(IWSDMessageParameters *This, IWSDAddress *pAddress) +{ + FIXME("(%p, %p)\n", This, pAddress); + return E_NOTIMPL; +} + +static HRESULT WINAPI IWSDMessageParametersImpl_GetRemoteAddress(IWSDMessageParameters *This, IWSDAddress **ppAddress) +{ + FIXME("(%p, %p)\n", This, ppAddress); + return E_NOTIMPL; +} + +static HRESULT WINAPI IWSDMessageParametersImpl_SetRemoteAddress(IWSDMessageParameters *This, IWSDAddress *pAddress) +{ + FIXME("(%p, %p)\n", This, pAddress); + return E_NOTIMPL; +} + +static HRESULT WINAPI IWSDMessageParametersImpl_GetLowerParameters(IWSDMessageParameters *This, IWSDMessageParameters **ppTxParams) +{ + FIXME("(%p, %p)\n", This, ppTxParams); + return E_NOTIMPL; +} + +/* IWSDUdpMessageParameters implementation */ +static HRESULT WINAPI IWSDUdpMessageParametersImpl_QueryInterface(IWSDUdpMessageParameters *iface, REFIID riid, void **ppv) +{ + IWSDUdpMessageParametersImpl *This = impl_from_IWSDUdpMessageParameters(iface); + + TRACE("(%p, %s, %p)\n", This, debugstr_guid(riid), ppv); + + if (!ppv) + { + WARN("Invalid parameter\n"); + return E_INVALIDARG; + } + + *ppv = NULL; + + if (IsEqualIID(riid, &IID_IUnknown) || + IsEqualIID(riid, &IID_IWSDMessageParameters) || + IsEqualIID(riid, &IID_IWSDUdpMessageParameters)) + { + *ppv = &This->base.IWSDMessageParameters_iface; + } + else + { + WARN("Unknown IID %s\n", debugstr_guid(riid)); + return E_NOINTERFACE; + } + + IUnknown_AddRef((IUnknown*)*ppv); + return S_OK; +} + +static ULONG WINAPI IWSDUdpMessageParametersImpl_AddRef(IWSDUdpMessageParameters *iface) +{ + return IWSDMessageParametersImpl_AddRef((IWSDMessageParameters *)iface); +} + +static ULONG WINAPI IWSDUdpMessageParametersImpl_Release(IWSDUdpMessageParameters *iface) +{ + return IWSDMessageParametersImpl_Release((IWSDMessageParameters *)iface); +} + +static HRESULT WINAPI IWSDUdpMessageParametersImpl_GetLocalAddress(IWSDUdpMessageParameters *This, IWSDAddress **ppAddress) +{ + return IWSDMessageParametersImpl_GetLocalAddress((IWSDMessageParameters *)This, ppAddress); +} + +static HRESULT WINAPI IWSDUdpMessageParametersImpl_SetLocalAddress(IWSDUdpMessageParameters *This, IWSDAddress *pAddress) +{ + return IWSDMessageParametersImpl_SetLocalAddress((IWSDMessageParameters *)This, pAddress); +} + +static HRESULT WINAPI IWSDUdpMessageParametersImpl_GetRemoteAddress(IWSDUdpMessageParameters *This, IWSDAddress **ppAddress) +{ + return IWSDMessageParametersImpl_GetRemoteAddress((IWSDMessageParameters *)This, ppAddress); +} + +static HRESULT WINAPI IWSDUdpMessageParametersImpl_SetRemoteAddress(IWSDUdpMessageParameters *This, IWSDAddress *pAddress) +{ + return IWSDMessageParametersImpl_SetRemoteAddress((IWSDMessageParameters *)This, pAddress); +} + +static HRESULT WINAPI IWSDUdpMessageParametersImpl_GetLowerParameters(IWSDUdpMessageParameters *This, IWSDMessageParameters **ppTxParams) +{ + return IWSDMessageParametersImpl_GetLowerParameters((IWSDMessageParameters *)This, ppTxParams); +} + +static HRESULT WINAPI IWSDUdpMessageParametersImpl_SetRetransmitParams(IWSDUdpMessageParameters *This, const WSDUdpRetransmitParams *pParams) +{ + FIXME("(%p, %p)\n", This, pParams); + return E_NOTIMPL; +} + +static HRESULT WINAPI IWSDUdpMessageParametersImpl_GetRetransmitParams(IWSDUdpMessageParameters *This, WSDUdpRetransmitParams *pParams) +{ + FIXME("(%p, %p)\n", This, pParams); + return E_NOTIMPL; +} + +static const IWSDUdpMessageParametersVtbl udpMsgParamsVtbl = +{ + /* IUnknown */ + IWSDUdpMessageParametersImpl_QueryInterface, + IWSDUdpMessageParametersImpl_AddRef, + IWSDUdpMessageParametersImpl_Release, + + /* IWSDMessageParameters */ + IWSDUdpMessageParametersImpl_GetLocalAddress, + IWSDUdpMessageParametersImpl_SetLocalAddress, + IWSDUdpMessageParametersImpl_GetRemoteAddress, + IWSDUdpMessageParametersImpl_SetRemoteAddress, + IWSDUdpMessageParametersImpl_GetLowerParameters, + + /* IWSDUdpMessageParameters */ + IWSDUdpMessageParametersImpl_SetRetransmitParams, + IWSDUdpMessageParametersImpl_GetRetransmitParams +}; + +HRESULT WINAPI WSDCreateUdpMessageParameters(IWSDUdpMessageParameters **ppTxParams) +{ + IWSDUdpMessageParametersImpl *obj; + + TRACE("(%p)\n", ppTxParams); + + if (ppTxParams == NULL) + { + WARN("Invalid parameter: ppTxParams == NULL\n"); + return E_POINTER; + } + + *ppTxParams = NULL; + + obj = HeapAlloc(GetProcessHeap(), 0, sizeof(*obj)); + if (!obj) return E_OUTOFMEMORY; + + obj->base.IWSDMessageParameters_iface.lpVtbl = (IWSDMessageParametersVtbl *)&udpMsgParamsVtbl; + obj->base.ref = 1; + + *ppTxParams = (IWSDUdpMessageParameters *)&obj->base.IWSDMessageParameters_iface; + TRACE("Returning iface %p\n", *ppTxParams); + + return S_OK; +} diff --git a/dlls/wsdapi/wsdapi.spec b/dlls/wsdapi/wsdapi.spec index a8f1aaa..41d62bb 100644 --- a/dlls/wsdapi/wsdapi.spec +++ b/dlls/wsdapi/wsdapi.spec @@ -24,7 +24,7 @@ @ stub WSDCreateMetadataAgent @ stub WSDCreateOutboundAttachment @ stdcall WSDCreateUdpAddress(ptr) -@ stub WSDCreateUdpMessageParameters +@ stdcall WSDCreateUdpMessageParameters(ptr) @ stub WSDCreateUdpTransport @ stdcall WSDDetachLinkedMemory(ptr) @ stdcall WSDFreeLinkedMemory(ptr)

1 0

Owen Rudge : wsdapi/tests: Add tests for CreateUdpMessageParameters.
by Alexandre Julliard 26 Jun '17

26 Jun '17

Module: wine Branch: master Commit: d0abe772739ac9642653e3d61a0d29be5a6ffd6d URL: http://source.winehq.org/git/wine.git/?a=commit;h=d0abe772739ac9642653e3d61… Author: Owen Rudge <orudge(a)codeweavers.com> Date: Tue Jun 20 22:25:54 2017 +0100 wsdapi/tests: Add tests for CreateUdpMessageParameters. Signed-off-by: Owen Rudge <orudge(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wsdapi/tests/Makefile.in | 1 + dlls/wsdapi/tests/msgparams.c | 76 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 77 insertions(+) diff --git a/dlls/wsdapi/tests/Makefile.in b/dlls/wsdapi/tests/Makefile.in index e2f6642..cd4b418 100644 --- a/dlls/wsdapi/tests/Makefile.in +++ b/dlls/wsdapi/tests/Makefile.in @@ -5,4 +5,5 @@ C_SRCS = \ address.c \ discovery.c \ memory.c \ + msgparams.c \ xml.c diff --git a/dlls/wsdapi/tests/msgparams.c b/dlls/wsdapi/tests/msgparams.c new file mode 100644 index 0000000..0de1860 --- /dev/null +++ b/dlls/wsdapi/tests/msgparams.c @@ -0,0 +1,76 @@ +/* + * Web Services on Devices + * Message Parameters tests + * + * Copyright 2017 Owen Rudge for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#define COBJMACROS + +#include <winsock2.h> +#include <windows.h> + +#include "wine/test.h" +#include "objbase.h" +#include "wsdapi.h" + +static void CreateUdpMessageParameters_tests(void) +{ + IWSDUdpMessageParameters *udpMessageParams = NULL, *udpMessageParams2 = NULL; + IWSDMessageParameters *messageParams = NULL; + IUnknown *unknown; + HRESULT rc; + ULONG ref; + + rc = WSDCreateUdpMessageParameters(NULL); + ok((rc == E_POINTER) || (rc == E_INVALIDARG), "WSDCreateUdpMessageParameters(NULL) failed: %08x\n", rc); + + rc = WSDCreateUdpMessageParameters(&udpMessageParams); + ok(rc == S_OK, "WSDCreateUdpMessageParameters(NULL, &udpMessageParams) failed: %08x\n", rc); + ok(udpMessageParams != NULL, "WSDCreateUdpMessageParameters(NULL, &udpMessageParams) failed: udpMessageParams == NULL\n"); + + /* Try to query for objects */ + rc = IWSDUdpMessageParameters_QueryInterface(udpMessageParams, &IID_IWSDUdpMessageParameters, (LPVOID*)&udpMessageParams2); + ok(rc == S_OK, "IWSDUdpMessageParams_QueryInterface(IID_IWSDUdpMessageParameters) failed: %08x\n", rc); + + if (rc == S_OK) + IWSDUdpMessageParameters_Release(udpMessageParams2); + + rc = IWSDUdpMessageParameters_QueryInterface(udpMessageParams, &IID_IWSDMessageParameters, (LPVOID*)&messageParams); + ok(rc == S_OK, "IWSDUdpMessageParams_QueryInterface(IID_WSDMessageParameters) failed: %08x\n", rc); + + if (rc == S_OK) + IWSDMessageParameters_Release(messageParams); + + rc = IWSDUdpMessageParameters_QueryInterface(udpMessageParams, &IID_IUnknown, (LPVOID*)&unknown); + ok(rc == S_OK, "IWSDUdpMessageParams_QueryInterface(IID_IUnknown) failed: %08x\n", rc); + + if (rc == S_OK) + IUnknown_Release(unknown); + + ref = IWSDUdpMessageParameters_Release(udpMessageParams); + ok(ref == 0, "IWSDUdpMessageParameters_Release() has %d references, should have 0\n", ref); +} + +START_TEST(msgparams) +{ + CoInitialize(NULL); + + CreateUdpMessageParameters_tests(); + + CoUninitialize(); +}

1 0

Owen Rudge : wsdapi/tests: Add tests for Get/SetLocalAddress.
by Alexandre Julliard 26 Jun '17

26 Jun '17

Module: wine Branch: master Commit: f21d3836a297cafc2f70123b3193d7bb3112a884 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f21d3836a297cafc2f70123b3… Author: Owen Rudge <orudge(a)codeweavers.com> Date: Tue Jun 20 22:26:05 2017 +0100 wsdapi/tests: Add tests for Get/SetLocalAddress. Signed-off-by: Owen Rudge <orudge(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wsdapi/tests/msgparams.c | 59 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 59 insertions(+) diff --git a/dlls/wsdapi/tests/msgparams.c b/dlls/wsdapi/tests/msgparams.c index 0de1860..d4a300a 100644 --- a/dlls/wsdapi/tests/msgparams.c +++ b/dlls/wsdapi/tests/msgparams.c @@ -66,11 +66,70 @@ static void CreateUdpMessageParameters_tests(void) ok(ref == 0, "IWSDUdpMessageParameters_Release() has %d references, should have 0\n", ref); } +static void LocalAddress_tests(void) +{ + IWSDUdpAddress *origUdpAddress = NULL; + IWSDAddress *returnedAddress = NULL; + IWSDUdpMessageParameters *udpMessageParams = NULL; + WCHAR address[] = {'1','.','2','.','3','.','4',0}; + WSADATA wsaData; + HRESULT rc; + int ret; + + ret = WSAStartup(MAKEWORD(2, 2), &wsaData); + ok(ret == 0, "WSAStartup failed: %d\n", ret); + + rc = WSDCreateUdpMessageParameters(&udpMessageParams); + ok(rc == S_OK, "WSDCreateUdpMessageParameters(NULL, &udpMessageParams) failed: %08x\n", rc); + ok(udpMessageParams != NULL, "WSDCreateUdpMessageParameters(NULL, &udpMessageParams) failed: udpMessageParams == NULL\n"); + + rc = IWSDUdpMessageParameters_GetLocalAddress(udpMessageParams, NULL); + todo_wine ok(rc == E_POINTER, "GetLocalAddress failed: %08x\n", rc); + ok(returnedAddress == NULL, "GetLocalAddress returned %p\n", returnedAddress); + + rc = IWSDUdpMessageParameters_GetLocalAddress(udpMessageParams, &returnedAddress); + todo_wine ok(rc == E_ABORT, "GetLocalAddress failed: %08x\n", rc); + ok(returnedAddress == NULL, "GetLocalAddress returned %p\n", returnedAddress); + + rc = WSDCreateUdpAddress(&origUdpAddress); + ok(rc == S_OK, "WSDCreateUdpAddress(NULL, &origUdpAddress) failed: %08x\n", rc); + ok(origUdpAddress != NULL, "WSDCreateUdpMessageParameters(NULL, &origUdpAddress) failed: origUdpAddress == NULL\n"); + + rc = IWSDUdpAddress_SetTransportAddress(origUdpAddress, address); + todo_wine ok(rc == S_OK, "SetTransportAddress failed: %08x\n", rc); + + rc = IWSDUdpMessageParameters_SetLocalAddress(udpMessageParams, (IWSDAddress *)origUdpAddress); + todo_wine ok(rc == S_OK, "SetLocalAddress failed: %08x\n", rc); + + rc = IWSDUdpMessageParameters_GetLocalAddress(udpMessageParams, &returnedAddress); + todo_wine ok(rc == S_OK, "GetLocalAddress failed: %08x\n", rc); + todo_wine ok(returnedAddress != NULL, "GetLocalAddress returned NULL\n"); + + /* Check if GetLocalAddress returns the same object */ + todo_wine ok(returnedAddress == (IWSDAddress *)origUdpAddress, "returnedAddress != origUdpAddress\n"); + + ret = IWSDUdpMessageParameters_Release(udpMessageParams); + ok(ret == 0, "IWSDUdpMessageParameters_Release() has %d references, should have 0\n", ret); + + if (returnedAddress != NULL) + { + ret = IWSDAddress_Release(returnedAddress); + ok(ret == 1, "IWSDAddress_Release() has %d references, should have 1\n", ret); + } + + ret = IWSDUdpAddress_Release(origUdpAddress); + ok(ret == 0, "IWSDUdpMessageParameters_Release() has %d references, should have 0\n", ret); + + ret = WSACleanup(); + ok(ret == 0, "WSACleanup failed: %d\n", ret); +} + START_TEST(msgparams) { CoInitialize(NULL); CreateUdpMessageParameters_tests(); + LocalAddress_tests(); CoUninitialize(); }

1 0

Owen Rudge : wsdapi/tests: Add tests for Get/SetRemoteAddress.
by Alexandre Julliard 26 Jun '17

26 Jun '17

Module: wine Branch: master Commit: b0fc3197c5cf3599186444e03577c779a744b682 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b0fc3197c5cf3599186444e03… Author: Owen Rudge <orudge(a)codeweavers.com> Date: Tue Jun 20 22:26:10 2017 +0100 wsdapi/tests: Add tests for Get/SetRemoteAddress. Signed-off-by: Owen Rudge <orudge(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wsdapi/tests/msgparams.c | 59 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 59 insertions(+) diff --git a/dlls/wsdapi/tests/msgparams.c b/dlls/wsdapi/tests/msgparams.c index d4a300a..ba0590f 100644 --- a/dlls/wsdapi/tests/msgparams.c +++ b/dlls/wsdapi/tests/msgparams.c @@ -124,12 +124,71 @@ static void LocalAddress_tests(void) ok(ret == 0, "WSACleanup failed: %d\n", ret); } +static void RemoteAddress_tests(void) +{ + IWSDUdpAddress *origUdpAddress = NULL; + IWSDAddress *returnedAddress = NULL; + IWSDUdpMessageParameters *udpMessageParams = NULL; + WCHAR address[] = {'1','.','2','.','3','.','4',0}; + WSADATA wsaData; + HRESULT rc; + int ret; + + ret = WSAStartup(MAKEWORD(2, 2), &wsaData); + ok(ret == 0, "WSAStartup failed: %d\n", ret); + + rc = WSDCreateUdpMessageParameters(&udpMessageParams); + ok(rc == S_OK, "WSDCreateUdpMessageParameters(NULL, &udpMessageParams) failed: %08x\n", rc); + ok(udpMessageParams != NULL, "WSDCreateUdpMessageParameters(NULL, &udpMessageParams) failed: udpMessageParams == NULL\n"); + + rc = IWSDUdpMessageParameters_GetRemoteAddress(udpMessageParams, NULL); + todo_wine ok(rc == E_POINTER, "GetRemoteAddress failed: %08x\n", rc); + ok(returnedAddress == NULL, "GetRemoteAddress returned %p\n", returnedAddress); + + rc = IWSDUdpMessageParameters_GetRemoteAddress(udpMessageParams, &returnedAddress); + todo_wine ok(rc == E_ABORT, "GetRemoteAddress failed: %08x\n", rc); + ok(returnedAddress == NULL, "GetRemoteAddress returned %p\n", returnedAddress); + + rc = WSDCreateUdpAddress(&origUdpAddress); + ok(rc == S_OK, "WSDCreateUdpAddress(NULL, &origUdpAddress) failed: %08x\n", rc); + ok(origUdpAddress != NULL, "WSDCreateUdpMessageParameters(NULL, &origUdpAddress) failed: origUdpAddress == NULL\n"); + + rc = IWSDUdpAddress_SetTransportAddress(origUdpAddress, address); + todo_wine ok(rc == S_OK, "SetTransportAddress failed: %08x\n", rc); + + rc = IWSDUdpMessageParameters_SetRemoteAddress(udpMessageParams, (IWSDAddress *)origUdpAddress); + todo_wine ok(rc == S_OK, "SetRemoteAddress failed: %08x\n", rc); + + rc = IWSDUdpMessageParameters_GetRemoteAddress(udpMessageParams, &returnedAddress); + todo_wine ok(rc == S_OK, "GetRemoteAddress failed: %08x\n", rc); + todo_wine ok(returnedAddress != NULL, "GetLocalAddress returned NULL\n"); + + /* Check if GetRemoteAddress returns the same object */ + todo_wine ok(returnedAddress == (IWSDAddress *)origUdpAddress, "returnedAddress != origUdpAddress\n"); + + ret = IWSDUdpMessageParameters_Release(udpMessageParams); + ok(ret == 0, "IWSDUdpMessageParameters_Release() has %d references, should have 0\n", ret); + + if (returnedAddress != NULL) + { + ret = IWSDAddress_Release(returnedAddress); + ok(ret == 1, "IWSDAddress_Release() has %d references, should have 1\n", ret); + } + + ret = IWSDUdpAddress_Release(origUdpAddress); + ok(ret == 0, "IWSDUdpMessageParameters_Release() has %d references, should have 0\n", ret); + + ret = WSACleanup(); + ok(ret == 0, "WSACleanup failed: %d\n", ret); +} + START_TEST(msgparams) { CoInitialize(NULL); CreateUdpMessageParameters_tests(); LocalAddress_tests(); + RemoteAddress_tests(); CoUninitialize(); }

1 0

Rosanne DiMesio : Add Keywords filter to Browse Apps page
by Jeremy Newman 26 Jun '17

26 Jun '17

Module: appdb Branch: master Commit: 6dacbd9dd79d0a5428d038def8269bb90f586764 URL: http://source.winehq.org/git/appdb.git/?a=commit;h=6dacbd9dd79d0a5428d038de… Author: Rosanne DiMesio <dimesio(a)earthlink.net> Date: Mon Jun 26 13:50:40 2017 -0500 Add Keywords filter to Browse Apps page Fixes bug 38168. Signed-off-by: Rosanne DiMesio <dimesio(a)earthlink.net> Signed-off-by: Jeremy Newman <jnewman(a)codeweavers.com> --- include/application.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/application.php b/include/application.php index 75d13ee..8962915 100644 --- a/include/application.php +++ b/include/application.php @@ -1122,7 +1122,9 @@ EOT; $oFilter->AddFilterInfo('appCategory', 'Category', array(FILTER_OPTION_ENUM), FILTER_VALUES_OPTION_ENUM, $aCatIds, $aCatNames); $oFilter->AddFilterInfo('appVersion.license', 'License', array(FILTER_EQUALS), FILTER_VALUES_ENUM, $aLicenses); $oFilter->AddFilterInfo('appFamily.appName', 'Name', array(FILTER_CONTAINS, FILTER_STARTS_WITH, FILTER_ENDS_WITH), FILTER_VALUES_NORMAL); + $oFilter->AddFilterInfo('appFamily.keywords', 'Keywords', array(FILTER_CONTAINS, FILTER_STARTS_WITH, FILTER_ENDS_WITH), FILTER_VALUES_NORMAL); $oFilter->AddFilterInfo('onlyDownloadable', 'Only show downloadable apps', array(FILTER_OPTION_BOOL), FILTER_VALUES_OPTION_BOOL, array('false','true')); + return $oFilter; }

1 0

Rosanne DiMesio : viewbugs.php: order display by appName
by Jeremy Newman 26 Jun '17

26 Jun '17

Module: appdb Branch: master Commit: 6d372a8c42520994968639645b56dd930b826731 URL: http://source.winehq.org/git/appdb.git/?a=commit;h=6d372a8c4252099496863964… Author: Rosanne DiMesio <dimesio(a)earthlink.net> Date: Mon Jun 26 12:18:08 2017 -0500 viewbugs.php: order display by appName Fixes bug 27807. Signed-off-by: Rosanne DiMesio <dimesio(a)earthlink.net> Signed-off-by: Jeremy Newman <jnewman(a)codeweavers.com> --- viewbugs.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/viewbugs.php b/viewbugs.php index f9646da..a75ef30 100644 --- a/viewbugs.php +++ b/viewbugs.php @@ -26,7 +26,7 @@ $hResult = query_parameters("SELECT appFamily.description as appDescription, WHERE appFamily.appId = appVersion.appId and buglinks.versionId = appVersion.versionId AND buglinks.bug_id = '?' - ORDER BY versionName", $aClean['iBugId']); + ORDER BY appName, versionName", $aClean['iBugId']); if(query_num_rows($hResult)) {

1 0