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

[PATCH 0/1] MR2852: winemac: Add missing event to dbgstr_event().
by Brendan Shanks (＠bshanks) 18 May '23

18 May '23

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

3 2

Re: [PATCH v4 0/1] MR203: vkd3d-shader/hlsl: Fold constant integral abs().
by Ethan Lee (＠flibitijibibo) 18 May '23

18 May '23

On Thu May 18 19:54:01 2023 +0000, Francisco Casas wrote: > This won't get the absolute value of components that come after the > component that is INT_MIN. > Consider an int vector such as `(-1, INT_MIN, -1, -1)`; This would end > up as `(1, INT_MIN, -1, -1)`. > I think it would be better to guard the assignment: > ```c > case HLSL_TYPE_INT: > /* abs(INT_MIN) is undefined, leave this expression > alone */ > if (src->value.u[k].i != INT_MIN) > dst->value.u[k].i = abs(src->value.u[k].i); > break; > ``` I might have misread, but doesn't the return value apply to the full vector and not its components? I thought returning false made it so the whole abs op was left alone if any component was INT_MIN, like an all-or-nothing deal. -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/203#note_33136

1 0

[PATCH v5 0/1] MR2346: winsqlite3: Add stub DLL.
by Tareque Md Hanif (＠tmdh) 18 May '23

18 May '23

Required to run winget package manager. -- v5: winsqlite3: Add stub DLL. https://gitlab.winehq.org/wine/wine/-/merge_requests/2346

3 4

Re: [PATCH v4 0/1] MR203: vkd3d-shader/hlsl: Fold constant integral abs().
by Francisco Casas (＠fcasas) 18 May '23

18 May '23

Francisco Casas (@fcasas) commented about libs/vkd3d-shader/hlsl_constant_ops.c: > + { > + switch (type) > + { > + case HLSL_TYPE_FLOAT: > + case HLSL_TYPE_HALF: > + dst->value.u[k].f = fabsf(src->value.u[k].f); > + break; > + > + case HLSL_TYPE_DOUBLE: > + dst->value.u[k].d = fabs(src->value.u[k].d); > + break; > + > + case HLSL_TYPE_INT: > + /* abs(INT_MIN) is undefined, leave this expression alone */ > + if (src->value.u[k].i == INT_MIN) > + return false; This won't get the absolute value of components that come after the component that is INT_MIN. Consider an int vector such as `(-1, INT_MIN, -1, -1)`; This would end up as `(1, INT_MIN, -1, -1)`. I think it would be better to guard the assignment: ```c case HLSL_TYPE_INT: /* abs(INT_MIN) is undefined, leave this expression alone */ if (src->value.u[k].i != INT_MIN) dst->value.u[k].i = abs(src->value.u[k].i); break; ``` -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/203#note_33133

1 0

[PATCH v10 0/4] MR2808: Draft: SNI tray implementation
by Cherser-s (＠Cherser-s) 18 May '23

18 May '23

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. -- v10: test host_os echoing https://gitlab.winehq.org/wine/wine/-/merge_requests/2808

3 5

[PATCH 0/1] MR2851: localspl: Fix wow64_start_doc Wow64 entry point.
by Piotr Caban (＠piotr) 18 May '23

18 May '23

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

4 3

[PATCH 0/1] MR2850: gdiplus: Fix GdipGetGenericFontFamily functions according to native gdiplus.dll
by Bartosz Kosiorek (＠gang65) 18 May '23

18 May '23

I have investigated `GdipGetGenericFontFamily*` functions from native gdiplus.dll. The result of investigation * GdipGetGenericFontFamilySansSerif: is taking first `"Microsoft Sans Serif"` font, then `Arial` [(here is the comparison)](http://www.identifont.com/differences?first=Microsoft+Sans+Ser…) The replacement for Arial is `"Liberation Sans"`. [(here is the comparison)](http://www.identifont.com/differences?first=Liberation+Sans&se…) If these fonts are not found then [`Tahoma` is taken.](http://www.identifont.com/differences?first=Liberation+Sans&second=… * GdipGetGenericFontFamilySerif: If `"Times New Roman"` is not found, then the fonts from `GdipGetGenericFontFamilySansSerif` is taken. * GdipGetGenericFontFamilyMonospace: If `"Courier New"` is not found, then the fonts from `GdipGetGenericFontFamilySansSerif` is taken. More information about Free Fonts: https://en.wikipedia.org/wiki/Liberation_fonts -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2850

3 2

[PATCH v9 0/3] MR2808: Draft: SNI tray implementation
by Cherser-s (＠Cherser-s) 18 May '23

18 May '23

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. -- v9: configure: fix winesni.drv not getting disabled https://gitlab.winehq.org/wine/wine/-/merge_requests/2808

3 4

Re: [PATCH v8 0/6] MR2786: Add support for AF_UNIX sockets
by Zebediah Figura (＠zfigura) 18 May '23

18 May '23

Here are a few more high-level comments I have: * I'd like to see more tests for how paths are translated, in particular I'd like to see tests for a DOS-style absolute path name (beginning with a drive letter) and an NT-style absolute path name [i.e. whatever RtlDosPathNameToNtPathName() returns for the same path], and a path containing a single or double dot. I'd also like to see the result of getsockname() on all tested paths. * I'd also like to see tests for sendto/recvfrom, if for no other reason than to make sure that the address translation is being done there. * What protocol do these sockets actually have? I.e. what does SO_PROTOCOL_INFO return for them? 0 is usually a wildcard. * I'm willing to assume that most socket functions probably work the same way (I honestly suspect that's not actually the case, but it's better just to leave that alone for now). That said, tests for any functions that your program uses and depends on would be good, lest we find out later that they're supposed to be implemented differently and break things. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2786#note_33118

1 0

Re: [PATCH v8 0/6] MR2786: Add support for AF_UNIX sockets
by Zebediah Figura (＠zfigura) 18 May '23

18 May '23

Zebediah Figura (@zfigura) commented about server/fd.c: > } > > + /* open(3) throws ENXIO when `path` is one of: > + * 1. a FIFO, and open(3) is called with O_WRONLY | O_NONBLOCK, > + * and no other process is currently attempting to read from `path`. > + * 2. a special device file, and the device it corresponds to does not exist. > + * 3. a UNIX socket. > + * We can confirm the third case to allow for unlinking a socket. > + */ > + if (errno == ENXIO && !stat( name, &st ) && S_ISSOCK(st.st_mode)) > + { > + if ((access & DELETE)) > + unlink( name ); > + else > + { > + set_error( ENXIO ); That's... not really correct, ENXIO isn't an NTSTATUS. Better just to let this hit the same file_set_error() path. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2786#note_33116

1 0