<div><div dir="auto">I can revert the git ignore thing.</div></div><div dir="auto">I Belive I messed up on squashing his commits as I was called to help him fixing a bad rebase. </div><div dir="auto"><br></div><div dir="auto">If you want I can do a followup commit fixing those issues.</div><div dir="auto"><br></div><div><br><div class="gmail_quote"><div dir="ltr">Em ter, 18 de set de 2018 às 19:42, Albert Astals Cid <<a href="mailto:noreply@phabricator.kde.org">noreply@phabricator.kde.org</a>> escreveu:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><table><tbody><tr><td>aacid added a comment.
</td><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/D15575" target="_blank">View Revision</a></td></tr></tbody></table><br><div><div><p>The commit log seems a bit strange, it says "Remove warnings by replacing deprecated methods", but i don't see [almost] any method replaced, it's mostly 0 -> nullptr and some static_cast to qobject_cast.</p>

<p>Can you explain the reason for the static_cast->qobject_cast change?</p></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/D15575#inline-84092" target="_blank">View Inline</a><span style="color:#4b4d51;font-weight:bold">.gitignore:1</span></div>
<div style="font:11px/15px "Menlo","Consolas","Monaco",monospace;white-space:pre-wrap;clear:both;padding:4px 0;margin:0"><div style="padding:0 8px;margin:0 4px;background:rgba(151,234,151,.6)">CMakeLists.txt.user
</div></div></div>
<div style="margin:8px 0;padding:0 12px"><p style="padding:0;margin:8px">We don't usually add .gitignore files to repos as far as i remember</p></div></div></div></div></div><br><div><strong>REPOSITORY</strong><div><div>R332 Kiten</div></div></div><br><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15575" target="_blank">https://phabricator.kde.org/D15575</a></div></div><br><div><strong>To: </strong>vallim, tcanabrava, KDE Edu<br><strong>Cc: </strong>aacid, kde-edu, narvaez, apol<br></div></blockquote></div></div>