User Details
- User Since
- Apr 18 2015, 6:27 AM (469 w, 6 d)
- Availability
- Available
Jul 5 2018
Jun 10 2018
May 16 2018
May 13 2018
May 12 2018
I like this very much!
Why was it disabled in the first place if the fix is that easy?
actually I liked the separate class for exception handling. It allows to distinguish between issues within KMyMoney and outside of it. Typically we do not/cannot handle things like std::bad_alloc so the “missing” catch is desired. Also there is some code which expects this behavior (I think the onlineJob casting stuff), just replacing everything is dangerous. You could inherit MyMoneyException from std::runtime_error. Then we can knowingly change the catches where it is useful and safe.
I am not into this code, so I can only give general advice.
Apr 22 2018
Did not test, but looks good! Can you push yourself?
Apr 21 2018
Apr 9 2018
Apr 8 2018
Apr 7 2018
I want to raise a usability concern. If the user is using the online banking and this page gets disabled (e.g. accidentally, packaging error, some other error) KMyMoney still offers to queue transactions. However, without this plugin the transaction seems to be lost. Queued transactions are also kept during restarts. This could lead to further trouble if the queue is sent after the user forgot about the (hidden) transaction(s).
Mar 26 2018
Feb 4 2018
I found a new warning which is generated by this patch. However, I would leave it as it is, because there will be a warning or we may miss when a change to CMake 3.10 becomes mandatory.
Feb 3 2018
Hi Thomas,
Jan 30 2018
The first two notes are not so important but the corresponding code should get a /**@todo */ comment.
Jan 29 2018
Jan 28 2018
I saw once, that the OLD behavior may cause other warnings. However, I cannot reproduce this. Since I am not able to push currently, I will observe this and push later (to branch 5.0).
Reverted last change - I am fighting with arc :(
Corrected build location of checkprinting plugin
Jan 10 2018
From my point of view, the only issue left is the line QCoreApplication::addLibraryPath(QCoreApplication::applicationDirPath() + QDir::separator() + "kmymoney-plugins");. Since this seems to be a highly emotional question so I do not think further discussions are of use here.
Jan 9 2018
Jan 7 2018
Jan 5 2018
Jan 3 2018
Dec 3 2017
Hi! Did you do a performance comparison? I am concerned that this patch can reduce performance significantly because some of the objects are created very often and in huge amounts. Additionally we perform searches with them. Also I am not sure if these very simple objects (often POD) can really benefit from a d-pointer, except ABI stability. The drawback is increased code complexity.
Nov 5 2017
Wow, this is an impressive huge amount of work.
Oct 29 2017
Sep 17 2017
Jun 29 2017
Jun 26 2017
I am with Łukasz. If all developers can compile KMyMoney I am all in. I would not worry about the distros because until we can release this version this won't be an issue anymore.
Jun 19 2017
Your changes improve the code a lot! They make this code better in so many ways, thank you!
Jun 1 2017
May 31 2017
Hello Lukasz,
Apr 29 2017
Apr 26 2017
Apr 23 2017
Apr 18 2017
Feb 18 2017
Feb 17 2017
Well this is not very convincing to me, whatever.
I am just curious, why should an app for private finance should have a logarithmic axis? This does not sound useful to me.