[Kmymoney-devel] Re: Review Request: Balance cache object for use by MyMoneyFile

[Fernando Vilas]

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

(Updated 2010-12-26 17:47:56.183315)


Review request for kmymoney.


Changes
-------

Incorporated changes suggested by Thomas. Still TODO: take advantage of balances for previous dates when calculating later balances, investigate precaching, investigate moving all balance calculation logic to MMFile.


Summary
-------

Initial cut at the balance cache for the MyMoneyFile layer. Nothing uses it other than the test driver yet, but that will change once this is committed. Does it have a reasonable interface? Does it need something else added, like a model interface?

This interface is subject to change for now, but should be stable enough to start the next phase if it does not need revision first.


Diffs (updated)
-----

  /trunk/extragear/office/kmymoney/kmymoney/mymoney/CMakeLists.txt 1209433 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneybalancecache.h PRE-CREATION 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneybalancecache.cpp PRE-CREATION 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneybalancecachetest.h PRE-CREATION 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneybalancecachetest.cpp PRE-CREATION 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyfile.h 1209433 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyfile.cpp 1209433 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/imymoneystorage.h 1209433 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneyseqaccessmgr.h 1209433 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneyseqaccessmgr.cpp 1209433 
  /trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneyseqaccessmgrtest.cpp 1209433 

Diff: http://svn.reviewboard.kde.org/r/5404/diff


Testing
-------

Built test driver, and the object passes all tests. Hooray, test-driven-development


Thanks,

Fernando

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kmymoney-devel/attachments/20101226/d1139160/attachment.htm