Review Request 113504: Fix krunner calculator letter check

Christoph Feck christoph at maxiom.de
Wed Oct 30 09:33:50 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113504/
-----------------------------------------------------------

(Updated Oct. 30, 2013, 9:33 a.m.)


Status
------

This change has been marked as submitted.


Review request for kde-workspace and Plasma.


Bugs: 326059
    http://bugs.kde.org/show_bug.cgi?id=326059


Repository: kde-workspace


Description
-------

The calculator runner now supports the syntax without the '=', so you can simply type in '3+5' and have the result. An additional commit added a letter check, so that it will not try to compute with normal text input, but this broke any calculations with any functions, such as '=6*asin 0.5' etc.

This commit removes the letter check in case the user had explicitly typed the '=', restoring 4.10 behavior. If I understand the letter check correctly, it is very hard to support functions in the case the user had omitted the '='


Diffs
-----

  plasma/generic/runners/calculator/calculatorrunner.cpp 0d52301 

Diff: http://git.reviewboard.kde.org/r/113504/diff/


Testing
-------


Thanks,

Christoph Feck

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131030/7437a45c/attachment-0001.html>


More information about the Plasma-devel mailing list