rev 12410 - in branches/kde4-backports/packages/kdelibs/debian: . patches

[Ana Beatriz Guerrero López]

Author: ana
Date: 2008-10-13 12:52:25 +0000 (Mon, 13 Oct 2008)
New Revision: 12410

Added:
   branches/kde4-backports/packages/kdelibs/debian/patches/02_r866736_kdedglobalaccel_crashes.diff
Modified:
   branches/kde4-backports/packages/kdelibs/debian/changelog
   branches/kde4-backports/packages/kdelibs/debian/patches/series
Log:
patch-y


Modified: branches/kde4-backports/packages/kdelibs/debian/changelog
===================================================================
--- branches/kde4-backports/packages/kdelibs/debian/changelog	2008-10-13 01:42:49 UTC (rev 12409)
+++ branches/kde4-backports/packages/kdelibs/debian/changelog	2008-10-13 12:52:25 UTC (rev 12410)
@@ -1,3 +1,9 @@
+kde4libs (4:4.1.2-1~lenny+2) lenny; urgency=low
+
+  * Fix kdedglobalaccel crashes. Patch 02.
+
+ -- Ana Beatriz Guerrero Lopez <ana at debian.org>  Mon, 13 Oct 2008 09:26:54 +0200
+
 kde4libs (4:4.1.2-1~lenny+1) lenny; urgency=low
 
   * Backport for Lenny.

Added: branches/kde4-backports/packages/kdelibs/debian/patches/02_r866736_kdedglobalaccel_crashes.diff
===================================================================
--- branches/kde4-backports/packages/kdelibs/debian/patches/02_r866736_kdedglobalaccel_crashes.diff	                        (rev 0)
+++ branches/kde4-backports/packages/kdelibs/debian/patches/02_r866736_kdedglobalaccel_crashes.diff	2008-10-13 12:52:25 UTC (rev 12410)
@@ -0,0 +1,46 @@
+SVN commit 866736 by mjansen:
+
+Fix those kdedglobalaccel crashes. It's not safe to assume the config file
+has a correct content. Somehow it get's messed up sometimes and contains
+something like this:
+
+  next=Meta+B,Meta+B,
+  nextTrack=Meta+B,Meta+B,
+
+BUG:171870
+CCBUG:171206
+
+ M  +10 -2     kdedglobalaccel.cpp  
+
+
+Index: kde4libs-4.1.2/kdeui/shortcuts/kdedglobalaccel.cpp
+===================================================================
+--- kde4libs-4.1.2.orig/kdeui/shortcuts/kdedglobalaccel.cpp	2008-07-03 07:06:28.000000000 +0200
++++ kde4libs-4.1.2/kdeui/shortcuts/kdedglobalaccel.cpp	2008-10-12 20:36:21.000000000 +0200
+@@ -361,8 +361,9 @@
+ 
+ void KdedGlobalAccel::unRegister(const QStringList &actionId)
+ {
+-    Q_ASSERT(actionId.size()==4);
++    kDebug(125) << actionId;
+ 
++    Q_ASSERT(actionId.size()==4);
+     if (actionId.size() < 4) {
+         return;
+     }
+@@ -586,7 +587,14 @@
+     
+             foreach (int key, ad->keys) {
+                 if (key != 0) {
+-                    d->keyToAction.insert(key, ad);
++                    if (d->keyToAction.contains(key)) {
++                        // The shortcut is already used. The config file is
++                        // broken. Ignore the request.
++                        ad->keys.removeAll(key);
++                        kWarning() << "Shortcut found twice in kglobalshortcutsrc.";
++                    } else {
++                        d->keyToAction.insert(key, ad);
++                    }
+                 }
+             }
+         }

Modified: branches/kde4-backports/packages/kdelibs/debian/patches/series
===================================================================
--- branches/kde4-backports/packages/kdelibs/debian/patches/series	2008-10-13 01:42:49 UTC (rev 12409)
+++ branches/kde4-backports/packages/kdelibs/debian/patches/series	2008-10-13 12:52:25 UTC (rev 12410)
@@ -1,4 +1,5 @@
 01_kross_version_11_r838337.diff
+02_r866736_kdedglobalaccel_crashes.diff
 08_add_debian_build_type.diff
 09_disable_debug_messages_if_not_explicitly_enabled.diff
 11_kde4_applications_menu.diff