July 2023 - Wine-gitlab - list.winehq.org

[PATCH v3 0/1] MR3247: d3d11/tests: Add basic test for shared resources.
by Paul Gofman (＠gofman) 08 Jul '23

08 Jul '23

-- v3: d3d11/tests: Add basic test for shared resources. https://gitlab.winehq.org/wine/wine/-/merge_requests/3247

3 3

[PATCH v2 0/2] MR3268: Move geolocation into correct dll
by Fabian Maurer (＠DarkShadow44) 07 Jul '23

07 Jul '23

-- v2: geolocation: Add tests for dll name geolocation: Rename from windows.devices.geolocation.geolocator https://gitlab.winehq.org/wine/wine/-/merge_requests/3268

3 3

[PATCH 0/3] MR3268: Move geolocation into correct dll
by Fabian Maurer (＠DarkShadow44) 07 Jul '23

07 Jul '23

Split into 3 commits for easier reviewing: 1) Just move the files 2) Fix paths 3) Add test Feel free to squish. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3268

2 3

[PATCH v4 0/1] MR2228: bcryptprimitives: ProcessRng stub
by Christopher Denton (＠cdenton) 07 Jul '23

07 Jul '23

`ProcessRng` is the only publicly documented function exported by `bcryptprimitives`. This stub simply forwards it to `RtlGenRandom` in `advapi32`. Documentation: https://learn.microsoft.com/en-us/windows/win32/seccng/processprng -- v4: bcryptprimitives: ProcessRng stub https://gitlab.winehq.org/wine/wine/-/merge_requests/2228

2 1

[PATCH v3 0/1] MR2228: bcryptprimitives: ProcessRng stub
by Christopher Denton (＠cdenton) 07 Jul '23

07 Jul '23

`ProcessRng` is the only publicly documented function exported by `bcryptprimitives`. This stub simply forwards it to `RtlGenRandom` in `advapi32`. Documentation: https://learn.microsoft.com/en-us/windows/win32/seccng/processprng -- v3: bcryptprimitives: ProcessRng stub https://gitlab.winehq.org/wine/wine/-/merge_requests/2228

2 3

Re: [PATCH v5 0/8] MR3249: windows.networking.hostname: Implement IHostNameFactory::CreateHostName().
by Nikolay Sivov (＠nsivov) 07 Jul '23

07 Jul '23

Nikolay Sivov (@nsivov) commented about dlls/windows.networking.hostname/hostname.c: > + struct hostname *impl; > + > + TRACE( "iface %p, name %s, value %p\n", iface, debugstr_hstring(name), value ); > + > + if (!value) return E_POINTER; > + if (!name) return E_INVALIDARG; > + if (!(impl = calloc( 1, sizeof(*impl) ))) return E_OUTOFMEMORY; > + > + impl->IHostName_iface.lpVtbl = &hostname_vtbl; > + impl->ref = 1; > + impl->rawname = name; > + > + *value = &impl->IHostName_iface; > + TRACE( "created IHostName %p.\n", *value ); > + return S_OK; > } Shouldn't this duplicate the string? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3249#note_38364

1 0

Re: [PATCH v2 0/14] MR3266: mshtml: dom tests leak fixes. - approved
by Alexandre Julliard (＠julliard) 07 Jul '23

07 Jul '23

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

1 0

[PATCH v2 0/6] MR3238: wined3d: Move Vulkan declarations to wined3d_vk.h, part 1/3.
by Zebediah Figura (＠zfigura) 07 Jul '23

07 Jul '23

This is a part of a larger effort to reorganize wined3d a bit so that backend functions are fully quarantined into their own files. There are a few advantages to this: * We avoid including (large) headers in files that don't need them. * This has helped me find functions that are currently incorrectly tied to backends [e.g. wined3d_texture_update_desc(), wined3d_texture_set_lod()]. * This may also help find code (struct members, helpers, etc.) which should be made local to a backend. * I find that this aids the reader in logically separating wined3d code. For some time the wined3d code base has resisted modularity of compilation units, simply on the grounds that this is not necessary. I find files comprising several thousand lines to be a bit unwieldy, however, and that comprehension and navigation are eased when drawing physical lines even when such separation was already obvious. It also can be nice for compilation speed. There are a few steps to this: - move GL and Vulkan declarations to wined3d_gl.h and wined3d_vk.h respectively - introduce "resource_gl.c" and "resource_vk.c", and move most resource code to those functions. This collapses the separation between buffer, texture, sampler, and view code, though, and there may be some benefit in keeping those separate. It's worth mentioning though that the bulk of those files ends up being the texture code. It's also worth mentioning that resources are generally interrelated—e.g. views delegate to buffers or textures—and we've talked about generalizing more of the buffer/texture code to share more code. Of course, *everything* is interrelated, so the lines that we're drawing now are kind of arbitrary. I at least find that GL/Vulkan are *useful* lines to draw. - add format_gl.c, because there are about 3000 lines of code to deal with GL format detection. - add ffp_gl.c, which contains the fixed-function vertex and fragment pipelines. This is about 3500 lines. - move other functions to backend-specific adapter or context functions, where it makes sense. Swapchain code mostly gets moved to context_*.c, though I'm not sure whether that makes the most sense or not. -- v2: wined3d: Move struct wined3d_adapter_vk to wined3d_vk.h. wined3d: Move struct wined3d_context_vk to wined3d_vk.h. wined3d: Move struct wined3d_query_vk to wined3d_vk.h. wined3d: Move struct wined3d_image_vk to wined3d_vk.h. wined3d: Move struct wined3d_bo_vk to wined3d_vk.h. wined3d: Move some definitions to wined3d_vk.h. https://gitlab.winehq.org/wine/wine/-/merge_requests/3238

3 7

Re: [PATCH v2 0/1] MR3265: gdi32: Fix DIB_PAL_INDICES usage handling in EMR_CREATEMONOBRUSH playback. - approved
by Alexandre Julliard (＠julliard) 07 Jul '23

07 Jul '23

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

1 0

Re: [PATCH v2 0/5] MR3257: mshtml: TextRange ref leak fixes. - approved
by Alexandre Julliard (＠julliard) 07 Jul '23

07 Jul '23

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

1 0