[Kmymoney-devel] Review Request: Migration to AlkValue

[Thomas Baumgart]

> On 2010-09-12 00:53:35, Alvaro Soliverez wrote:
> > Just one thing to note about this, before I review the whole patch.
> > If there is any change to libalkimia, that has to be ported upstream to the original repository. We will only have a copy of the original code.
> 
> Carlos da Silva wrote:
>     Just a clarification.
>     I have created the patch assuming that I can not change libalkimia, since I don't know who is responsible for it, and it is probably used in other projects.
> 
> Thomas Baumgart wrote:
>     The KDE finance group - where KMyMoney is a member - takes care of it. We also make sure, that we don't break things in the future. So far, KMyMoney will be the first application using it.
>     
>     As I write, I have fixed your problem in upstream Alkimia code. Please update from its SVN repo.
> 
> Jose Villanova wrote:
>     0.07 is a valid octal and 0.08, 0.09 aren't, maybe that's the problem
>     
>     I'm building my build environment, couldn't test it so far :-(
> 
> Carlos da Silva wrote:
>     Thanks Thomas and Jose,
>     
>     I have updated the libalkimia from the upstram Alkimia code and it is now working.
>     
>     Before I update the code in the review, I have another doubt about the String constructor of libalkimia.
>     Apparently, AlkValue String constructor can not deal with "1,123.". It seems that the problem is caused by the "," in the converted string, which is not supported by the GMP mpq_class class.
>     
>     As a temporary fix, I copied the AlkValue string constructor to MyMoneyMoney class and eliminated the thousand separator. It is around line 192 of the mymoneymoney.cpp file.
>     
>     192 // take care of thousand separator
>     193 while(res.count(_thousandSeparator) > 0) {
>     194   pos = res.indexOf(_thousandSeparator);
>     195   res.remove(pos, 1);
>     196 }
>     
>     Is it another fix for the AlkValue class?

Yes, unfortunately. Please see http://websvn.kde.org/trunk/playground/office/alkimia/libalkimia/alkvalue.cpp?r1=1174435&r2=1174787 for the details of the fix which is now in SVN.


- Thomas


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


On 2010-09-11 20:19:08, Carlos da Silva wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5310/
> -----------------------------------------------------------
> 
> (Updated 2010-09-11 20:19:08)
> 
> 
> Review request for kmymoney.
> 
> 
> Summary
> -------
> 
> Need help in porting the mymoneymoney class to AlkValue (from alkimia) in order to deal with overflows (Bug 245214).
> The libalkimia, included in the patch, requires the installation of the GMP library (http://gmplib.org/).
> 
> Notes about the development of the patch are in http://community.kde.org/KMyMoney/PlayingWithAlkValue.
> 
> I need some help with the string constructor.
> It works for the string "0.07", but it does not work for the string "0.08".
> The examples are in the mymoneymoneytest.cpp file.
> 
> 
> This addresses bug 245214.
>     https://bugs.kde.org/show_bug.cgi?id=245214
> 
> 
> Diffs
> -----
> 
>   /trunk/extragear/office/kmymoney/CMakeLists.txt 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/CMakeLists.txt 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/mymoney/CMakeLists.txt 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneymoney.h 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneymoney.cpp 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneymoneytest.h 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneymoneytest.cpp 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/plugins/CMakeLists.txt 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/plugins/icalendarexport/CMakeLists.txt 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/plugins/ofximport/CMakeLists.txt 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/plugins/printcheck/CMakeLists.txt 1174271 
>   /trunk/extragear/office/kmymoney/kmymoney/plugins/reconciliationreport/CMakeLists.txt 1174271 
>   /trunk/extragear/office/kmymoney/libalkimia/CMakeLists.txt PRE-CREATION 
>   /trunk/extragear/office/kmymoney/libalkimia/alkvalue.h PRE-CREATION 
>   /trunk/extragear/office/kmymoney/libalkimia/alkvalue.cpp PRE-CREATION 
> 
> Diff: http://svn.reviewboard.kde.org/r/5310/diff
> 
> 
> Testing
> -------
> 
> All unit tests present in the code.
> Created a new test in the mymoneymoneytest.cpp to show where is the problem.
> 
> 
> Thanks,
> 
> Carlos
> 
>

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