[gnome-desktop/gnome-40] xkbinfo: continue after skipping a duplicate layout, don't return



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]