- wine-gitlab - winehq.org

[PATCH 0/1] MR6782: gdi32: Add support of monochrome bitmap to get_bitmap_info()
by AlexeyLushnikov (＠AlexeyLushnikov) 22 Nov '24

22 Nov '24

Emfdc.c. Function get_bitmap_info. Added filling of bmiColors for monochrome bitmaps when get_bitmap_info creates a new bitmap. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6782

3 4

[PATCH v19 0/2] MR6869: timeout: Windows 10 introduce the timeout command
by Michele Dionisio (＠michele.dionisio) 22 Nov '24

22 Nov '24

Windows 10 and 11 introduce the timeout.exe command. This is a similar program with same argument options -- v19: timeout: add minimal test suite timeout: Windows 10 introduce the timeout command https://gitlab.winehq.org/wine/wine/-/merge_requests/6869

4 5

[PATCH 0/1] MR6880: gitlab: Wait for the fvwm process to start.
by Rémi Bernon (＠rbernon) 22 Nov '24

22 Nov '24

I've seen some cases where the X server is a bit slow to start, and I suspect it could be a reason for some rare spurious test failures. Not 100% sure but at least this should be harmless and better than leaving it to chance. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6880

3 3

[PATCH 0/1] MR6884: Draft: Iphlpapi.dll: Included SendARP
by Thi Garlet (＠garlett) 22 Nov '24

22 Nov '24

Returns MAC address by: - search local interfaces, or - send UDP packed that does not need linux privilege, then search local cache solve https://bugs.winehq.org/show_bug.cgi?id=9418 files updated - /dlls/iphlpapi/iphlpapi_main.c - /dlls/iphlpapi/tests/iphlpapi.c -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6884

4 8

[PATCH v2 0/1] MR6883: msxml3: Add indentation test.
by Orin Varley (＠ovarley1) 22 Nov '24

22 Nov '24

Adding a test to make sure the indentation of xml elements is a tab even after moving to another thread. To prevent regressions like the bug bellow. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57325 -- v2: msxml3: Add indentation test. https://gitlab.winehq.org/wine/wine/-/merge_requests/6883

2 1

[PATCH v18 0/2] MR6869: timeout: Windows 10 introduce the timeout command
by Michele Dionisio (＠michele.dionisio) 22 Nov '24

22 Nov '24

Windows 10 and 11 introduce the timeout.exe command. This is a similar program with same argument options -- v18: timeout: add minimal test suite https://gitlab.winehq.org/wine/wine/-/merge_requests/6869

3 4

[PATCH v17 0/2] MR6869: timeout: Windows 10 introduce the timeout command
by Michele Dionisio (＠michele.dionisio) 22 Nov '24

22 Nov '24

Windows 10 and 11 introduce the timeout.exe command. This is a similar program with same argument options -- v17: timeout: add minimal test suite https://gitlab.winehq.org/wine/wine/-/merge_requests/6869

4 4

[PATCH 0/3] MR6855: server: Fix renaming file to a hardlink of itself.
by Gabriel Ivăncescu (＠insn) 22 Nov '24

22 Nov '24

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

3 13

Re: [PATCH v2 0/8] MR6887: mshtml: Implement proper prototype chain for text nodes and events.
by Gabriel Ivăncescu (＠insn) 22 Nov '24

22 Nov '24

Oh yeah I forgot about that. Quick test shows it doesn't. I guess I'll go with custom interface then and just forward. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88537

1 0

Re: [PATCH v2 0/8] MR6887: mshtml: Implement proper prototype chain for text nodes and events.
by Jacek Caban (＠jacek) 22 Nov '24

22 Nov '24

On Fri Nov 22 16:13:26 2024 +0000, Gabriel Ivăncescu wrote: > What about just using `IHTMLCommentElement2` for CharacterData's > interface, since it exposes the exact same props that it has? I mean > instead of the custom interface. Sounds better but should `QI(IHTMLCommentElement2)` succeeded on text nodes? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88536

1 0

Re: [PATCH v2 0/8] MR6887: mshtml: Implement proper prototype chain for text nodes and events.
by Gabriel Ivăncescu (＠insn) 22 Nov '24

22 Nov '24

On Fri Nov 22 16:09:59 2024 +0000, Jacek Caban wrote: > The ugly part of forwarding to `IHTMLDOMTextNode2` is that you need to > implement `IHTMLDOMTextNode2` in an object that's not a text node > (comment node). > With a private interface, we could give it a proper set of properties, > so no hooks would be needed, and just forward to it from both > `IHTMLDOMTextNode*` and `IHTMLCommentElement*` interfaces. We could even > share its entire implementation with something like: > ``` > struct HTMLCharacterData { > IHTMLCharacterData IHTMLCharacterData_iface; > DispatchEx *dispex; // used to forward IDispatch to comment's or > text node's dispex > nsIDOMCharacterData *character_data; // used for the actual implementation > }; > ``` > You could then embed it both in both comment and text node structs and > just use the single `IHTMLCharacterData` implementation. What about just using `IHTMLCommentElement2` for CharacterData's interface, since it exposes the exact same props that it has? I mean instead of the custom interface. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88535

1 0

Re: [PATCH v2 0/8] MR6887: mshtml: Implement proper prototype chain for text nodes and events.
by Jacek Caban (＠jacek) 22 Nov '24

22 Nov '24

On Fri Nov 22 15:47:04 2024 +0000, Gabriel Ivăncescu wrote: > Other than this, only CSS-related styles/stylesheets have a similar case > like this, but those are more convoluted. > I also don't see how introducing a private interface helps to simplify > it. We'd still have to expose it (and forward) in comment elements, but > now we'd also have to expose and forward it in text nodes, making it > actually less simple, IMO. > But is my forwarding a good idea? The ugly part of forwarding to `IHTMLDOMTextNode2` is that you need to implement `IHTMLDOMTextNode2` in an object that's not a text node (comment node). With a private interface, we could give it a proper set of properties, so no hooks would be needed, and just forward to it from both `IHTMLDOMTextNode*` and `IHTMLCommentElement*` interfaces. We could even share its entire implementation with something like: ``` struct HTMLCharacterData { IHTMLCharacterData IHTMLCharacterData_iface; DispatchEx *dispex; // used to forward IDispatch to comment's or text node's dispex nsIDOMCharacterData *character_data; // used for the actual implementation }; ``` You could then embed it both in both comment and text node structs and just use the single `IHTMLCharacterData` implementation. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88534

1 0

[PATCH v15 0/2] MR6869: timeout: Windows 10 introduce the timeout command
by Michele Dionisio (＠michele.dionisio) 22 Nov '24

22 Nov '24

Windows 10 and 11 introduce the timeout.exe command. This is a similar program with same argument options -- v15: timeout: add minimal test suite https://gitlab.winehq.org/wine/wine/-/merge_requests/6869

3 3

[PATCH 0/1] MR6883: msxml3: Add indentation test.
by Orin Varley (＠ovarley1) 22 Nov '24

22 Nov '24

Adding a test to make sure the indentation of xml elements is a tab even after moving to another thread. To prevent regressions like the bug bellow. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57325 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6883

3 5

Re: [PATCH v2 0/8] MR6887: mshtml: Implement proper prototype chain for text nodes and events.
by Gabriel Ivăncescu (＠insn) 22 Nov '24

22 Nov '24

On Fri Nov 22 15:44:33 2024 +0000, Jacek Caban wrote: > Yeah, it shows that those interfaces do not reflect prototypes. In a > sense, `IHTMLDOMTextNode2` is as bad for `CharacterData` as > `IHTMLCommentElement2`. Do we have a lot of cases like that? I'm > wondering if we need to change the mechanism or could we just introduce > a private `IHTMLCharacterData` interface. Other than this, only CSS-related styles/stylesheets have a similar case like this, but those are more convoluted. I also don't see how introducing a private interface helps to simplify it. We'd still have to expose it (and forward) in comment elements, but now we'd also have to expose and forward it in text nodes, making it actually less simple, IMO. But is my forwarding a good idea? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88530

1 0

Re: [PATCH v2 0/8] MR6887: mshtml: Implement proper prototype chain for text nodes and events.
by Jacek Caban (＠jacek) 22 Nov '24

22 Nov '24

On Fri Nov 22 15:40:33 2024 +0000, Gabriel Ivăncescu wrote: > On second thought maybe just forwarding to `IHTMLCommentElement2` would > be enough, but we still need the `IHTMLDOMTextNode2` so the prototype > works right? Yeah, it shows that those interfaces do not reflect prototypes. In a sense, `IHTMLDOMTextNode2` is as bad for `CharacterData` as `IHTMLCommentElement2`. Do we have a lot of cases like that? I'm wondering if we need to change the mechanism or could we just introduce a private `IHTMLCharacterData` interface. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88529

1 0

Re: [PATCH v2 0/8] MR6887: mshtml: Implement proper prototype chain for text nodes and events.
by Gabriel Ivăncescu (＠insn) 22 Nov '24

22 Nov '24

On Fri Nov 22 15:39:38 2024 +0000, Gabriel Ivăncescu wrote: > Interesting, but I'm not sure. The issue here is that these are in the > `CharacterDataPrototype`, not in the comment element itself. So, won't > they fail if we don't handle `IHTMLDOMTextNode2` when called on the > comment element? Since it's looked up by IID/DISPID pair, not name. > `IHTMLCommentElement2` might be used for legacy modes though, I guess I > will need to test it from C code to know. On second thought maybe just forwarding to `IHTMLCommentElement2` would be enough, but we still need the `IHTMLDOMTextNode2` so the prototype works right? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88528

1 0

[PATCH v3 0/3] MR6882: secur32, urlmon, wininet: Fix some failing tests.
by Rémi Bernon (＠rbernon) 22 Nov '24

22 Nov '24

Probably caused by some winehq.org HTTP server update, it now supports HTTP/2 and sends "Upgrade, Keep-Alive" connection strings (was "Keep-Alive" before). -- v3: wininet: Parse multi-token Connection strings for Keep-Alive. https://gitlab.winehq.org/wine/wine/-/merge_requests/6882

3 6

Re: [PATCH v2 0/8] MR6887: mshtml: Implement proper prototype chain for text nodes and events.
by Gabriel Ivăncescu (＠insn) 22 Nov '24

22 Nov '24

On Fri Nov 22 15:33:09 2024 +0000, Jacek Caban wrote: > Shouldn't it be `IHTMLCommentElement2` instead? Interesting, but I'm not sure. The issue here is that these are in the `CharacterDataPrototype`, not in the comment element itself. So, won't they fail if we don't handle `IHTMLDOMTextNode2` when called on the comment element? Since it's looked up by IID/DISPID pair, not name. `IHTMLCommentElement2` might be used for legacy modes though, I guess I will need to test it from C code to know. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88525

1 0

Re: [PATCH v2 0/8] MR6887: mshtml: Implement proper prototype chain for text nodes and events.
by Jacek Caban (＠jacek) 22 Nov '24

22 Nov '24

Jacek Caban (@jacek) commented about dlls/mshtml/htmlcomment.c: > struct HTMLCommentElement { > HTMLElement element; > IHTMLCommentElement IHTMLCommentElement_iface; > + IHTMLDOMTextNode IHTMLDOMTextNode_iface; > + IHTMLDOMTextNode2 IHTMLDOMTextNode2_iface; Shouldn't it be `IHTMLCommentElement2` instead? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6887#note_88523

1 0

[PATCH v14 0/2] MR6869: timeout: Windows 10 introduce the timeout command
by Michele Dionisio (＠michele.dionisio) 22 Nov '24

22 Nov '24

Windows 10 and 11 introduce the timeout.exe command. This is a similar program with same argument options -- v14: timeout: add minimal test suite https://gitlab.winehq.org/wine/wine/-/merge_requests/6869

3 3

[PATCH v16 0/2] MR6869: timeout: Windows 10 introduce the timeout command
by Michele Dionisio (＠michele.dionisio) 22 Nov '24

22 Nov '24

Windows 10 and 11 introduce the timeout.exe command. This is a similar program with same argument options -- v16: timeout: add minimal test suite https://gitlab.winehq.org/wine/wine/-/merge_requests/6869

2 2

[PATCH v13 0/2] MR6869: timeout: Windows 10 introduce the timeout command
by Michele Dionisio (＠michele.dionisio) 22 Nov '24

22 Nov '24

Windows 10 and 11 introduce the timeout.exe command. This is a similar program with same argument options -- v13: timeout: add minimal test suite https://gitlab.winehq.org/wine/wine/-/merge_requests/6869

3 3

[PATCH v2 0/3] MR6882: secur32, urlmon, wininet: Fix some failing tests.
by Rémi Bernon (＠rbernon) 22 Nov '24

22 Nov '24

Probably caused by some winehq.org HTTP server update, it now supports HTTP/2 and sends "Upgrade, Keep-Alive" connection strings (was "Keep-Alive" before). -- v2: wininet: Parse multi-token Connection strings for Keep-Alive. https://gitlab.winehq.org/wine/wine/-/merge_requests/6882

2 5

[PATCH v12 0/2] MR6869: timeout: Windows 10 introduce the timeout command
by Michele Dionisio (＠michele.dionisio) 22 Nov '24

22 Nov '24

Windows 10 and 11 introduce the timeout.exe command. This is a similar program with same argument options -- v12: timeout: add minimal test suite https://gitlab.winehq.org/wine/wine/-/merge_requests/6869

3 3

2025

2024

2023

2022

wine-gitlab