"Unfortunately, last time I tried to write tests for Unix sockets, I started triggering kernel panics in Windows."
It seems like a Windows specific issue! A kernel panic should never occur, even when provided with invalid data.
@dropbear I think you could use the code and turn it into a test for theses patches?
I think it's best to ignore Windows <-> Unix communications tests for now as I don't think `winetest` allow running tests in both Windows and Unix space at the same time, that …
[View More]would require `winetest` supporting `wsl` when testing on Windows10+, and I think `wsl` support is out of scope of this PR.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2786#note_32726
[View Less]
On Sun May 14 12:03:33 2023 +0000, Loïc Rebmeister wrote:
> Nevermind I saw `org.freedesktop.StatusNotifierItem` used in the rest of
> the code and assumed it was already supported, but I checked commit
> names and saw it was intentional.
> I saw the rest of the code reference `org.freedesktop.StatusNotifierItem`
> A wine standard would be to remove the
> `org.freedesktop.StatusNotifierItem` from the KDE SNI Implementation
> commit, and then add the references back on the …
[View More]commit that add the
> `org.freedesktop.StatusNotifierItem` based implementation.
Maybe just remove freedesktop interface altogether?
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2808#note_32724
[View Less]
On Sun May 14 11:59:45 2023 +0000, Loïc Rebmeister wrote:
> `watcher_interface_name` is `org.kde.StatusNotifierItem` and this code
> is only checking that.
> Is this intentional that `org.freedesktop.StatusNotifierItem` is not checked?
Nevermind I saw `org.freedesktop.StatusNotifierItem` used in the rest of the code and assumed it was already supported, but I checked commit names and saw it was intentional.
I saw the rest of the code reference `org.freedesktop.StatusNotifierItem`
A …
[View More]wine standard would be to remove the `org.freedesktop.StatusNotifierItem` from the KDE SNI Implementation commit, and then add the references back on the commit that add the `org.freedesktop.StatusNotifierItem` based implementation.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2808#note_32723
[View Less]
virtual_free_teb called on thread exit can save memory. pthread_create run start_thread_wrapper on a small size stack, start_thread_wrapper calls start_thread and switch on a big size thread stack. when thread exit, switch stack pointer return to the small size stack and free memory of the thread.
Signed-off-by: Fan WenJie <fanwj(a)mail.ustc.edu.cn>
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2814
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.
--
v3: explorer: add winesni.drv tray implementation support
winesni.drv: add KDE StatusNotifierItem implementation for tray
https://gitlab.winehq.org/wine/wine/-/merge_requests/2808
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.
--
v2: explorer: add winesni.drv tray implementation support
winesni.drv: add KDE StatusNotifierItem implementation for tray
https://gitlab.winehq.org/wine/wine/-/merge_requests/2808
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.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2808