« Return to Thread: patch to fix iterators in khomeview (for kde4 version)

Re: patch to fix iterators in khomeview (for kde4 version)

by Thomas Baumgart :: Rate this Message:

Reply to Author | View in Thread

On Tuesday 30 June 2009 03:33:47 Alvaro Soliverez wrote:

> Ok. I checked the patch. I had noticed the error but I had not figured out
> why it was happening. Nice catch!

Does this also apply for the Qt3 version? I am not sure, why it should fail on
Qt3. Can someone literate explain?

> We will have to check other similar iterators for the same problems.
> Perhaps that's a problem in the Qt iterator template?
>
> The only comment I have about the patch is that the boolean should be
> called removeAccount, we don't use Hungarian notation in KMM. The rest is
> fine.

--

Regards

Thomas Baumgart

GPG-FP: E55E D592 F45F 116B 8429   4F99 9C59 DB40 B75D D3BA
-------------------------------------------------------------
C makes it easy for you to shoot yourself in the foot. C++ makes that
harder, but when you do, it blows away your whole leg.
 -- Bjarne Stroustrup
-------------------------------------------------------------



------------------------------------------------------------------------------

_______________________________________________
KMyMoney2-developer mailing list
KMyMoney2-developer@...
https://lists.sourceforge.net/lists/listinfo/kmymoney2-developer

signature.asc (232 bytes) Download Attachment

 « Return to Thread: patch to fix iterators in khomeview (for kde4 version)