This partially reverts c709d625.
garcon-marshal.h is installed for some reason.

Although not documented, at least its values should be displayed, since
it is a property of GarconMenuNode.

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

Amends and completes the previous commit, by systematically using
`g_signal_connect_object()` whenever GarconMenu and GarconGtkMenu are
linked by the connection.

Related: #27, !18, 1ea47477

GarconMenu and GarconGtkMenu have separate life cycles. Therefore, there
is no assurance that the GarconMenu is finalized when the GarconGtkMenu
releases the reference to it upon its own finalization.

Xfdesktop, for example, keeps the same GarconMenu throughout its
lifecycle, while it creates a new GarconGtkMenu each time its context
menu is shown.

Fixes: #27
Related: !18

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

See https://mail.xfce.org/pipermail/xfce4-dev/2022-May/032915.html

This loading makes many disk accesses when reading the `.desktop` files,
which can take several seconds for only a hundred entries, on machines
equipped with mechanical disks and when these files have not yet been
cached.

This way, the loading is done as soon as possible in parallel, and we
wait until it is finished if necessary during a request to show the
menu. We can then start populating the GtkMenu synchronously, according
to the process described in the previous commit message.

Helps: #5, xfce4-panel#485
Fixes: #3, xfce4-panel#184
MR: !18

When the display of menu item icons is required, the creation of the
entire application menu, typically, can take several tenths of a second
or even more than a second on low performance machines. This change
reduces the latency of the menu display, spreading the load over the
display of its submenus and thus doing the rest of the work only if
necessary.

Some of the code in `garcon_gtk_menu_add()` needs to be duplicated to
test for displayable items in a submenu without entering a recursive
process. If a submenu has displayable items, it is added as a menu item,
but is not populated until later, if its display is required.

Unfortunately, for reasons internal to GTK, this process is not easily
parallelized in a GTask like loading the GarconMenu in the next commit.
This is why we have no choice but to delay the task by executing it only
when necessary.

Helps: #5, xfce4-panel#485
MR: !18

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

Gaël Bonithon authored 2 years ago and Alexander Schwinn committed 2 years ago

This partially reverts 1045e3f0, where
the static qualifier was removed without reason and without making the
rest of the function consistent with this change. The
GarconMenuSeparator is not intended to be packed in a container, so it
can effectively be created once and then only referenced.

This also fixes a memory leak reported by Valgrind on the weak reference
added on `separator`, which did not make sense anymore with a non-static
variable.

Gaël Bonithon authored 3 years ago and Alexander Schwinn committed 2 years ago

Triggers `-Wincompatible-pointer-types`, see documentation of
`g_once_init_enter()`.

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

In particular, fixes #24.

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).

Avinash Sonawane authored 3 years ago and Andre Miranda committed 3 years ago

Found by: scan-build

38 translated messages.

Transifex (https://www.transifex.com/xfce/public/).