Remove the need for explicit <= 0, >= 360 etc handling by using % 360.

1 files changed, 4 insertions, 11 deletions

diff --git a/core/applets/rotateapplet/rotate.cpp b/core/applets/rotateapplet/rotate.cpp
index 4743388..3a5a3c9 100644
--- a/core/applets/rotateapplet/rotate.cpp
+++ b/core/applets/rotateapplet/rotate.cpp
@@ -96,40 +96,33 @@ void RotateApplet::activated ( )
 
     Config cfg( "qpe" );
     cfg.setGroup( "Appearance" );
 
     // 0 -> 90° clockwise, 1 -> 90° counterclockwise
     bool rotDirection = cfg.readBoolEntry( "rotatedir" );
 
     // hide inputs methods before rotation
     QCopEnvelope en( "QPE/TaskBar", "hideInputMethod()" );
 
     if ( m_flipped )  {
         if ( rotDirection  )  {
-            newRotation = currentRotation - 90;
-            if (newRotation <=0) newRotation = 270;
-	//ipaqs like the 36xx have the display rotated
-	// to 270 as default, and -90 is invalid => handle this here
+            newRotation = ( currentRotation + 270 ) % 360;
         } else {
-            newRotation = currentRotation + 90;
-            if(newRotation >= 360) newRotation = 0;	//ipaqs like the 36xx have the display
-            //rotated to 270 as default, so 360 does nothing => handle this here
+            newRotation = ( currentRotation + 90 ) % 360;
         }
     } else {
         if ( rotDirection )  {
-            newRotation = currentRotation + 90;
-             if(newRotation >= 360) newRotation = 0;
+            newRotation = ( currentRotation + 90 ) % 360;
         } else {
-            newRotation = currentRotation - 90;
-            if (newRotation <=0) newRotation = 270;
+            newRotation = ( currentRotation + 270 ) % 360;
          }
     }
     QCopEnvelope env( "QPE/System", "setCurrentRotation(int)" );
     env << newRotation;
     QCopEnvelope env2( "QPE/System", "setDefaultRotation(int)" );
     env2 << newRotation;
 
     m_flipped = !m_flipped;
 }
 
 
 QRESULT RotateApplet::queryInterface ( const QUuid &uuid, QUnknownInterface **iface )