[prev in list] [next in list] [prev in thread] [next in thread]
List: koffice-devel
Subject: kpresenter patch
From: Thorsten Zachmann <t.zachmann () zagge ! de>
Date: 2003-08-31 10:22:22
[Download RAW message or body]
Hello all
I found some problems with flipping pie objects. ( no flipping when angle =
180° || 270° || 360°, after 2 time horizontal flipping of a pie with angle
45° and length 170° no more flipping possible).
The attached patch solves the problems and simpifies the code.
Is it ok to commit?
Thorsten
["patch48" (text/x-diff)]
Index: kppieobject.cc
===================================================================
RCS file: /home/kde/koffice/kpresenter/kppieobject.cc,v
retrieving revision 1.42
diff -u -3 -p -r1.42 kppieobject.cc
--- kppieobject.cc 27 Aug 2003 14:45:39 -0000 1.42
+++ kppieobject.cc 31 Aug 2003 10:16:08 -0000
@@ -167,24 +167,15 @@ void KPPieObject::flip( bool horizontal
KP2DObject::flip( horizontal );
if ( horizontal )
{
- if ( p_angle <= 90*16 )
- p_angle = (360*16 - p_angle -p_len);
- else if ( p_angle >90*16 && p_angle <180*16 )
- p_angle = (p_angle + 90*16 + p_len) ;
- else if ( p_angle >180*16 && p_angle <270*16 )
- p_angle = (360*16 - (p_angle + p_len))%(360*16);
- else if ( p_angle >270*16 && p_angle <360*16 )
- p_angle = (360*16-p_angle -p_len)%(360*16);
+ p_angle = 360*16 - p_angle -p_len;
}
else
{
- if ( p_angle <= 90*16 )
- p_angle = 180*16- p_angle - p_len;
- else if ( p_angle >90*16 && p_angle <180*16 )
- p_angle = 180*16 - p_angle - p_len ;
- else if ( p_angle >180*16 && p_angle <270*16 )
- p_angle = 360*16 - (p_angle - 180*16) - p_len;
- else if ( p_angle >270*16 && p_angle <360*16 )
- p_angle = 180*16+ (360*16 - (p_angle+p_len));
+ p_angle = 180*16 - p_angle - p_len;
+ }
+
+ // angle smaller 0°
+ while ( p_angle < 0 ) {
+ p_angle += 360*16;
}
}
_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
http://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic