On Wed, May 30, 2007 at 03:59:15PM -0400, Richard Heck wrote:
> + if (obj == availableLV) {
> + if (event->type() != QEvent::KeyPress)
> + return QObject::eventFilter(obj, event);
> + QKeyEvent * keyEvent = static_cast<QKeyEvent *>(event);
> + int const keyPressed = keyEvent->key();
> + Qt::KeyboardModifiers const keyModifiers =
> keyEvent->modifiers();
> + //Enter key without modifier will add current item
> + //...with control modifier will add it and close the dialog
> + if ((keyPressed == Qt::Key_Enter || keyPressed ==
> Qt::Key_Return) &&
------------------------^
Space. Also please spaces after the ///
> + (!keyModifiers ||
> + (keyModifiers == Qt::ControlModifier) ||
> + (keyModifiers == Qt::KeypadModifier) ||
> + (keyModifiers == (Qt::ControlModifier |
> Qt::KeypadModifier))
Would that be equivalent to
!(keyModifiers & ~(Qt::ControlModifier | Qt::KeypadModifier))
Hm... even if so, it does it look simpler? I am uncertain.
I never used KeypadModifier myself...
Andre'
