----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/6169/#review9335 -----------------------------------------------------------
/trunk/KDE/kdenetwork/kopete/protocols/skype/skypecalldialog.cpp <http://svn.reviewboard.kde.org/r/6169/#comment10340> KLocale::formatMoney accepts a double as its first argument -- why not declare value as a double here? /trunk/KDE/kdenetwork/kopete/protocols/skype/skypecalldialog.cpp <http://svn.reviewboard.kde.org/r/6169/#comment10339> Extra whitespace? /trunk/KDE/kdenetwork/kopete/protocols/skype/skypecalldialog.cpp <http://svn.reviewboard.kde.org/r/6169/#comment10344> It's probably a best practice to use a C++ cast here: value = static_cast<double>(balance) / currencyCode.subunitsPerUnit(); Have you checked if there might be any rounding problems with this division? - Raphael On 2010-12-20 11:14:11, Jeffery MacEachern wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboard.kde.org/r/6169/ > ----------------------------------------------------------- > > (Updated 2010-12-20 11:14:11) > > > Review request for Kopete and Pali Rohár. > > > Summary > ------- > > Originally, the balance display was hard-coded for Euros, and mistakenly > showed no balance otherwise. Using KCurrencyCode to parse the currency > abbreviation from Skype allows for it to display any currency Skype supports. > > > Diffs > ----- > > /trunk/KDE/kdenetwork/kopete/protocols/skype/skypecalldialog.cpp 1207974 > > Diff: http://svn.reviewboard.kde.org/r/6169/diff > > > Testing > ------- > > Tried switching currencies on the Skype website to a few different ones, and > observing the result. I am not entirely certain that my use of > KCurrencyCode::subunitsPerUnit() is correct across all currencies, but it > worked in my tests. Confirmation of that would be appreciated. > > > Thanks, > > Jeffery > >
_______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel
