| Summary: | PATCH/FIXME: remove global variable keytab_serial | ||
|---|---|---|---|
| Product: | [Applications] konsole | Reporter: | Kurt Hindenburg <khindenburg> |
| Component: | general | Assignee: | Konsole Bugs <konsole-bugs-null> |
| Status: | RESOLVED NOT A BUG | ||
| Severity: | wishlist | ||
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
A rather stupid bug for such a simple thing. |
Version: (using KDE Devel) Installed from: Compiled sources OS: Linux Any reason not to do this? =================================================================== RCS file: /home/kde/kdebase/konsole/konsole/keytrans.cpp,v retrieving revision 1.48 diff -u -p -r1.48 keytrans.cpp --- keytrans.cpp 22 Sep 2004 20:02:16 -0000 1.48 +++ keytrans.cpp 28 Sep 2004 18:17:07 -0000 @@ -636,8 +636,6 @@ KeyTransSymbols::KeyTransSymbols() // Global material ----------------------------------------------------------- -static int keytab_serial = 0; //FIXME: remove,localize - static QIntDict<KeyTrans> * numb2keymap = 0L; KeyTrans* KeyTrans::find(int numb) @@ -665,7 +663,7 @@ int KeyTrans::count() void KeyTrans::addKeyTrans() { - m_numb = keytab_serial ++; + m_numb = numb2keymap->count(); // or m_numb=count(); numb2keymap->insert(m_numb,this); }