Am Montag 31 August 2009 16:35:52 schrieb Matteo Agostinelli:
> Hello list,
>
> first of all a disclaimer: this is my first contribution to KDE, so
> bear with me if I made any mistakes. I moved my Qalculate! plasmoid to
> kdereview aiming for an inclusion in kdeplasma-addons.
>
> Basically it is an extension of the existing calculator plasmoid,
> using the quite powerful Qalculate! library
> (http://qalculate.sourceforge.net/reference). Among the benefits:
> - unit conversion
> - symbolic expression evaluation
> - basic matrix and vector operations
>
> Since I am not an experienced KDE developer (nor a C++ guru, although
> I have some experience) I would really appreciate some feedback, to
> improve the quality of the applet and the code. 

Launching the Settings of qualculate i see a lot of 
"Settings(I18N_EXCESS_ARGUMENTS_SUPPLIED)" caused by broken i18n calls like
"i18nc("Evaluation", "Evaluation Settings", m_applet->name())"

The applet does not respect the locale settings for decimal separators (',' = 
comma in a german locale) in the input line.

Same problem as with kdeedu/step and libqalculate, see
https://bugs.kde.org/show_bug.cgi?id=181598

Does the applet support translated unit names in conversion?
Afaik libcalculate does not :-(

-- 
Burkhard Lück
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel