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

Re: [PATCH v2] msacm32: Rewrite PCM conversion functions.
by Marvin 13 Jun '17

13 Jun '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=31625 Your paranoid android. === build (build) === Patch failed to apply

2 1

Re: [2/2] ntdll: Add support for relay debugging on ARM64
by Alexandre Julliard 13 Jun '17

13 Jun '17

André Hentschel <nerv(a)dawncrow.de> writes: > Signed-off-by: André Hentschel <nerv(a)dawncrow.de> > --- > dlls/ntdll/relay.c | 65 +++++++++++++++++++++++++++++++++++++++++++++++++++--- > 1 file changed, 62 insertions(+), 3 deletions(-) It doesn't build with clang: aarch64-linux-android-clang -c -o relay.o ../../../wine/dlls/ntdll/relay.c -I. -I../../../wine/dlls/ntdll -I../../include -I../../../wine/include -D__WINESRC__ -D_NTSYSTEM_ -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Wstrict-prototypes -Wtype-limits -Wvla -Wwrite-strings -Wpointer-arith -gdwarf-2 -gstrict-dwarf -g -O2 <inline asm>:35:2: error: too few operands for instruction Makefile:561: recipe for target 'relay.o' failed -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [Tools v2] testbot/TestLauncher: Provide proper replacements for the str*_s() functions.
by Alexandre Julliard 13 Jun '17

13 Jun '17

Francois Gouget <fgouget(a)codeweavers.com> writes: > Windows XP's msvcrt does not have the str*_s() functions but mapping > them to their non _s() counterparts resulted in the test name sometimes > not being NUL-terminated. As a result the start and done lines would not > be recognized. > > An alternative solution would be to eschew the buffer safety and switch > to the non _s() functions with a couple of minor code adjustments. That would be preferable IMO. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 3/3] msacm32/tests: Add tests for PCM conversion.
by Marvin 13 Jun '17

13 Jun '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=31623 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 2/3] msacm32/tests: Move stream tests to separate file.
by Marvin 13 Jun '17

13 Jun '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=31622 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH v2 2/2] winemp3.acm: Fix error in acmStreamSize().
by Andrew Eikum 13 Jun '17

13 Jun '17

Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> On Mon, Jun 12, 2017 at 02:36:36PM -0500, Zebediah Figura wrote: > Fixes https://bugs.winehq.org/show_bug.cgi?id=33740. > > Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> > --- > dlls/winemp3.acm/mpegl3.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/dlls/winemp3.acm/mpegl3.c b/dlls/winemp3.acm/mpegl3.c > index cc6008da59e..13a6109a77a 100644 > --- a/dlls/winemp3.acm/mpegl3.c > +++ b/dlls/winemp3.acm/mpegl3.c > @@ -940,11 +940,11 @@ static LRESULT MPEG3_StreamSize(PACMDRVSTREAMINSTANCE adsi, PACMDRVSTREAMSIZE ad > adsi->pwfxDst->wFormatTag == WAVE_FORMAT_MPEG)) > { > nblocks = adss->cbSrcLength / (adsi->pwfxSrc->nBlockAlign * 1152); > - if (nblocks == 0) > - return ACMERR_NOTPOSSIBLE; > if (adss->cbSrcLength % (DWORD)(adsi->pwfxSrc->nBlockAlign * 1152)) > /* Round block count up. */ > nblocks++; > + if (nblocks == 0) > + return ACMERR_NOTPOSSIBLE; > adss->cbDstLength = 3000 + nblocks * (DWORD)(adsi->pwfxDst->nAvgBytesPerSec * 1152 / adsi->pwfxDst->nSamplesPerSec + 0.5); > } > else if ((adsi->pwfxSrc->wFormatTag == WAVE_FORMAT_MPEGLAYER3 || > @@ -952,11 +952,11 @@ static LRESULT MPEG3_StreamSize(PACMDRVSTREAMINSTANCE adsi, PACMDRVSTREAMSIZE ad > adsi->pwfxDst->wFormatTag == WAVE_FORMAT_PCM) > { > nblocks = adss->cbSrcLength / (DWORD)(adsi->pwfxSrc->nAvgBytesPerSec * 1152 / adsi->pwfxSrc->nSamplesPerSec); > - if (nblocks == 0) > - return ACMERR_NOTPOSSIBLE; > if (adss->cbSrcLength % (DWORD)(adsi->pwfxSrc->nAvgBytesPerSec * 1152 / adsi->pwfxSrc->nSamplesPerSec)) > /* Round block count up. */ > nblocks++; > + if (nblocks == 0) > + return ACMERR_NOTPOSSIBLE; > adss->cbDstLength = nblocks * 1152 * adsi->pwfxDst->nBlockAlign; > } > else > -- > 2.13.0 > > >

1 0

Re: [PATCH] dinput: Limit axes remap and fix loop variable in the js driver
by Sebastian Lackner 13 Jun '17

13 Jun '17

On 13.06.2017 08:06, Bruno Jesus wrote: > Tested with the Xbox 360 controller, Wii Classic controler, PS2 controller + usb converter and a virtual device with 24 axes. > > Fixes https://bugs.winehq.org/show_bug.cgi?id=43120 > > Signed-off-by: Bruno Jesus <bjesus(a)codeweavers.com> > --- > dlls/dinput/joystick_linux.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/dlls/dinput/joystick_linux.c b/dlls/dinput/joystick_linux.c > index 1032659621..dea8f17324 100644 > --- a/dlls/dinput/joystick_linux.c > +++ b/dlls/dinput/joystick_linux.c > @@ -253,9 +253,11 @@ static INT find_joystick_devices(void) > /* If no axes were configured but there are axes assume a 1-to-1 (wii controller) */ > if (joydev.axis_count && !found_axes) > { > + int axes_limit = min(joydev.axis_count, 8); /* generic driver limit */ Is it intentional that you are ignoring any POV sticks here? The limit of 8 only seems to apply to the buttons. > + > ERR("Incoherent joystick data, advertised %d axes, detected 0. Assuming 1-to-1.\n", > joydev.axis_count); > - for (j = 0; j < joydev.axis_count; j++) > + for (j = 0; j < axes_limit; j++) > joydev.dev_axes_map[j] = j; > } > } > @@ -472,7 +474,7 @@ static HRESULT alloc_device(REFGUID rguid, IDirectInputImpl *dinput, > df->dwNumObjs = newDevice->generic.devcaps.dwAxes + newDevice->generic.devcaps.dwPOVs + newDevice->generic.devcaps.dwButtons; > if (!(df->rgodf = HeapAlloc(GetProcessHeap(), 0, df->dwNumObjs * df->dwObjSize))) goto FAILED; > > - for (i = 0; i < newDevice->generic.device_axis_count; i++) > + for (i = 0; i < newDevice->generic.devcaps.dwAxes; i++) > { > int wine_obj = newDevice->generic.axis_map[i]; > > -- 2.11.0 > This change looks very suspicious because you are changing the for-loop, but not the dwNumObjs counter above. You are allocating dwPOVs additional entries which are now unused after this change. Is this really correct, and did you test with a joystick with POV switch? Best regards, Sebastian

2 1

Re: [2/2] ws2_32/tests: Test IP address conversion in getaddrinfo.
by Bruno Jesus 12 Jun '17

12 Jun '17

On Thu, Jun 8, 2017 at 6:26 PM, Owen Rudge <orudge(a)codeweavers.com> wrote: > Signed-off-by: Owen Rudge <orudge(a)codeweavers.com> > --- > dlls/ws2_32/tests/sock.c | 101 > +++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 101 insertions(+) It looks like this patch is missing some freeaddrinfo in the tests related to verify_ipv6_addrinfo, and could be simplified a bit , I'm making minor changes and will submit a new version of the series.

2 1

Re: [PATCH] wmvcore: add WMCheckURLScheme stub
by Alexandre Julliard 12 Jun '17

12 Jun '17

Austin English <austinenglish(a)gmail.com> writes: > +HRESULT WINAPI WMCheckURLScheme(const WCHAR *scheme) > +{ > + FIXME("(%s): stub\n", wine_dbgstr_w(scheme)); > + > + return ERROR_INVALID_NAME; That's not an HRESULT. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 1/4] include: Pack WAVEFORMAT structs.
by Alexandre Julliard 12 Jun '17

12 Jun '17

Andrew Eikum <aeikum(a)codeweavers.com> writes: > This looks OK to me, and fixes the size of the MPEGLAYER3WAVEFORMAT > structure to match native, but I don't know how to use our packing > headers well enough to be confident. It looks like the entire file should be using 1-byte packing. -- Alexandre Julliard julliard(a)winehq.org

1 0