<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 6, 2017 at 2:03 PM, Luigi Toscano <span dir="ltr"><<a href="mailto:luigi.toscano@tiscali.it" target="_blank">luigi.toscano@tiscali.it</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Lays Rodrigues ha scritto:<br>
<span class="">> Hi guys of kde-core.<br>
><br>
> Any new review of AtCore? =D<br>
<br>
</span>I think that there were questions open on your side. Did you address the two<br>
issues, namely:<br>
<span class=""><br>
> On Fri, Jun 23, 2017 at 11:19 PM, Lays Rodrigues  wrote:<br>
<br>
>     -> aacid<br>
><br>
>     "Partially, i personally still think it'd be better if you move the<br>
>     PrinterState AXIS and MeasuramentUnits enums inside AtCore (or make them C++11<br>
>     "enum class").<br>
><br>
>     Also note how PrinterState AXIS MeasuramentUnits is not consistent naming<br>
>     "<br>
><br>
>     For that, I think this is the diff: <a href="https://phabricator.kde.org/D6363" rel="noreferrer" target="_blank">https://phabricator.kde.org/<wbr>D6363</a><br>
</span>>     <<a href="https://phabricator.kde.org/D6363" rel="noreferrer" target="_blank">https://phabricator.kde.org/<wbr>D6363</a>><br>
<br>
This seems to be merged; Albert, does it address your concern?<br>
<span class=""><br>
<br>
><br>
>     -> Luigui<br>
>     "In addition to Albert's comment, I noticed now (still going through the<br>
>     backlog after vacation) that atcore use tr() for messages, but there is no<br>
>     Messages.sh file to extract the strings (which should be called atcore_qt,<br>
>     check the similar files in step or marble or in tier1 frameworks)."​<br>
<br>
</span>I don't think that this has been addressed<br></blockquote><div><br></div><div>It was not, and it's my fault.<br></div><div>I was looking on marble sources but I didn't understood how the Messages.sh works,<br></div><div>I'm reading and will update that as soon as I can.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Luigi<br>
</font></span></blockquote></div><br></div></div>