<table><tr><td style="">piotrkakol marked an inline comment as done.<br />piotrkakol 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/D25642">View Revision</a></tr></table><br /><div><div><p>Yes, I have tried it. At first I just launched the game and checked if the functionality worked. After your last comment I deleted all files and checked all scenarios I could think of from the start. I'm sorry that I didn't do it before I submitted this diff. I didn't think about the initial state of the game.</p>

<p>Also, I have replied to both of your comments inline. I can't see my comments, though, when I'm not logged in. Don't know if you can see them when you're logged in (unintuitive, if they somehow are not visible for logged in users). Here they are:</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>You're correct. When the game is first launched after the update the starting value would be 0 instead of -1 and would never be updated. Thank you.<br />
 You're right again. Sorry that I didn't reread the entire code after your first comment. Reread it now and changed 1 more thing. I couldn't find a scenario, though, in which this 0 would be a problem (but of course better safe than sorry with -1).</p></blockquote></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D25642#inline-144878">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">dealer.cpp:1751</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">shouldn't this be -1 too?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">You're right again. Sorry that I didn't reread the entire code after your first comment. Reread it now and changed 1 more thing. I couldn't find a scenario, though, in which this 0 would be a problem (but of course better safe than sorry with -1).</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D25642#inline-144701">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">statisticsdialog.cpp:125</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Shouldn't that 0 be -1 ?</p>

<p style="padding: 0; margin: 8px;">Otherwise for already recorded stats you'll get a 0, no?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">You're correct. When the game is first launched after the update the starting value would be 0 instead of -1 and would never be updated. Thank you.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R410 KPatience</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25642">https://phabricator.kde.org/D25642</a></div></div><br /><div><strong>To: </strong>piotrkakol, KDE Games<br /><strong>Cc: </strong>aacid, kde-games-devel<br /></div>