<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">https://phabricator.kde.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>-- <br><div class="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></div>