[Kst] extragear/graphics/kst/kst

Matthew D Truch matt at truch.net
Wed Nov 23 01:05:47 CET 2005


On Tue, Nov 22, 2005 at 11:37:26PM +0000, George Staikos wrote:
> SVN commit 482453 by staikos:
> --- trunk/extragear/graphics/kst/kst/kstviewobject.cpp #482452:482453
> @@ -674,10 +673,7 @@
>      }
>  
>      for (KstViewObjectList::Iterator it = childrenCopy.begin(); it != childrenCopy.end(); ++it) {
> -      distance  = ceil( (double)rows * 2.0 * (*it)->aspectRatio().y );
> -      distance  = rows * d2i( distance );
> -      distance += (*it)->aspectRatio().x * rows;
> -      distance += (*it)->aspectRatio().y;
> +      double distance = ceil(double(rows) * 2.0 * (*it)->aspectRatio().y) + rows * d2i(distance) + (*it)->aspectRatio().x * rows + (*it)->aspectRatio().y;
>  
>        if (it == childrenCopy.begin() || distance < minDistance) {
>          minDistance = distance;

I don't understand this commit.  Since the second deleted line didn't
have a += but actually had an =, isn't distance (the argument to d2i)
undefined in the line you added?  Am I missing some kind of voodoo
recursive definition thing here?

-- 
"All your base are belong to us!"
--------------------------
Matthew Truch
Department of Physics
Brown University
matt at truch.net
http://matt.truch.net/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kst/attachments/20051123/4b7dc016/attachment.pgp


More information about the Kst mailing list