<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Feb 27, 2014 at 5:15 PM, John Layt <span dir="ltr"><<a href="mailto:jlayt@kde.org" target="_blank">jlayt@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi,<br>
<br>
I've been reviewing KUnitConversion (KUC for short) and doing some small<br>
clean-ups, and I'm slowly coming to the conclusion I'm not a fan of the api.<br>
However it is used in a few places, so rather than try rewrite the api in the<br>
time remaining, I'll finish up the clean-ups and we can release it as is and<br>
look to replace it later on.<br>
<br>
The main clean-ups are:<br>
<br>
* Remove use of KCurrencyCode.  Currently it's only used to get the currency<br>
name of the 50 currencies KUC supports and it seems overkill to keep all 207<br>
data files and the api in KUC just for that purpose.  Instead I'd move it to<br>
kde4support with the rest of KLocale (we also need to move the config files<br>
from kde-runtime).  This would also remove the dependency on KConfigCore.<br>
<br>
* Try port away from ki18n to tr().  KUC makes extensive use of ki18nc() and<br>
ki18ncp(), but I need to check with Chusselove if all the plural translations<br>
can be handled with Qt or if any are scripted.<br>
<br>
* If we remove those two dependencies then KUC becomes Tier 1, but Tier 2 is<br>
fine either way.<br>
<br>
* Convert more classes to QSharedData<br>
<br>
* Some small API changes<br>
<br>
* Lots of docs clean-ups, code style, etc are needed but can be done later.<br>
<br>
I've done the first step, and I just need a volunteer to do the git magic<br>
required to:<br>
<br>
* Move kcurrencycode.h and kcurrencycode.cpp including history from<br>
"kunitconversion/src" to "kde4support/src/kdecore"<br>
<br>
* Move "kde-runtime/localization" and everything in it including history to<br>
"kde4support/src/localization" or "kde4support/runtime/localization" or<br>
wherever deemed appropriate<br>
<br>
* Move "kde-runtime/l10n" and everything in it including history into the same<br>
"localization" folder as above but folder renamed from "l10n" to<br>
"localization/country" (i.e. at same level as "currency" folder)<br>
<br>
* Make sure the data files are built and installed, but need to think about<br>
parallel installs with KDE4 data files<br>
<br>
The data file moves are part of the kde-runtime clean-up, but it makes sense<br>
to do them alongside the code moves.<br>
<br>
Longer term, I've started work on a new Tier 1 library tentatively called<br>
KStandards to support the different ISO code standards, and measurement<br>
standards and conversions.  I've already converted KCurrencyCode to a new<br>
KCurrency class using json files, and KCountry and KUnits should follow in due<br>
course to provide a future migration path for apps that need them.<br>
<br>
Cheers!<br>
<br>
John.<br>
<br>
_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
</blockquote></div><br></div><div class="gmail_extra">Can we get an update on what's the status of this? Is KCurrency going to kde4support? Maybe it's not needed to deprecate it yet? It can get deprecated at some point in the future within KUnitConversion and then we can add new classes when the ISO alternatives are available.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Aleix</div></div>