Re: [PATCH v41 0/8] MR2808: SNI tray implementation

[Chih-Hsuan Yen (＠yan12125)]

Chih-Hsuan Yen (@yan12125) commented about dlls/win32u/snidrv/dbus.c:
> -
>    p_dbus_message_iter_close_container(&sIter, &bIter);
> - }
> - else
> -
>    goto fail;
> -
> - p_dbus_message_iter_close_container(&aIter, &sIter);
> - p_dbus_message_iter_close_container(iter, &aIter);
> - return TRUE;
>
> +fail:
> - return FALSE;
>
> +}
> -
>
> +static void handle_title(DBusConnection* conn, DBusMessageIter *iter, const struct tray_icon* icon)
> +{
> - const char* tiptext = (const char*)icon->tiptext;
> - p_dbus_message_iter_append_basic(iter, 's', &tiptext);

Is it a good idea to assign `tiptext` to SNI `Title` property? SNI `ToolTip` property sounds a better target.

Per [1], `Title` is "a name that describes the application", while `tiptext` may be something random.

[1] https://www.freedesktop.org/wiki/Specifications/StatusNotifierItem/StatusNot... (yeah I know this MR is for `org.kde.StatusNotifierItem` instead of `org.freedesktop.StatusNotifierItem`, but I cannot find an authoritative description for the former)