Hi guys of kde-core. Any new review of AtCore? =D
Cheers, On Fri, Jun 23, 2017 at 11:19 PM, Lays Rodrigues < laysrodriguessi...@gmail.com> wrote: > Yay, now we are on track again. This will be the right thread for the > review. > I will address here the issues that aacid and Luigui opened on the > previous thread and allow the others AtCore devels answer it. > > -> aacid > > "Partially, i personally still think it'd be better if you move the > PrinterState AXIS and MeasuramentUnits enums inside AtCore (or make them > C++11 > "enum class"). > > Also note how PrinterState AXIS MeasuramentUnits is not consistent naming > " > > For that, I think this is the diff: https://phabricator.kde.org/D6363 > > -> Luigui > "In addition to Albert's comment, I noticed now (still going through the > backlog after vacation) that atcore use tr() for messages, but there is no > Messages.sh file to extract the strings (which should be called atcore_qt, > check the similar files in step or marble or in tier1 frameworks)." > > Cheers, > -- > *Lays Rodrigues* > *Software Developer at KDE* > *Intern at Rede Globo* > *Computer Science student at Federal Fluminense University* > *laysrodriguesdev.wordpress.com <http://laysrodriguesdev.wordpress.com>* > *Telegram: @lays147* > *IRC: lays147* > *Phone: +55 22 981520012 <+55%2022%2098152-0012>* > -- *Lays Rodrigues* *Software Developer at KDE* *Intern at Rede Globo* *Computer Science student at Federal Fluminense University* *laysrodriguesdev.wordpress.com <http://laysrodriguesdev.wordpress.com>* *Telegram: @lays147* *IRC: lays147* *Phone: +55 22 981520012*