[gnome-desktop/gnome-40] xkbinfo: continue after skipping a duplicate layout, don't return
- From: Michael Catanzaro <mcatanzaro src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-desktop/gnome-40] xkbinfo: continue after skipping a duplicate layout, don't return
- Date: Thu, 22 Apr 2021 15:20:52 +0000 (UTC)
commit 81c6cd79529371c7b7810316abbdfcb61470a04c
Author: Peter Hutterer <peter hutterer who-t net>
Date: Thu Apr 22 03:27:43 2021 +0000
xkbinfo: continue after skipping a duplicate layout, don't return
Copy-paste error introduced in aa7e3440511. We're inside the loop listing
through all layouts here so returning early means we skip any layout after the
duplicate one.
Before:
$ test-xkb-info | wc -l
3913
Now:
$ test-xkb-info | wc -l
3923
Fixes aa7e344051170ea47585d3d72b1a36e3991121f9
Found by Jason Francis (@jf)
(cherry picked from commit bab269813e8d5254abd9f07b613d3646230be8c3)
libgnome-desktop/gnome-xkb-info.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
---
diff --git a/libgnome-desktop/gnome-xkb-info.c b/libgnome-desktop/gnome-xkb-info.c
index f2a3214b..2bb3036c 100644
--- a/libgnome-desktop/gnome-xkb-info.c
+++ b/libgnome-desktop/gnome-xkb-info.c
@@ -271,7 +271,7 @@ add_layouts (GnomeXkbInfo *self,
if (g_hash_table_contains (priv->layouts_table, l->id))
{
g_clear_pointer (&l, free_layout);
- return;
+ continue;
}
g_hash_table_replace (priv->layouts_table, l->id, l);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]