[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: [Patch] new feature - Folder properties dialog now displays
From:       Tim Edwards <tkedwards () optusnet ! com ! au>
Date:       2004-05-09 8:51:45
Message-ID: 1084092704.31609.0.camel () coolserver
[Download RAW message or body]

That's been fixed - I must have done it when I was fiddling around
trying to get it to work and forgotten to change it back.

See http://bugs.kde.org/show_bug.cgi?id=81127

On Sun, 2004-05-09 at 17:44, Michael Pyne wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On Saturday 08 May 2004 04:11, Christian Loose wrote:
> > 3) The following changes in KDirSize::slotEntries() look useless too me:
> >
> > -    for (; it != end; ++it) {
> > +    for (; it != end; it++) {
> >
> > and
> >
> > -        for( ; it2 != (*it).end(); it2++ ) {
> > +        for( ; it2 != (*it).end(); ++it2 ) {
> 
> Actually, now that I think about it, those changes are worse than useless.  
> Changing the ++it form to a it++ form is less efficient, since the compiler 
> must copy the iterator into a temporary, increment the iterator, and return 
> the temporary.  It's not a horrible performance hit by any means, but it bad 
> form for C++ iterators.
> 
> Regards,
>  - Michael Pyne
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.2.3 (GNU/Linux)
> 
> iD8DBQFAneFzqjQYp5Omm0oRAjWaAJ94kV8blM81Fei0FEWUpY6kLCFPegCfR9Ir
> jNbCySPhOKYPXrB+OKRTxjk=
> =3//M
> -----END PGP SIGNATURE-----
>  
> >> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]