<div dir="ltr"><div class="gmail_default" style="font-size:small">Hi guys of kde-core.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Any new review of AtCore? =D</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Cheers,</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 23, 2017 at 11:19 PM, Lays Rodrigues <span dir="ltr"><<a href="mailto:laysrodriguessilva@gmail.com" target="_blank">laysrodriguessilva@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small">Yay, now we are on track again. This will be the right thread for the review.</div><div class="gmail_default" style="font-size:small">I will address here the issues that aacid and Luigui opened on the previous thread and allow the others AtCore devels answer it.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">-> aacid</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">"<span style="font-size:12.8px">Partially, i personally still think it'd be better if you move the</span></div><span style="font-size:12.8px">PrinterState AXIS and MeasuramentUnits enums inside AtCore (or make them C++11</span><br style="font-size:12.8px"><span style="font-size:12.8px">"enum class").</span><br style="font-size:12.8px"><br style="font-size:12.8px"><span style="font-size:12.8px">Also note how PrinterState AXIS MeasuramentUnits is not consistent naming<div class="gmail_default" style="font-size:small;display:inline">"</div></span><div><br></div><div><div class="gmail_default" style="font-size:small">For that, I think this is the diff: <a href="https://phabricator.kde.org/D6363" target="_blank">https://phabricator.kde.<wbr>org/D6363</a></div><br></div><div><div class="gmail_default" style="font-size:small">-> Luigui</div><div class="gmail_default" style="font-size:small">"<span style="font-size:12.8px">In addition to Albert's comment, I noticed now (still going through the</span></div><span style="font-size:12.8px">backlog after vacation) that atcore use tr() for messages, but there is no</span><br style="font-size:12.8px"><span style="font-size:12.8px">Messages.sh file to extract the strings (which should be called atcore_qt,</span><br style="font-size:12.8px"><div class="gmail_default" style="font-size:small"><span style="font-size:12.8px">check the similar files in step or marble or in tier1 frameworks)."</span>​</div><div><br></div><div><div class="gmail_default" style="font-size:small">Cheers,</div></div><span class="HOEnZb"><font color="#888888">-- <br><div class="m_-1783090699367028210gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><font color="#000000"><b style="font-size:12.8px">Lays Rodrigues</b><br></font></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">Software Developer at KDE</font></b></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">Intern at Rede Globo</font></b></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">Computer Science student at Federal Fluminense University</font></b></div><div><b><a href="http://laysrodriguesdev.wordpress.com" target="_blank"><font color="#000000">laysrodriguesdev.wordpress.com</font></a></b></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">Telegram: @lays147</font></b></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">IRC: lays147</font></b></div><div><b><font color="#000000">Phone: <a href="tel:+55%2022%2098152-0012" value="+5522981520012" target="_blank">+55 22 981520012</a></font></b><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>
</font></span></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><font color="#000000"><b style="font-size:12.8px">Lays Rodrigues</b><br></font></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">Software Developer at KDE</font></b></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">Intern at Rede Globo</font></b></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">Computer Science student at Federal Fluminense University</font></b></div><div><b><a href="http://laysrodriguesdev.wordpress.com" target="_blank"><font color="#000000">laysrodriguesdev.wordpress.com</font></a></b></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">Telegram: @lays147</font></b></div><div><b><font style="background-color:rgb(255,255,255)" color="#000000">IRC: lays147</font></b></div><div><b><font color="#000000">Phone: +55 22 981520012</font></b><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>
</div>