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

List:       kde-devel
Subject:    Re: bugfix kdelibs (kglobalaccel.h)
From:       Andreas Simon <yuipx () gmx ! net>
Date:       2002-02-26 22:33:31
[Download RAW message or body]

On Wed, 2002-02-27 at 06:27, Ellis Whitehead wrote:
> On Monday 25 February 2002 17:14, Andreas Simon wrote:

> Thanks for the patch.  Passing strings for the shortcut is poor practice 
> though, because key conversion is then dozens of times slower (or hundreds?).  
> Also because they aren't compile-time checked.  If you pass "Altt+D", for 
> example, you'll have a run-time bug which would have been avoided by using 
> the Qt enums.

Thanks for pointing this out. I picked the keyz module from noatun as
example code. I guess that was no good idea ;-) I sent a patch to the
respective maintainer to replace the strings there.

Cheers,
Andreas Simon

 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]