<table><tr><td style="">mglb added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D19859">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D19859#434395" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19859#434395</a>, <a href="https://phabricator.kde.org/p/hindenburg/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@hindenburg</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>To get it compile w/ 5.9.4, I have to comment out all the QCOMPAREs  - I thought I tested this w/ 5.9.7 but I appear to have upgraded my VMs</p>

<p>src/autotests/KeyboardTranslatorTest.cpp:112: error: undefined reference to 'bool QTest::qCompare<Qt::Key, int>(Qt::Key const&, int const&, char const*, char const*, char const*, int)'</p></div>
</blockquote>

<p>Turns out it was a code with undefined behavior:</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>QCOMPARE is very strict on the data types. Both actual and expected have to be of the same type, otherwise the test won't compile. This prohibits unspecified behavior from being introduced; that is behavior that usually occurs when the compiler implicitly casts the argument.<br />
 <a href="https://doc.qt.io/qt-5.9/qtest.html#QCOMPARE" class="remarkup-link" target="_blank" rel="noreferrer">https://doc.qt.io/qt-5.9/qtest.html#QCOMPARE</a></p></blockquote>



<blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D19859#434854" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19859#434854</a>, <a href="https://phabricator.kde.org/p/hindenburg/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@hindenburg</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I changed it to 5.9.4; it required some #ifs in KeyboardTranslatorTest.cpp;  the only 5.9.x I could find were .4 and .5</p></div>
</blockquote>

<p>When I need different Qt version than available in OS repos, I use prebuilt Qt from qt.io - all Qt5 versions available (at least in online installer).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19859">https://phabricator.kde.org/D19859</a></div></div><br /><div><strong>To: </strong>mglb, Konsole<br /><strong>Cc: </strong>hindenburg, konsole-devel, gennad, maciejn, thsurrel, ngraham, maximilianocuria<br /></div>