switched keyboard filter to the new singleton class OKeyFilter from within

opiecore2 so we can better handle it.

3 files changed, 140 insertions, 130 deletions

diff --git a/noncore/applets/zkbapplet/zkbwidget.cpp b/noncore/applets/zkbapplet/zkbwidget.cpp
index 38bfba9..8499500 100644
--- a/noncore/applets/zkbapplet/zkbwidget.cpp
+++ b/noncore/applets/zkbapplet/zkbwidget.cpp
@@ -1,161 +1,168 @@
 #include <opie2/otaskbarapplet.h>
+#include <opie2/okeyfilter.h>
 #include <qpe/qcopenvelope_qws.h>
 #include <qpe/applnk.h>
 #include <qpe/qpeapplication.h>
 #include <qpe/resource.h>
 #include <stdio.h>
 #include <unistd.h>
 #include "zkbwidget.h"
 #include "zkbcfg.h"
 
 using namespace Opie::Ui;
-ZkbWidget::ZkbWidget(QWidget* parent):QLabel(parent),keymap(0),
-	disabled(Resource::loadPixmap("zkb-disabled")) {
 
-	labels = new QPopupMenu();
-	connect(labels, SIGNAL(activated(int)), this,
-		SLOT(labelChanged(int)));
+ZkbWidget::ZkbWidget(QWidget* parent)
+    :QLabel(parent),keymap(0),disabled(Resource::loadPixmap("zkb-disabled")) {
 
-	loadKeymap();
+    labels = new QPopupMenu();
+    connect(labels, SIGNAL(activated(int)), this,
+        SLOT(labelChanged(int)));
 
-	channel = new QCopChannel("QPE/zkb", this);
-	connect(channel, SIGNAL(received(const QCString&,const QByteArray&)),
-		this, SLOT(signalReceived(const QCString&,const QByteArray&)));
+    loadKeymap();
+
+    channel = new QCopChannel("QPE/zkb", this);
+    connect(channel, SIGNAL(received(const QCString&,const QByteArray&)),
+        this, SLOT(signalReceived(const QCString&,const QByteArray&)));
     setFixedWidth ( AppLnk::smallIconSize()  );
     setFixedHeight ( AppLnk::smallIconSize()  );
 }
 
-ZkbWidget::~ZkbWidget() {
+ZkbWidget::~ZkbWidget()
+{
+    if (keymap != 0) {
+        delete keymap;
+        keymap = 0;
+    }
 }
 
 int ZkbWidget::position()
 {
-	return 8;
+    return 8;
 }
 
 bool ZkbWidget::loadKeymap() {
-	ZkbConfig c(QPEApplication::qpeDir()+"/share/zkb");
-	QFontMetrics fm(font());
-
-	if (keymap != 0) {
-		delete keymap;
-		keymap = 0;
-	}
-
-	Keymap* km = new Keymap();
-
-	if (!c.load("zkb.xml", *km, "")) {
-		delete km;
-		setPixmap(disabled);
-		return false;
-	}
-
-	connect(km, SIGNAL(stateChanged(const QString&)),
-		this, SLOT(stateChanged(const QString&)));
-
-	qwsServer->setKeyboardFilter(km);
-
-	Keymap* oldkm = keymap;
-	keymap = km;
-
-	if (oldkm != 0) {
-		delete oldkm;
-	}
-
-	setText(keymap->getCurrentLabel());
-
-	labels->clear();
-	QStringList l = keymap->listLabels();
-	labels->insertItem(disabled, 0, 0);
-	int n = 1;
-	w = 0;
-	for(QStringList::Iterator it = l.begin(); it != l.end();
-		++it, n++) {
-
-//		printf("label: %s\n", (const char*) (*it).utf8());
-
-		labels->insertItem(*it, n, n);
-		int lw = fm.width(*it);
-		if (lw > w) {
-			w = lw;
-		}
-	}
-
-	if (w == 0) {
-		hide();
-	} else {
-		show();
-	}
-	return true;
+    ZkbConfig c(QPEApplication::qpeDir()+"/share/zkb");
+    QFontMetrics fm(font());
+
+    if (keymap != 0) {
+        delete keymap;
+        keymap = 0;
+    }
+
+    Keymap* km = new Keymap();
+
+    if (!c.load("zkb.xml", *km, "")) {
+        delete km;
+        setPixmap(disabled);
+        return false;
+    }
+
+    connect(km, SIGNAL(stateChanged(const QString&)),
+        this, SLOT(stateChanged(const QString&)));
+
+    Opie::Core::OKeyFilter::inst()->addHandler(km);
+
+    Keymap* oldkm = keymap;
+    keymap = km;
+
+    if (oldkm != 0) {
+        delete oldkm;
+    }
+
+    QString ltext = keymap->getCurrentLabel();
+    if (ltext.length()==0) ltext = "??";
+    setText(ltext);
+
+    labels->clear();
+    QStringList l = keymap->listLabels();
+    labels->insertItem(disabled, 0, 0);
+    int n = 1;
+    w = 0;
+    for(QStringList::Iterator it = l.begin(); it != l.end();
+        ++it, n++) {
+
+//      printf("label: %s\n", (const char*) (*it).utf8());
+
+        labels->insertItem(*it, n, n);
+        int lw = fm.width(*it);
+        if (lw > w) {
+            w = lw;
+        }
+    }
+
+    if (w == 0) {
+        hide();
+    } else {
+        show();
+    }
+    return true;
 }
 
 QSize ZkbWidget::sizeHint() const {
     return QSize(AppLnk::smallIconSize(),AppLnk::smallIconSize());
 }
 
 void ZkbWidget::stateChanged(const QString& s) {
-//	odebug << "stateChanged: " << s.utf8() << "\n" << oendl; 
-	setText(s);
+//  odebug << "stateChanged: " << s.utf8() << "\n" << oendl;
+    setText(s);
 }
 
 void ZkbWidget::labelChanged(int id) {
-	if (id == 0) {
-		keymap->disable();
-		setPixmap(disabled);
-		return;
-	}
-
-	keymap->enable();
-
-	QStringList l = keymap->listLabels();
-	QString lbl = l[id-1];
-
-//	printf("labelChanged: %s\n", (const char*) lbl.utf8());
-	State* state = keymap->getStateByLabel(lbl);
-	if (state != 0) {
-		keymap->setCurrentState(state);
-		setText(lbl);
-	}
+    if (id == 0) {
+        keymap->disable();
+        setPixmap(disabled);
+        return;
+    }
+
+    keymap->enable();
+
+    QStringList l = keymap->listLabels();
+    QString lbl = l[id-1];
+
+//  printf("labelChanged: %s\n", (const char*) lbl.utf8());
+    State* state = keymap->getStateByLabel(lbl);
+    if (state != 0) {
+        keymap->setCurrentState(state);
+        setText(lbl);
+    }
 }
 
 void ZkbWidget::mouseReleaseEvent(QMouseEvent*) {
-	QSize sh = labels->sizeHint();
-	QPoint p = mapToGlobal(QPoint((width()-sh.width())/2,-sh.height()));
-	labels->exec(p);
+    QSize sh = labels->sizeHint();
+    QPoint p = mapToGlobal(QPoint((width()-sh.width())/2,-sh.height()));
+    labels->exec(p);
 }
 
 void ZkbWidget::signalReceived(const QCString& msg, const QByteArray& data) {
-	QDataStream stream(data, IO_ReadOnly);
-
-	if (msg == "enable()") {
-		keymap->enable();
-	} else if (msg == "disable()") {
-		keymap->disable();
-	} else if (msg == "reload()") {
-		QCopEnvelope("QPE/System", "busy()");
-		QTimer::singleShot(0, this, SLOT(reload()));
-	} else if (msg == "switch(QString)") {
-		QString lbl;
-		stream >> lbl;
-
-		if (keymap != 0) {
-			State* state = keymap->getStateByLabel(lbl);
-			if (state != 0) {
-				keymap->setCurrentState(state);
-				setText(lbl);
-			}
-		}
-	} else if (msg == "debug(QString)") {
-		QString flag;
-		stream >> flag;
-	}
+    QDataStream stream(data, IO_ReadOnly);
+
+    if (msg == "enable()") {
+        keymap->enable();
+    } else if (msg == "disable()") {
+        keymap->disable();
+    } else if (msg == "reload()") {
+        QCopEnvelope("QPE/System", "busy()");
+        QTimer::singleShot(0, this, SLOT(reload()));
+    } else if (msg == "switch(QString)") {
+        QString lbl;
+        stream >> lbl;
+
+        if (keymap != 0) {
+            State* state = keymap->getStateByLabel(lbl);
+            if (state != 0) {
+                keymap->setCurrentState(state);
+                setText(lbl);
+            }
+        }
+    } else if (msg == "debug(QString)") {
+        QString flag;
+        stream >> flag;
+    }
 }
 
 void ZkbWidget::reload() {
-	loadKeymap();
-	QCopEnvelope("QPE/System", "notBusy()");
+    loadKeymap();
+    QCopEnvelope("QPE/System", "notBusy()");
 }
 
 EXPORT_OPIE_APPLET_v1( ZkbWidget )
-
-
diff --git a/noncore/applets/zkbapplet/zkbwidget.h b/noncore/applets/zkbapplet/zkbwidget.h
index 7c67794..9bce85a 100644
--- a/noncore/applets/zkbapplet/zkbwidget.h
+++ b/noncore/applets/zkbapplet/zkbwidget.h
@@ -1,39 +1,39 @@
 #ifndef ZKBWIDGET_H
 #define ZKBWIDGET_H
 
 #include <qwidget.h>
 #include <qlabel.h>
 #include <qpopupmenu.h>
 #include <qpixmap.h>
 #include <qcopchannel_qws.h>
 
 #include "zkb.h"
 
 class ZkbWidget : public QLabel {
 Q_OBJECT
 
 public:
-	ZkbWidget(QWidget* parent);
-	~ZkbWidget();
-	static int position();
+    ZkbWidget(QWidget* parent);
+    ~ZkbWidget();
+    static int position();
 
-	QSize sizeHint() const;
+    QSize sizeHint() const;
 
 protected:
-	QLabel* label;
-	Keymap* keymap;
-	QPopupMenu* labels;
-	QCopChannel* channel;
-	int w, h;
-	QPixmap disabled;
+    QLabel* label;
+    Keymap* keymap;
+    QPopupMenu* labels;
+    QCopChannel* channel;
+    int w, h;
+    QPixmap disabled;
 
-	bool loadKeymap();
-	void mouseReleaseEvent(QMouseEvent*);
+    bool loadKeymap();
+    void mouseReleaseEvent(QMouseEvent*);
 
 protected slots:
-	void stateChanged(const QString&);
-	void labelChanged(int id);
-	void signalReceived(const QCString& msg, const QByteArray& data);
-	void reload();
+    void stateChanged(const QString&);
+    void labelChanged(int id);
+    void signalReceived(const QCString& msg, const QByteArray& data);
+    void reload();
 };
 #endif
diff --git a/noncore/apps/keyz-cfg/zkb.cpp b/noncore/apps/keyz-cfg/zkb.cpp
index c9e1dc5..a357b88 100644
--- a/noncore/apps/keyz-cfg/zkb.cpp
+++ b/noncore/apps/keyz-cfg/zkb.cpp
@@ -1,52 +1,53 @@
 #include "zkb.h"
 
 /* OPIE */
 #include <opie2/odebug.h>
+#include <opie2/okeyfilter.h>
 
 #include <stdio.h>
 
 // Implementation of Action class
 Action::Action():state(0), keycode(0), unicode(0), flags(0) {
 }
 
 Action::Action(State* s, ushort kc, ushort uni, int f):
     state(s), keycode(kc), unicode(uni), flags(f) {
 }
 
 Action::~Action() {
 }
 
 State* Action::getState() const {
     return state;
 }
 
 void Action::setState(State* s) {
     state = s;
     setDefined(true);
 }
 
 bool Action::hasEvent() const {
     return flags & Event;
 }
 
 void Action::setEvent(bool e) {
     flags = (flags & ~Event) | ((e) ? Event : 0);
 
     if (e) {
         setDefined(true);
     } else {
         if (state == 0) {
             setDefined(false);
         }
     }
 }
 
 bool Action::isDefined() const {
     return flags & Defined;
 }
 
 void Action::setDefined(bool d) {
     flags = (flags & ~Defined) | ((d) ? Defined : 0);
 }
 
 int Action::getKeycode() const {
@@ -201,96 +202,98 @@ bool State::set(int keycode, bool pressed, Action& action) {
     if (n==-1 || keys==0) {
         return false;
     }
 
     if (pressed) {
         n += Key_Max + 1;
     }
 
     keys[n] = action;
     return true;
 }
 
 State* State::getParent() const {
     return parent;
 }
 
 void State::setParent(State* s) {
     parent = s;
 }
 
 int State::translateKeycode(int keycode) const {
     if (keycode < 0x20) {
         return -1;
     }
 
     if (keycode < 0x80) {
         return x1[keycode - 0x20];
     }
 
     if (keycode < 0x1000) {
         return -1;
     }
 
     if (keycode < 0x1057) {
         return x2[keycode - 0x1000];
     }
 
     return -1;
 }
 
 // Implementation of Keymap class
 Keymap::Keymap():enabled(true), currentState(0), autoRepeatAction(0), repeater(this) {
     repeatDelay=400;
     repeatPeriod=80;
     connect(&repeater, SIGNAL(timeout()), this, SLOT(autoRepeat()));
 }
 
 Keymap::~Keymap() {
+    odebug << "removing keyboard filter for zkb"<<oendl;
+    Opie::Core::OKeyFilter::inst()->remHandler(this);
     QMap<QString, State*>::Iterator it;
     for(it = states.begin(); it != states.end(); ++it) {
         delete it.data();
     }
     states.clear();
 }
 
 bool Keymap::filter(int unicode, int keycode, int modifiers,
     bool isPress, bool autoRepeat) {
 
     odebug << "filter: >>> unicode=" << unicode << ", keycode=" << keycode
            << ", modifiers=" << modifiers << ", ispressed=" << isPress << oendl;
 
     if (!enabled) {
         return false;
     }
 
     // the second check is workaround to make suspend work if
     // the user pressed it right after he did resume. for some
     // reason the event sent by qt has autoRepeat true in this
     // case
     if (autoRepeat && keycode != 4177) {
         return true;
     }
 
     (void) unicode; (void) modifiers;
 
     Action* action = currentState->get(keycode, isPress, true);
     if (action==0 || !action->isDefined()) {
         return true;
     }
 
     if (action->hasEvent()) {
         odebug << "filter:<<< unicode=" << action->getUnicode() << ", keycode=" << action->getKeycode()
                << ", modifiers=" << action->getModifiers() << ", ispressed=" << action->isPressed() << oendl;
 
         QWSServer::sendKeyEvent(action->getUnicode(),
             action->getKeycode(), action->getModifiers(),
             action->isPressed(), false);
     }
 
     if (action->isAutorepeat()) {
         autoRepeatAction = action;
         repeater.start(repeatDelay, TRUE);
     } else {
         autoRepeatAction = 0;
     }