tags 447206 +patch
thanks

To me it looks like sloppy programming style. The qMin/qMax templates
are only defined for arguments of the same type, but the second argument
in tis case is a (non-const) double. The appended patch fixes the build
failure in VectorCurve.cpp on mips.


Thiemo


--- qtiplot/src/VectorCurve.cpp.old     2007-11-06 00:20:15.000000000 +0000
+++ qtiplot/src/VectorCurve.cpp 2007-11-06 00:22:34.000000000 +0000
@@ -251,27 +251,27 @@ else
        switch(d_position)
                {
                case Tail:
-                       rect.setTop(QMIN(rect.top(), 
rect.top()+mag*sin(angle)));
-                       rect.setBottom(QMAX(rect.bottom(), 
rect.bottom()+mag*sin(angle)));
-                       rect.setLeft(QMIN(rect.left(), 
rect.left()+mag*cos(angle)));
-                       rect.setRight(QMAX(rect.right(), 
rect.right()+mag*cos(angle)));
+                       rect.setTop(QMIN(rect.top(), rect.top()+(const 
float)(mag*sin(angle))));
+                       rect.setBottom(QMAX(rect.bottom(), rect.bottom()+(const 
float)(mag*sin(angle))));
+                       rect.setLeft(QMIN(rect.left(), rect.left()+(const 
float)(mag*cos(angle))));
+                       rect.setRight(QMAX(rect.right(), rect.right()+(const 
float)(mag*cos(angle))));
                break;
 
                case Middle:
                        {
                        mag *= 0.5;
-                       rect.setTop(QMIN(rect.top(), rect.top() - 
fabs(mag*sin(angle))));
-                       rect.setBottom(QMAX(rect.bottom(), rect.bottom() + 
fabs(mag*sin(angle))));
-                       rect.setLeft(QMIN(rect.left(), rect.left() - 
fabs(mag*cos(angle))));
-                       rect.setRight(QMAX(rect.right(), rect.right() + 
fabs(mag*cos(angle))));
+                       rect.setTop(QMIN(rect.top(), rect.top() - (const 
float)fabs(mag*sin(angle))));
+                       rect.setBottom(QMAX(rect.bottom(), rect.bottom() + 
(const float)fabs(mag*sin(angle))));
+                       rect.setLeft(QMIN(rect.left(), rect.left() - (const 
float)fabs(mag*cos(angle))));
+                       rect.setRight(QMAX(rect.right(), rect.right() + (const 
float)fabs(mag*cos(angle))));
                        }
                break;
 
                case Head:
-                       rect.setTop(QMIN(rect.top(), rect.top() - 
mag*sin(angle)));
-                       rect.setBottom(QMAX(rect.bottom(), rect.bottom() - 
mag*sin(angle)));
-                       rect.setLeft(QMIN(rect.left(), rect.left() - 
mag*cos(angle)));
-                       rect.setRight(QMAX(rect.right(), rect.right() - 
mag*cos(angle)));
+                       rect.setTop(QMIN(rect.top(), rect.top() - (const 
float)(mag*sin(angle))));
+                       rect.setBottom(QMAX(rect.bottom(), rect.bottom() - 
(const float)(mag*sin(angle))));
+                       rect.setLeft(QMIN(rect.left(), rect.left() - (const 
float)(mag*cos(angle))));
+                       rect.setRight(QMAX(rect.right(), rect.right() - (const 
float)(mag*cos(angle))));
                break;
                }
        }



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to