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 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
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 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
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