Andre Poenitz wrote:
+void MathButton::mouseReleaseEvent(QMouseEvent *event)
+{
+       // this one triggers the action and untoggles the button
+       QToolButton::mouseReleaseEvent(event);
+       // this one forwards the event to the parent
+       QWidget::mouseReleaseEvent(event);
+}

I'd consider

event->ignore()

done


+QWidget * IconPalette::createWidget(QWidget * parent)
+{
+       QWidget * panelwidget_ = new QWidget(parent);
+       QGridLayout * layout_ = new QGridLayout(panelwidget_);

Why the trailing underscore for automatic variables?


because i just knew you'd like 'em... ;-)


+// FIXME: this can go when we move to Qt 4.3
+#define QT_VERSION_CHECK(major, minor, patch) ((major<<16)|(minor<<8)|(patch))
+
+#if QT_VERSION >= QT_VERSION_CHECK(4, 2, 0)

Wouldn't that give a 'redefine macro' ith 4.3?

Why not simply using 0x40200?

because my compiler doesn't like it?

Reply via email to