June 2023 - Wine-GitLab - list.winehq.org

Re: [PATCH v25 0/7] MR2786: Add support for AF_UNIX sockets
by Ally Sommers (＠dropbear) June 18, 2023

June 18, 2023

On Fri Jun 16 17:31:21 2023 +0000, Zebediah Figura wrote: > I don't think the {} construction is portable. We have offsetof() for > this anyway. It's portable for C99 if I hadn't forgotten to include the `0` (and C23 as-is), but Wine does use C89. I'll replace it with an `offsetof()`. It's worth noting that this entire initializer is only portable for C99 due to the use of designators. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2786#note_36048

1 0

Re: [PATCH v25 0/7] MR2786: Add support for AF_UNIX sockets
by Ally Sommers (＠dropbear) June 18, 2023

June 18, 2023

On Fri Jun 16 17:31:27 2023 +0000, Zebediah Figura wrote: > Similarly this isn't going to work without converting from Unix to win32 paths. In addition to the size mismatch issue, if the conversion must be lossless, it's not practical to provide a Windows path for a Unix one, since relativity, UNC prefixes, etc. would probably change. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2786#note_36047

1 0

Re: [PATCH v25 0/7] MR2786: Add support for AF_UNIX sockets
by Ally Sommers (＠dropbear) June 18, 2023

June 18, 2023

On Fri Jun 16 17:31:20 2023 +0000, Zebediah Figura wrote: > This can't work, though; we need to convert it to Unix first. (Or print > a FIXME and fail for DGRAM sockets.) > Linux man-pages call out returning sizeof(sockaddr_un) as okay on Linux, > but is it portable? We may want to return strlen()+1 just to be safe. Converting the path cannot realistically be done due to size differences for the same path: ``` $ echo -n 'C:\tmp\foo.sock' | wc -c 15 $ winepath 'C:\tmp\foo.sock' | tr -d '\n' | wc -c 47 ``` For the same reason as needing to change directories to actually bind and connect, path translation in these functions and their server equivalents is not feasible. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2786#note_36046

1 0

Re: [PATCH v25 0/7] MR2786: Add support for AF_UNIX sockets
by Ally Sommers (＠dropbear) June 18, 2023

June 18, 2023

On Fri Jun 16 17:31:23 2023 +0000, Zebediah Figura wrote: > We shouldn't need this anymore now that we're pssing the real unix path, right? This is still needed because we need to change directories to the Unix path. We can't use the provided Unix path directly because the Unix path length can be larger than the Windows path due to the wineprefix, and that length may exceed `sizeof(sun_path)` even if the Windows path length does not. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2786#note_36045

1 0

[PATCH v2 0/1] MR3089: hhctrl: Initialize item->items to NULL in parse_index_sitemap_object (scan-build).
by Alex Henrie (＠alexhenrie) June 18, 2023

June 18, 2023

Initializing to NULL makes it more clear that the initial value is only used as a parameter to realloc. -- v2: hhctrl: Initialize item->items to NULL in parse_index_sitemap_object (scan-build). https://gitlab.winehq.org/wine/wine/-/merge_requests/3089

3 2

[PATCH 0/1] MR3089: hhctrl: Simplify allocation size in parse_index_sitemap_object (scan-build).
by Alex Henrie (＠alexhenrie) June 18, 2023

June 18, 2023

sizeof(IndexSubItem) is more clear than 2 * sizeof(void *) and it avoids any questions about struct padding. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3089

3 5

[PATCH v18 0/5] MR2808: Draft: SNI tray implementation
by Cherser-s (＠Cherser-s) June 17, 2023

June 17, 2023

Adds the tray icons implementation based on org.kde.StatusNotifierItem interface usage. Does allow restarting StatusNotifierWatcher object, but will fallback to XEMBED or internal tray, if wine gets initialized when there is no StatusNotifierWatcher object registered. -- v18: winesni.drv: replaced the dbus connection logic with a single connection per each SNI object winesni.drv: wrap functions with pthread mutex locking winesni.drv: implement basic balloon notification support explorer: add winesni.drv tray implementation support winesni.drv: add KDE StatusNotifierItem implementation for tray https://gitlab.winehq.org/wine/wine/-/merge_requests/2808

2 5

[PATCH 0/1] MR3083: comctl32: Remove duplicate declarations of MONTHCAL functions.
by Alex Henrie (＠alexhenrie) June 17, 2023

June 17, 2023

MONTHCAL_MonthLength and MONTHCAL_CalculateDayOfWeek are already declared in dlls/comctl32/comctl32.h. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3083

4 3

[PATCH v5 0/3] MR3075: winegstreamer: Implement Wow64 entrypoints in the Unix library.
by Kurt Kartaltepe (＠kkartaltepe) June 17, 2023

June 17, 2023

This also converts the opaque pointers passed across the PE->Unix interface to opaque types represented as UINT64 which store the unix side pointer. This makes the thunking simpler and should avoid anyone accidently dereferencing the unix pointer in the PE code. This was tested with warcraft 3 (1.26, the last non-reforged release), and is consistent with the standard 32bit build. But since movies on warcraft 3 are broken in standard 32bit builds I could only test that that the gstreamer graph gets built up similarly and not verify the video playback. Id appreciate if someone knew of a working exercise of the gstreamer code. -- v5: Fixup: assert struct sizes with bool/enum Fixup: designated initializers winegstreamer: Implement Wow64 entrypoints in the Unix library. https://gitlab.winehq.org/wine/wine/-/merge_requests/3075

2 4

[PATCH v4 0/1] MR3075: winegstreamer: Implement Wow64 entrypoints in the Unix library.
by Kurt Kartaltepe (＠kkartaltepe) June 17, 2023

June 17, 2023

This also converts the opaque pointers passed across the PE->Unix interface to opaque types represented as UINT64 which store the unix side pointer. This makes the thunking simpler and should avoid anyone accidently dereferencing the unix pointer in the PE code. This was tested with warcraft 3 (1.24, the last non-reforged release), and is consistent with the standard 32bit build. But since movies on warcraft 3 are broken in standard 32bit builds I could only test that that the gstreamer graph gets built up similarly and not verify the video playback. Id appreciate if someone knew of a working exercise of the gstreamer code. -- v4: winegstreamer: Implement Wow64 entrypoints in the Unix library. https://gitlab.winehq.org/wine/wine/-/merge_requests/3075

4 6