November 2023 - Wine-GitLab

Re: [PATCH v2 0/1] MR4553: ntdll: Allow specifying per-process channels. - approved
by Alexandre Julliard (＠julliard) Nov. 30, 2023

Nov. 30, 2023

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4553

1 0

Re: [PATCH v4 0/2] MR4029: winmm: Reimplement joystick axis mapping. - approved
by Alexandre Julliard (＠julliard) Nov. 30, 2023

Nov. 30, 2023

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4029

1 0

[PATCH v2 0/5] MR4551: Server-side thread priorities implementation (Part 1)
by Marc-Aurel Zent (＠mzent) Nov. 30, 2023

Nov. 30, 2023

This is the current proton thread priority implementation by @rbernon rebased for upstream with a few `#ifdef`s added since AFAIK Linux is the only operating system where threads have a unique PID which can be used to set niceness on. I also ran `./tools/make_requests` on https://gitlab.winehq.org/mzent/wine/-/commit/6705d3481be0409f7e971c1d2c7a3… as well and `autoconf` on https://gitlab.winehq.org/mzent/wine/-/commit/d7bafe40c411753662b2ad97148a6… (which does blow up the line count a bit). A few tiny changes (with the ready variable for example) are in anticipation for Part 2, which also adds Mach thread priorities and recalculates thread priorities on process priority change. Since this is a rather large MR, I hope the split here is appropriate (with the second part being slightly smaller), but I think logically it makes the most sense here. -- v2: server: Check wineserver privileges on init with -20 https://gitlab.winehq.org/wine/wine/-/merge_requests/4551

3 13

Re: [PATCH v2 0/2] MR4532: uxtheme: Use CRT allocation functions. - approved
by Alexandre Julliard (＠julliard) Nov. 30, 2023

Nov. 30, 2023

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4532

1 0

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

Nov. 30, 2023

On Thu Nov 30 21:56:36 2023 +0000, Alfred Agrell wrote: > 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. If you've tested manually, that's probably fine. My concern is primarily that we're actually translating formats correctly; GStreamer and DirectShow don't always store them in the same way, and it's too easy to make assumptions (cf. [1]). [1] https://gitlab.winehq.org/wine/wine/-/merge_requests/3636#note_45288 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4449#note_54471

1 0

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

Nov. 30, 2023

Zebediah Figura (@zfigura) commented about dlls/wmvcore/tests/wmvcore.c: > ok(callback->next_pts[1] == 13270000, "Got pts %I64d.\n", callback->next_pts[1]); > ok(callback->sample_count > 0, "Got no samples.\n"); > callback->sample_count = 0; > + ok(!callback->todo_rewind, "Didn't rewind as expected\n"); This also seems unnecessary? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4449#note_54470

1 0

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

Nov. 30, 2023

Zebediah Figura (@zfigura) commented about dlls/wmvcore/tests/wmvcore.c: > todo_wine > ok(hr == E_UNEXPECTED, "Got hr %#lx.\n", hr); > > + if (winetest_platform_is_wine) I don't think you need the if (winetest_platform_is_wine) here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4449#note_54469

1 0

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

Nov. 30, 2023

Zebediah Figura (@zfigura) commented about dlls/wmvcore/tests/wmvcore.c: > trace("%lu: %04lx: IWMReaderCallback::OnSample(output %lu, time %I64u, duration %I64u, flags %#lx)\n", > GetTickCount(), GetCurrentThreadId(), output, time, duration, flags); > > + if (callback->last_pts[output] > time && callback->todo_rewind) > + { > + callback->todo_rewind = false; > + todo_wine ok(0, "changing compression state in Wine rewinds the stream\n"); > + callback->last_pts[0] = 0; > + callback->last_pts[1] = 0; > + } > + "if (x) ok(0)" should almost always be "ok(!x)". -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4449#note_54468

1 0

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

Nov. 30, 2023

Zebediah Figura (@zfigura) commented about dlls/winegstreamer/wm_reader.c: > > wg_parser_stream_release_buffer(stream->wg_stream); > > - if (!buffer->has_pts) > + if (buffer->has_pts) > + *pts = buffer->pts; > + else > FIXME("Missing PTS.\n"); > - if (!buffer->has_duration) > + if (buffer->has_duration) > + *duration = buffer->duration; > + else > FIXME("Missing duration.\n"); > > - *pts = buffer->pts; > - *duration = buffer->duration; This seems unrelated, and what's the motivation? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4449#note_54467

1 0

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

Nov. 30, 2023

Zebediah Figura (@zfigura) commented about dlls/winegstreamer/wm_reader.c: > parser = reader->wg_parser; > LeaveCriticalSection(&reader->init_cs); > > + if (!parser) > + { > + Sleep(10); > + continue; > + } > + This is ugly. Either we should explicitly block the read thread, or shut it down and restart it. The latter is probably simpler. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4449#note_54465

1 0