[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdelibs/kdeui/shortcuts
From:       Michael Jansen <kde () michael-jansen ! biz>
Date:       2008-09-17 17:57:52
Message-ID: 1221674272.043657.28873.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 861976 by mjansen:

Only complain if there is something wrong.

KShortcut("") is not needed but it is safe to assume the author meant
KShortcut().

 M  +3 -1      kshortcut.cpp  


--- trunk/KDE/kdelibs/kdeui/shortcuts/kshortcut.cpp #861975:861976
@@ -107,7 +107,9 @@
         QString k = sCuts.at(0);
         k.replace( "Win+", "Meta+" ); // workaround for KDE3-style shortcuts
         d->primary = QKeySequence::fromString(k);
-        if (d->primary.isEmpty()) {
+        // Complain about a unusable shortcuts sequence only if we have got
+        // something.
+        if (d->primary.isEmpty() and not k.isEmpty()) {
             kDebug(240) << "unusable primary shortcut sequence " << sCuts[0];
         }
     }
[prev in list] [next in list] [prev in thread] [next in thread]