On Mon, Jun 18, 2007 at 06:58:32PM +0200, Leuven, E. wrote: > Andre Poenitz wrote: > > On Mon, Jun 18, 2007 at 04:22:09PM +0200, Leuven, E. wrote: > >> + /* i didn't find a smarter way to find out > >> + * whether a move was made by the user > >> + * because we also move the widget ourselves > >> + * to make it popup in the right position... > >> + */ > >> + switch (event->type()) { > >> + case QEvent::WindowActivate: > >> + active_ = true; > >> + case QEvent::WindowDeactivate: > >> + active_ = false; > > > > So active_ == false in both cases? (missing 'break'?) > > i changed it to the attached...
I get a compile error. What about my proposed patch which leaves things as they are now but allows compilation with any Qt version? g++ -DHAVE_CONFIG_H -I. -I../../../../src/frontends/qt4 -I../../../src -DQT_CLEAN_NAMESPACE -DQT_GENUINE_STR -DQT_NO_STL -DQT_NO_KEYWORDS -DQ_CYGWIN_WIN -I../../../../src -I../../../../src/frontends -I../../../../images -I/usr/local/qt4-cygwin/include -I/usr/local/qt4-cygwin/include/QtCore -I/usr/local/qt4-cygwin/include/QtGui -I../../../../boost -I../../../../src/frontends/controllers -Wno-uninitialized -O2 -MT IconPalette.lo -MD -MP -MF .deps/IconPalette.Tpo -c ../../../../src/frontends/qt4/IconPalette.cpp -o IconPalette.o ../../../../src/frontends/qt4/IconPalette.cpp: In member function `virtual void lyx::frontend::IconPalette::moveEvent(QMoveEvent*)': ../../../../src/frontends/qt4/IconPalette.cpp:49: error: invalid use of undefined type `struct QMoveEvent' /usr/local/qt4-cygwin/include/QtGui/qwidget.h:61: error: forward declaration of `struct QMoveEvent' make[7]: *** [IconPalette.lo] Error 1 -- Enrico