On Thursday 01 April 2010 20:11:41 Aaron J. Seigo wrote: > On April 1, 2010, you wrote: > > Hi, > > > > I am currently working on improving the calculator runner, by adding an > > optional dependency on libqalculate. If the library is found, the > > calculator runner will be built against libqalculate and it will support > > "advanced" operations (such as currency conversion, unit conversion, > > equation solving, ...). Otherwise it will fall back to the existing code > > that uses QtScript. I have tested the runner for some weeks now and it is > > stable. > > great; can you post a patch to review-board for this so we can triage it in > asap? > > > I would like to hear some opinions about these points: > > - currently there are two calculator plasmoids, with and without > > libqalculate support (btw I am the author of the qalculate one). I think > > that we could use a similar approach here as with the runner, i.e. have a > > unified calculator plasmoid that optionally depends on libqalculate. > > agreed > > > - a Qalculate backend for Cantor is currently in playground (not > > developed by me). It is already quite stable and I think It would be > > nice to have it included in SC 4.5. > > > > - one problem is that the runner, applet and Cantor backend have some > > duplicate code (e.g. evaluation options, support for QString, ...). As > > Aaron suggested some time ago, maybe it is worth it to add a wrapper > > library for libqalculate use in KDE (something like libkdeqalculate, feel > > free to suggest a nicer name ;). Where should this library be placed? > > Would > > kdebase/workspace/libs be an appropriate place? > > only if it is used by things in kdebase-workspace alone. otherwise, i'd > recommend kdesupport and ship it as a separate library. > > > Of course I volunteer to take care of these tasks before 4.5 freeze ;) > > terrific :) So, the proposed patch has been sitting in reviewboard for a couple of weeks now (http://reviewboard.kde.org/r/3483/). Did someone test it? I would appreciate some feedback before going on with the other points I mentioned.
Thanks, Matteo
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel