[Kmymoney-devel] Review Request 115551: Performance optimization for MyMoneySeqAccessMgr::calculateBalance().

Cristian Oneț onet.cristian at gmail.com
Sat Feb 8 12:49:20 UTC 2014


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

Review request for KMymoney and Thomas Baumgart.


Repository: kmymoney


Description
-------

Performance optimization for MyMoneySeqAccessMgr::calculateBalance().

It's still the most expensive method of MyMoneySeqAccessMgr but it's
faster now.

I don't really see why the balance of multiple accounts was computed
just the return the balance of the requested account while throwing
away the rest of the data.

Thomas, I need you opinion on this. 


Diffs
-----

  kmymoney/mymoney/storage/mymoneyseqaccessmgr.cpp ccf2d5873c6cd0ff039c317cd15d237b82426d2d 

Diff: https://git.reviewboard.kde.org/r/115551/diff/


Testing
-------

Ran unit-tests and checked account balances againts the previous version.


Thanks,

Cristian Oneț

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20140208/6fe0094a/attachment.html>


More information about the KMyMoney-devel mailing list