Georg Baum wrote:
[EMAIL PROTECTED] wrote:
Author: baum
Date: Wed Jan 31 17:07:37 2007
New Revision: 16981
URL: http://www.lyx.org/trac/changeset/16981
Log:
Fix bug 3107
Modified:
lyx-devel/trunk/src/frontends/qt4/GuiApplication.C
Modified: lyx-devel/trunk/src/frontends/qt4/GuiApplication.C
URL:
http://www.lyx.org/trac/file/lyx-devel/trunk/src/frontends/qt4/GuiApplication.C?rev=16981
==============================================================================
--- lyx-devel/trunk/src/frontends/qt4/GuiApplication.C (original) +++
lyx-devel/trunk/src/frontends/qt4/GuiApplication.C Wed Jan 31 17:07:37
2007 @@ -212,7 +212,7 @@
return true;
}
default:
- return false;
+ return QApplication::event(e);
}
}
I knew that this was no qt bug. Abdel, if there was any reason for returning
false here please refine this fix, but it is clear that we need to call
QApplication::event for all events we do not handel ourselves.
It seems clear to me as well but it's not my code, see below.
In general, as I also wrote on bugzilla: Please don't waist time to discuss
bug severity, rather try to fix the bug. Fixing this did cost me 15 minutes
of reading the QAction and QToolTip docs and debugging.
Sometimes looking with fresh eyes is what's needed.
Georg
PS: I would not be surprised if more bugs like this where something went
wrong during the qt3->qt4 port are discovered.
This event() method does not date backs from my port but was recently
introduced by JMarc in revision 16099.
Thanks for looking at this.
Abdel.
