<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/127131/">https://git.reviewboard.kde.org/r/127131/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On February 21st, 2016, 6:49 p.m. UTC, <b>Michael Pyne</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Jenkins has reported a build failure (https://build.kde.org/job/kcoreaddons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/117/console), looks like we might have dueling compiler errors:</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><div class="codehilite" style="background: #f8f8f8"><pre style="line-height: 125%"><span style="color: #666666">18:07:28</span> <span style="color: #BB6688">/home/jenkins/builds/kcoreaddons/kf5-qt5/src/lib/io/</span>kdirwatch<span style="color: #666666">.</span><span style="color: #7D9029">cpp</span><span style="color: #666666">:137:1:</span> error<span style="color: #666666">:</span> control reaches end of non<span style="color: #666666">-</span><span style="color: #008000; font-weight: bold">void</span> <span style="color: #008000; font-weight: bold">function</span> <span style="color: #666666">[-</span>Werror<span style="color: #666666">=</span><span style="color: #008000; font-weight: bold">return</span><span style="color: #666666">-</span>type<span style="color: #666666">]</span>
</pre></div>
</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Seems that the GCC used on CI doesn't recognize that every possible error case is handled in the switch (or alternately, it assumes that someone might try to shoehorn an out-of-range enumeration value into the function). GCC 5.3 works fine, which is what I'd used for testing. But it might be easier to revert the switch/case portions of the patch, perhaps add warning suppressions instead?</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Thanks for the heads up. I added a return statement after the switch, should keep both compilers happy.</p></pre>
<br />
<p>- David</p>
<br />
<p>On February 21st, 2016, 5:39 p.m. UTC, David Faure wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KDE Frameworks and Michael Pyne.</div>
<div>By David Faure.</div>
<p style="color: grey;"><i>Updated Feb. 21, 2016, 5:39 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kcoreaddons
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">(C casts, global functions without static, unused default statements...)
This detected a small bug in kdirwatch, where fall-through was unintended
(I know, because I wrote that code).</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Compiled with clang; ran autotests.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>autotests/kjobtest.cpp <span style="color: grey">(aedde39f35ec7bafeb13ffd33d8eb871f60b29eb)</span></li>
<li>autotests/kprocesstest.cpp <span style="color: grey">(2ac756c973b12b895af6fa34ff5077d2d064af8b)</span></li>
<li>src/desktoptojson/main.cpp <span style="color: grey">(45bdcba8f5e7ab395f52ebf8d31ccde826e77434)</span></li>
<li>src/lib/caching/kshareddatacache.cpp <span style="color: grey">(0c1c61cc7f70969748d102c8313fadc73cef55a5)</span></li>
<li>src/lib/io/kautosavefile.cpp <span style="color: grey">(e540630179b17b18b7dce5480969aab89d45be34)</span></li>
<li>src/lib/io/kdirwatch.cpp <span style="color: grey">(4b900228c74c609d26e52542bd42e124c6d1d39b)</span></li>
<li>src/lib/io/kdirwatch_p.h <span style="color: grey">(af74ff15869b6f9c16bb14f81b6c488dd082de19)</span></li>
<li>src/lib/io/kfilesystemtype.cpp <span style="color: grey">(3526de8ab710f31dabad09026b9b993785658033)</span></li>
<li>src/lib/io/kmessage.h <span style="color: grey">(590a1fe961a220bb2e879b35b4936bf9a1cdfdbb)</span></li>
<li>src/lib/io/kprocess.cpp <span style="color: grey">(2bd8aee1293adf3b5fe549fa2cff675160a5ea96)</span></li>
<li>src/lib/jobs/kjob.cpp <span style="color: grey">(1ad2c2858ee7ed2167a740a10fb7b185d8833288)</span></li>
<li>src/lib/kaboutdata.cpp <span style="color: grey">(3193808797c53586bea94768dc81867e35a8a535)</span></li>
<li>src/lib/plugin/desktopfileparser.cpp <span style="color: grey">(1122af8ba3640a7f06819063157069b652a4f904)</span></li>
<li>src/lib/plugin/desktopfileparser_p.h <span style="color: grey">(f19be800e5af2cc120a8ad4f9561e92dbf9fd2c7)</span></li>
<li>src/lib/plugin/kpluginloader.cpp <span style="color: grey">(530e7b865022b680a473fceceb7d659196e9420d)</span></li>
<li>src/lib/randomness/krandom.cpp <span style="color: grey">(93b917d23c2ab8ec7a10277ea8bd49d6f9a61fd7)</span></li>
<li>src/lib/randomness/krandomsequence.cpp <span style="color: grey">(01490bfac5e5fda0ba509fb791bdd1e791f90cbd)</span></li>
<li>src/lib/text/kmacroexpander_unix.cpp <span style="color: grey">(f773291ffe961c1a01235eb6eaf79488461eac59)</span></li>
<li>src/lib/text/ktexttohtml.cpp <span style="color: grey">(0fe2a2d69de1d42341cb3ad50c236c21c0aa1f06)</span></li>
<li>src/lib/text/ktexttohtmlemoticonsinterface.h <span style="color: grey">(8fcd30f6c93424aeb103243a0e080798a56fdcc1)</span></li>
<li>src/lib/util/kformatprivate.cpp <span style="color: grey">(9af072dcf3068d8056582be046e38bab48a548a5)</span></li>
<li>src/lib/util/kshell.cpp <span style="color: grey">(0f2e1d46e0761320b871949da594a9b60e2feaf5)</span></li>
<li>src/lib/util/kuser_unix.cpp <span style="color: grey">(01120f6928e04682ca11c0a0822047b43c2f047b)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/127131/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>