- wine-gitlab - winehq.org

[PATCH 0/1] MR1377: dsound: Use standard C functions for memory allocation in propset.c.
by Alex Henrie (＠alexhenrie) 16 Nov '22

16 Nov '22

The big win here is getting rid of the reimplementation of wcsdup. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1377

3 2

[PATCH 0/1] MR1375: ole32: Correct warning message in composite_get_leftmost.
by Alex Henrie (＠alexhenrie) 16 Nov '22

16 Nov '22

Looks like a copy-paste error. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1375

3 2

[PATCH 0/1] MR1378: sspicli: Use standard C functions for memory allocation.
by Alex Henrie (＠alexhenrie) 16 Nov '22

16 Nov '22

The big win here is getting rid of the reimplementation of wcsdup. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1378

2 1

[PATCH v11 0/10] MR1148: ntdll: Mark created files beginning with a dot as not hidden.
by Torge Matthies (＠tmatthies) 16 Nov '22

16 Nov '22

Also unifies hidden file handling. Fixes bug [53826](https://bugs.winehq.org/show_bug.cgi?id=53826). -- v11: ntdll: Try to avoid requesting the file name for a handle. ntdll: Ensure that attributes are kept the same when renaming or hard-linking a file. ntdll: Set xattr in NtCreateFile if inferred and requested attributes don't match. ntdll: Only infer hidden attribute from file name if xattr is not present. ntdll: Handle hidden file names in fd_get_file_info. ntdll: Pass file path into fd_get_file_info if available. ntdll: Do not compute file attributes for info classes that don't need them. ntdll: Handle hidden file names inside get_file_info instead of after it. ntdll: Do not open-code hidden file handling in get_dir_data_entry. ntdll/tests: Add test for file attributes of files with names beginning with a dot. https://gitlab.winehq.org/wine/wine/-/merge_requests/1148

2 10

Re: [PATCH v3 0/4] MR1311: wmvcore: Implement DedicatedDeliveryThread async reader output setting.
by Zebediah Figura (＠zfigura) 15 Nov '22

15 Nov '22

On Tue Nov 15 21:40:55 2022 +0000, Rémi Bernon wrote: > Because this is a blocking call, which can call application callbacks, > or otherwise also blocking until there's a sample ready. > We want to be able to deliver samples from the callback thread, on other > streams, without being blocked and without blocking the reader. There's > no point in making all this asynchronous otherwise. GetNextSample() already calls callbacks with the lock held. I.e. this is basically doing the same thing as described above. I also don't see how delivering samples can block this, or the reverse, since we drop the CS in order to deliver samples. What deadlock results from not dropping the CS here? I would guess that AllocateForOutput() is blocking in IMemAllocator::GetBuffer(), but if that's the case, according to my reading of 2/4, we should either be inside of IWMReaderCallback::OnSample() or still holding the CS and about to call it. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1311#note_15909

1 0

Re: [PATCH v3 0/4] MR1311: wmvcore: Implement DedicatedDeliveryThread async reader output setting.
by Zebediah Figura (＠zfigura) 15 Nov '22

15 Nov '22

On Tue Nov 15 21:40:55 2022 +0000, Rémi Bernon wrote: > The most common case where this happens is on EOS, I don't think we want > to print error. When streams are deselected this would happen as well if > it interrupts the wait on buffer. Sure, but then we probably should avoid printing a WARN either, since that's expected behaviour. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1311#note_15908

1 0

[PATCH 0/14] MR912: Draft: Import OpenLDAP client libraries and use them in wldap32
by Hans Leidekker (＠hans) 15 Nov '22

15 Nov '22

The commits in libs/ don't build until the last one so they should be squashed. I kept them separate to show the changes I made to upstream source. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/912

2 15

Re: [PATCH v3 0/4] MR1311: wmvcore: Implement DedicatedDeliveryThread async reader output setting.
by Rémi Bernon (＠rbernon) 15 Nov '22

15 Nov '22

On Tue Nov 15 20:31:00 2022 +0000, Zebediah Figura wrote: > I guess, but it depends on the fact that "read_result" is > zero-initialized and never written, which seems non-obvious and a little > fragile. In fact, I think it can already break if a stream was selected > and then later becomes deselected, even if the filter was stopped in > between. Checking "stream->read_requested" would be more obvious but I > think still broken for the same reasons (but simple enough to fix.) > Note that if you do that then read_result becomes superfluous and > doesn't really need to be stored anywhere. `read_result` is written to E_PENDING whenever a read is requested. Every requested read will eventually be completed, writing the result to `read_result`, either `S_OK` or an error. If a stream was selected and a read requested to it, then later becomes deselected, we will consider it a valid thread wrt sample delivery until it completes its read and writes to `read_result`. It may complete it with a sample if the deselection happened late, or with an error if its read was interrupted. `read_requested` is a one way flag to tell reader to start reading. It needs to be reset before the `GetNextSample` call, and checked again on return to correctly synchronize flush and read requests from the callback thread, for instance when seeking. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1311#note_15905

1 0

Re: [PATCH v3 0/4] MR1311: wmvcore: Implement DedicatedDeliveryThread async reader output setting.
by Rémi Bernon (＠rbernon) 15 Nov '22

15 Nov '22

On Tue Nov 15 20:31:01 2022 +0000, Zebediah Figura wrote: > Linux cuts off thread names at 15 characters (not counting the null > terminator), which somewhat unfortunately makes all of these identical > to each other and the deliver threads. Thus far I think Brendan had been > trying to make the thread names distinct, if not per se trying to fit > the entire name within 15 characters. Unfortunate... -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1311#note_15906

1 0

Re: [PATCH v3 0/4] MR1311: wmvcore: Implement DedicatedDeliveryThread async reader output setting.
by Rémi Bernon (＠rbernon) 15 Nov '22

15 Nov '22

On Tue Nov 15 20:31:01 2022 +0000, Zebediah Figura wrote: > > We need the read threads to always be running and returning their next > sample because we then deliver them in PTS order. > Somewhat out of curiosity, did you check whether this still applies when > DedicatedDeliveryThread is set? I believe I did and it still applies (with the same offset as without threads, when samples are decoded), and there's another setting named `DeliverOnReceive` that is the one which changes that behavior (though I don't think we need it). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1311#note_15901

1 0

Re: [PATCH v3 0/4] MR1311: wmvcore: Implement DedicatedDeliveryThread async reader output setting.
by Rémi Bernon (＠rbernon) 15 Nov '22

15 Nov '22

On Tue Nov 15 20:30:58 2022 +0000, Zebediah Figura wrote: > Similarly to the earlier question about IWMSyncReader locking, do we > need to drop the CS here? Because this is a blocking call, which can call application callbacks, or otherwise also blocking until there's a sample ready. We want to be able to deliver samples from the callback thread, on other streams, without being blocked and without blocking the reader. There's no point in making all this asynchronous otherwise. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1311#note_15902

1 0

Re: [PATCH v3 0/4] MR1311: wmvcore: Implement DedicatedDeliveryThread async reader output setting.
by Rémi Bernon (＠rbernon) 15 Nov '22

15 Nov '22

On Tue Nov 15 20:30:59 2022 +0000, Zebediah Figura wrote: > Missed this the first time, but: I'd make this an ERR, personally. (An > application can trigger it, but it's one of those cases where it's a > sign that something went catastrophically wrong, and also we're pretty > much guessing how to handle it if not.) The most common case where this happens is on EOS, I don't think we want to print error. When streams are deselected this would happen as well if it interrupts the wait on buffer. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1311#note_15903

1 0

Re: [PATCH v3 0/4] MR1311: wmvcore: Implement DedicatedDeliveryThread async reader output setting.
by Rémi Bernon (＠rbernon) 15 Nov '22

15 Nov '22

On Tue Nov 15 20:30:59 2022 +0000, Zebediah Figura wrote: > What's the point of this? We never check the return code of > async_reader_get_next_sample(), only whether it FAILED(). No, it's checked outside of the loop to see if all streams have received EOS (in which case `hr == NS_E_NO_MORE_SAMPLES`), or if there's still some streams with pending requests or pending samples to deliver (`hr == E_PENDING`). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1311#note_15904

1 0

Re: [PATCH v2 0/1] MR1344: d3d11/tests: Introduce a clear_uav() helper. - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

Re: [PATCH v3 0/1] MR1370: d2d1: Silently ignore non-default state block implementations. - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

Re: [PATCH v2 0/1] MR1369: d3dx9: Add missing path conversion call in D3DXLoadVolumeFromFileA(). - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

Re: [PATCH v2 0/7] MR1372: Import OpenLDAP client libraries and use them in wldap32 - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

Re: [PATCH v2 0/1] MR1350: comctl32: Fix memory leak on error path in EDIT_MakeUndoFit. - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

Re: [PATCH v2 0/1] MR1366: winspool.drv: Ignore input in DocumentPropertiesA unless DM_IN_BUFFER is passed. - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

Re: [PATCH v2 0/1] MR1368: vbscript: Use CRT allocation functions. - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

Re: [PATCH v2 0/4] MR1361: kernel32: Validate globalmem pointers before calling heap functions. - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

Re: [PATCH v4 0/6] MR1298: winegstreamer: MPEG layer-3 decoder output media type enumeration and allocator properties. - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

Re: [PATCH v2 0/1] MR1367: ws2_32: Return success for setting SO_ERROR. - approved
by Alexandre Julliard (＠julliard) 15 Nov '22

15 Nov '22

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

1 0

[PATCH v2 0/7] MR1301: Add ir50_32 video codec for winmm.
by Shaun Ren (＠shaunren) 15 Nov '22

15 Nov '22

This video codec is used by games such as Richman 4, and Zwei: The Arges Adventure. The decoder logic is based on code from the FFmpeg project. -- v2: loader/wine.inf: Enable ir50_32 video codec. ir50_32: Implement decompression to 15-bit RGB. https://gitlab.winehq.org/wine/wine/-/merge_requests/1301

5 14

[PATCH 0/2] MR1295: Enable compilation with long types in d3dx9_36/tests.
by eric pouech (＠epo) 15 Nov '22

15 Nov '22

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

3 4

2025

2024

2023

2022

wine-gitlab