[Kmymoney-devel] playground/office/kmymoney/kmymoney2

[Fernando Vilas]

On Wednesday 28 October 2009 03:47:33 Cristian Oneţ wrote:
> On Wed, Oct 28, 2009 at 10:38 AM, Thomas Baumgart <thb at net-bembel.de> wrote:
> > On Wednesday 28 October 2009 08:53:37 Cristian Oneţ wrote:
> >> On Wed, Oct 28, 2009 at 4:29 AM, Alvaro Soliverez <asoliverez at gmail.com>
> >
> > wrote:
> >> > On Tue, Oct 27, 2009 at 7:17 PM, Cristian Oneţ
> >> > <onet.cristian at gmail.com>
[snip]
> >> >
> >> > Ok, then. I agree that it's not necessary to have a whole ui class
> >> > just for that.
> >> > BTW, just looking at the commits yesterday I realized what a great
> >> > team we have. Cristian ported the methods to detect multiple
> >> > instances, then Fernando fixed some iterators that caused trouble
> >> > after that. I was trying to compile in the meantime, so Fernando's fix
> >> > did the trick. I just want to say that I'm sure that with this kind of
> >> > joint effort, we are on the right track to build a great application.
> >> > :)
> >>
> >> My sandbox didn't have the check strict iterators Qt flag (don't know
> >> why I haven't updated before the commit) so thanks to Fernando for the
> >> fix :).
> >
> > stupid question: where do I find this check? How do I activate it?
> 
> It was added by Laurent in revision 1040342.
> 
> Here is the diff:
[snip diff]

There were a lot of fixes committed recently that fixed iterators (by Christian 
and Laurent, I think). When this flag was finally activated, my bit was fairly 
small. It was the other folks who did the heavy lifting there.

I really like having this flag, and I would like to find some others like it. 
It's always better to have something break at compile time vs run time or 
worse, a subtle bug that gives bad data but doesn't crash.

-- 
Thanks,
Fernando Vilas
fvilas at iname.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/kmymoney-devel/attachments/20091028/698052e8/attachment.sig