<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/125906/">https://git.reviewboard.kde.org/r/125906/</a>
</td>
</tr>
</table>
<br />
<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 Games.</div>
<div>By Martin Walch.</div>
<p style="color: grey;"><i>Updated Nov. 4, 2015, 12:45 vorm.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">removed whitespace cleanups and spelling fixes in comments that are not part of the API documentation
Add some include cleanups</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Summary (updated)</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;">Fix lots of warnings (gcc and krazy), API doc fixes, cleanup includes ...</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
lskat
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description (updated)</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;">Fix warnings:</p>
<ul style="padding: 0;text-rendering: inherit;margin: 0 0 0 1em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">explicit constructors</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">naming convention of include guards</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">allocate dialogs on heap and hold in QPointer</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">add newline at EOF</li>
</ul>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Update website in README, main.cpp, and lskat.lsm</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix some issues in API documentation</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Fix some spelling errors in strings</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Clean up includes, in particular remove unused includes</p></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 on KF5.
Started and played some rounds, no obvious downsides encountered.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>README <span style="color: grey">(d6131309dfd7d2dc6bd60dad15c089d51c6184c0)</span></li>
<li>lskat.lsm <span style="color: grey">(bd90d94eb9a38625ee9de2c5bd5c95ccec4c29a6)</span></li>
<li>src/abstractengine.h <span style="color: grey">(e6aae39027ca6c6ca917f53881d80780f1d6a23f)</span></li>
<li>src/abstractinput.h <span style="color: grey">(d1d831ad0455ffa90bb805d1e18477bb205f0cbd)</span></li>
<li>src/aiinput.h <span style="color: grey">(a9aac3a2bd54e3ee3a493ecdc48d3678f20b94c8)</span></li>
<li>src/cardsprite.h <span style="color: grey">(074fab9abbbaf7312f916b7606e052753b4d2514)</span></li>
<li>src/cardsprite.cpp <span style="color: grey">(a2e23c66344206861f0527985f9bffa5e9a99b2d)</span></li>
<li>src/config-src.h.cmake <span style="color: grey">(2375fd30d6de1ab735a4c1ad94774ccf403d7339)</span></li>
<li>src/config_two.h <span style="color: grey">(7e7b256a1ff580dd9c9948dceeb28310e3d3867c)</span></li>
<li>src/deck.h <span style="color: grey">(68a1d927315b127c7f3bfaf15278dd5c7b09fc17)</span></li>
<li>src/display_intro.cpp <span style="color: grey">(027822d694e9b4f1a7b4cfefe7eef7d01835390c)</span></li>
<li>src/fromlibkdegames/cardcache.h <span style="color: grey">(8d5f99e7c0983bd7da510b4c2e0e16d60df67cd6)</span></li>
<li>src/fromlibkdegames/cardcache.cpp <span style="color: grey">(ec6e78c0fd521722e66b44a76514302493b7f861)</span></li>
<li>src/fromlibkdegames/cardcache_p.h <span style="color: grey">(effd6180c5104af6276e6e8e84e43dfac11b0be4)</span></li>
<li>src/fromlibkdegames/carddeckinfo.h <span style="color: grey">(57cd7b530e7192e9b016539b83098e491437da2d)</span></li>
<li>src/fromlibkdegames/carddeckinfo_p.h <span style="color: grey">(70df1bffe3b32ff4cf772ee5f22731bef27a22c4)</span></li>
<li>src/fromlibkdegames/kcarddialog.h <span style="color: grey">(040fd9d8a59f22890207b6a1b64778edc663d8e0)</span></li>
<li>src/fromlibkdegames/kcarddialog.cpp <span style="color: grey">(84602e29f075a7a85c417227cdadd0f47a3a5170)</span></li>
<li>src/gameview.h <span style="color: grey">(d052b010f267ef9ba9896b62cb19fc1038cd823d)</span></li>
<li>src/gameview.cpp <span style="color: grey">(11fc524d83cbfe40a580fcb63d27c4db19c8a655)</span></li>
<li>src/main.cpp <span style="color: grey">(51afdc7b6bfa00bda81dc7bfe3416570c02ce756)</span></li>
<li>src/mainwindow.h <span style="color: grey">(e516498b27be7fb2041da4c0db43edfdf111fd72)</span></li>
<li>src/mainwindow.cpp <span style="color: grey">(82a11ca6d4e29396b74d2927faa717d9aa4a8e8b)</span></li>
<li>src/mouseinput.h <span style="color: grey">(3c15cc3c452963a186f5e0a5dbc3663d743e1fbd)</span></li>
<li>src/namedialogwidget.h <span style="color: grey">(9a05f0e25ec424fe7a7ba3c79405e766ce2d59c7)</span></li>
<li>src/pixmapsprite.h <span style="color: grey">(979fd8b923452be0bbb649e4ebd761da8957779c)</span></li>
<li>src/pixmapsprite.cpp <span style="color: grey">(ee0e09f182eb48808b328915f0f1ace2f7ff8f15)</span></li>
<li>src/scoresprite.h <span style="color: grey">(b14fdc90223d895df3389a6c859578009688bbbc)</span></li>
<li>src/scoresprite.cpp <span style="color: grey">(2b0be3136bb5c2abfd857d36bd47c54778826388)</span></li>
<li>src/textsprite.h <span style="color: grey">(15cf431fd2aec239a31a9bc261900dd8a913afbd)</span></li>
<li>src/thememanager.cpp <span style="color: grey">(74caf04f7ee6c0a9383fcfbacb41bb57f472eee4)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/125906/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>