November 2023 - wine-gitlab

Re: [PATCH v7 0/6] MR4449: winegstreamer: WMReader compressed output
by Zebediah Figura (＠zfigura) 30 Nov '23

30 Nov '23

Zebediah Figura (@zfigura) commented about dlls/winegstreamer/quartz_parser.c: > else > format->u.video_wmv.format = WG_WMV_VIDEO_FORMAT_UNKNOWN; > > + format->u.video_wmv.codec_data_len = mt->cbFormat - sizeof(VIDEOINFOHEADER); > + if (format->u.video_wmv.codec_data_len > sizeof(format->u.video_wmv.codec_data)) > + { > + WARN("Too big codec_data value (%u).\n", (UINT)format->u.video_wmv.codec_data_len); This cast should be unnecessary. I'd also make this an ERR or FIXME. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4449#note_54464

1 0

Re: [PATCH v7 0/6] MR4449: winegstreamer: WMReader compressed output
by Alfred Agrell (＠Alcaro) 30 Nov '23

30 Nov '23

On Thu Nov 30 21:42:02 2023 +0000, Zebediah Figura wrote: > The codec_data introduced in 1/6 isn't validated in the tests. Why is that? Because WMV version 1 doesn't have codec data. ``` $ gst-launch-1.0 -vvv filesrc location=dlls/wmvcore/tests/test.wmv ! asfdemux ! avdec_wmv1 ! fakesink [...] /GstPipeline:pipeline0/avdec_wmv1:avdec_wmv1-0.GstPad:sink: caps = video/x-wmv, wmvversion=(int)1, width=(int)64, height=(int)48, pixel-aspect-ratio=(fraction)1/1, format=(string)WMV1 $ gst-launch-1.0 -vvv filesrc location=waga.wmv ! asfdemux ! avdec_wmv3 ! fakesink [...] /GstPipeline:pipeline0/avdec_wmv3:avdec_wmv3-0.GstPad:sink: caps = video/x-wmv, wmvversion=(int)3, format=(string)WMV3, width=(int)1280, height=(int)720, codec_data=(buffer)4ff1080100, pixel-aspect-ratio=(fraction)1/1, framerate=(fraction)10000000/333333 ``` (Its absense is tested by the cbFormat check in compare_media_types().) But yes, certainly suboptimal. I'd be happy to test it if someone can find a suitable test file. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4449#note_54462

1 0

Re: [PATCH v7 0/6] MR4449: winegstreamer: WMReader compressed output
by Zebediah Figura (＠zfigura) 30 Nov '23

30 Nov '23

The codec_data introduced in 1/6 isn't validated in the tests. Why is that? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4449#note_54461

1 0

[PATCH v27 0/3] MR3504: ntdll: An implementation of SRWLOCK that closer matches Windows'.
by Yuxuan Shui (＠yshui) 30 Nov '23

30 Nov '23

This comes from behavioral study of Windows, which doesn't seem to check if the lock is actually exclusively held, and just simply decrement the value stored in the lock. This fixes a dead lock which prevents WeCom from starting up. -- v27: ntdll: allow SRWLOCKs to be quickly re-acquired ntdll: An implementation of SRWLOCK that closer matches Windows'. https://gitlab.winehq.org/wine/wine/-/merge_requests/3504

4 12

[PATCH 0/1] MR4495: setupapi: Add stubs for SetupDiGetCustomDeviceProperty{A|W}().
by Paul Gofman (＠gofman) 30 Nov '23

30 Nov '23

Fixes periodic (each ~0.5s) frame hitches / frame drops in Nickelodeon All Start Brawl 2. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4495

3 2

[PATCH v2 0/1] MR4548: vbscript: Sub argument has precedence over global const.
by Robert Wilhelm (＠sloper42) 30 Nov '23

30 Nov '23

https://bugs.winehq.org/show_bug.cgi?id=55502 -- v2: vbscript: Sub argument has precedence over global const and local dim. https://gitlab.winehq.org/wine/wine/-/merge_requests/4548

3 3

[PATCH v2 0/2] MR4559: gitlab: Add clang build.
by Jacek Caban (＠jacek) 30 Nov '23

30 Nov '23

-- v2: gitlab: Add clang build. https://gitlab.winehq.org/wine/wine/-/merge_requests/4559

2 2

[PATCH 0/1] MR4556: ntdll: Set correct neutral language for zh-HK, zh-MO, and zh-SG.
by Jactry Zeng (＠jactry) 30 Nov '23

30 Nov '23

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/4556

3 2

[PATCH 0/1] MR4568: gdiplus: Add GdipDrawImageFX() stub.
by Vijay Kiran Kamuju (＠infyquest) 30 Nov '23

30 Nov '23

Signed-off-by: Vijay Kiran Kamuju [infyquest@gmail.com](mailto:infyquest@gmail.com) Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55945 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4568

2 1

[PATCH v4 0/5] MR4441: Adding tests for CreateProcess and std handles.
by eric pouech (＠epo) 30 Nov '23

30 Nov '23

Various new tests for CreateProcess showing a couple of issues in current implementation. -- v4: kernelbase: Reset std handles gotten by GetStartupInfo(). ntdll: Restrict cases for std handle inheritance in CreateProcess(). kernel32/tests: Add more tests about CreateProcess. kernel32/tests: Enable CreateProcess() tests on 64bit compilation. kernel32/tests: Introduce a new infrastructure for testing CreateProcess(). https://gitlab.winehq.org/wine/wine/-/merge_requests/4441

4 10

2025

2024

2023

2022

wine-gitlab November 2023