D15463: Corrected behaviour on repeated pressing of operator buttons

[Maximilian Schiller]

mschiller created this revision.
mschiller added reviewers: cfeck, teran, KDE Applications.
Herald added a subscriber: kde-utils-devel.
mschiller requested review of this revision.

REVISION SUMMARY
  Fixes "wrong" behavior to the correct one as described in the bug report.
  Before the fix pressing "10 * / 5 =" normally would result in 20 (Stack: (10,*), (10,/), (5,=)) but with those changes it results in 2 (Stack: (10, /), (5, =)).
  
  Internally this is achieved by "blocking" node adding after a operator (*,-,/,...) has been pressed until a number is entered. If another operator is pressed whilst blocked the top (last) operator is changed instead.
  
  BUG: 57349

REPOSITORY
  R353 KCalc

REVISION DETAIL
  https://phabricator.kde.org/D15463

AFFECTED FILES
  kcalc.cpp
  kcalc_core.cpp
  kcalc_core.h

To: mschiller, cfeck, teran, #kde_applications
Cc: kde-utils-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20180912/f709cf6b/attachment.html>