Ticket #12996: 0010-Fix-compiler-enumeral-and-non-enumeral-type-in-condi.patch

File 0010-Fix-compiler-enumeral-and-non-enumeral-type-in-condi.patch, 1.7 KB (added by David Hampton <mythtv@…>, 3 years ago)

Fix 'enumeral and non-enumeral type' warning in mythmainwindow.cpp.

  • mythtv/libs/libmythui/mythmainwindow.cpp

    Fix compiler 'enumeral and non-enumeral type in conditional
    expression' warning.
    
    Rewrite the TranslateKeyNum function to eliminate the use of the
    number zero and only use defined constants.
    ---
     mythtv/libs/libmythui/mythmainwindow.cpp | 17 +++++++++++------
     1 file changed, 11 insertions(+), 6 deletions(-)
    
    diff --git a/mythtv/libs/libmythui/mythmainwindow.cpp b/mythtv/libs/libmythui/mythmainwindow.cpp
    index 8a903c6..8be5ff7 100644
    a b int MythMainWindowPrivate::TranslateKeyNum(QKeyEvent* e) 
    317317        // if modifiers have been pressed, rebuild keynum
    318318        if ((modifiers = e->modifiers()) != Qt::NoModifier)
    319319        {
    320             int modnum = (((modifiers & Qt::ShiftModifier) &&
    321                             (keynum > 0x7f) &&
    322                             (keynum != Qt::Key_Backtab)) ? Qt::SHIFT : 0) |
    323                          ((modifiers & Qt::ControlModifier) ? Qt::CTRL : 0) |
    324                          ((modifiers & Qt::MetaModifier) ? Qt::META : 0) |
    325                          ((modifiers & Qt::AltModifier) ? Qt::ALT : 0);
     320            int modnum = Qt::NoModifier;
     321            if ((modifiers & Qt::ShiftModifier) &&
     322                (keynum > 0x7f) &&
     323                (keynum != Qt::Key_Backtab))
     324                modnum |= Qt::SHIFT;
     325            if (modifiers & Qt::ControlModifier)
     326                modnum |= Qt::CTRL;
     327            if (modifiers & Qt::MetaModifier)
     328                modnum |= Qt::META;
     329            if (modifiers & Qt::AltModifier)
     330                modnum |= Qt::ALT;
    326331            modnum &= ~Qt::UNICODE_ACCEL;
    327332            return (keynum |= modnum);
    328333        }