Remove some useless code 37/10737/2
Sylvestre Ledru [Thu, 7 Mar 2013 18:16:45 +0000 (19:16 +0100)]
Change-Id: I4937a9ccbc0a55effa9dc9bcc3281bd59fb25ce9

scilab/modules/graphics/sci_gateway/c/sci_plot2d.c
scilab/modules/graphics/sci_gateway/c/sci_rubberbox.c
scilab/modules/graphics/sci_gateway/c/sci_xgetmouse.c

index aacab57..a1d1f73 100644 (file)
@@ -41,7 +41,6 @@ int sci_plot2d( char * fname, unsigned long fname_len )
 
     /* F.Leray 18.05.04 : log. case test*/
     int size_x = 0, size_y = 0;
-    double xd[2];
     char dataflag = 0;
 
     char   * logFlags = NULL  ;
@@ -291,8 +290,6 @@ int sci_plot2d( char * fname, unsigned long fname_len )
             switch ( dataflag )
             {
                 case 'e' :
-                    xd[0] = 1.0;
-                    xd[1] = (double)m1;
                     size_x = (m1 != 0) ? 2 : 0 ;
                     break;
                 case 'o' :
index 1ff3d47..700c517 100644 (file)
@@ -146,7 +146,6 @@ int sci_rubberbox(char * fname, unsigned long fname_len)
 
     /* [final_rect, btn] = rubberbox([initial_rect],[edition_mode]) */
 
-    int button = 0;
     double initialRect[4] = {0.0, 0.0, 0.0, 0.0};
     int initialRectSize = 0;
 
index a1d8168..2a270aa 100644 (file)
@@ -34,9 +34,9 @@
 #include "graphicObjectProperties.h"
 #include "getGraphicObjectProperty.h"
 /*--------------------------------------------------------------------------*/
-int sci_xgetmouse( char *fname,unsigned long fname_len )
+int sci_xgetmouse( char *fname, unsigned long fname_len )
 {
-    int  m1 = 1, n1 = 3, l1 = 0,l2 = 0;
+    int  m1 = 1, n1 = 3, l1 = 0, l2 = 0;
     int mouseButtonNumber = 0;
     int windowsID = 0;
     int sel[2], m = 0, n = 0;
@@ -47,40 +47,38 @@ int sci_xgetmouse( char *fname,unsigned long fname_len )
 
     int selPosition = 0;
 
-    int iFigureId = 0;
+    CheckRhs(0, 1);
+    CheckLhs(1, 2);
 
-    CheckRhs(0,1);
-    CheckLhs(1,2);
-
-    switch(Rhs)
+    switch (Rhs)
     {
-    case 1:
-        if (GetType(1)==sci_boolean)
-        {
-            selPosition = 1;
-        }
-        else
-        {
-            Scierror(999, _("%s: Wrong type for input argument #%d: Boolean vector expected.\n"), fname, 1);
-            return FALSE;
-        }
-        break;
-    default:
-        // Call Java xgetmouse
-        // No need to set any option.
-        break;
+        case 1:
+            if (GetType(1) == sci_boolean)
+            {
+                selPosition = 1;
+            }
+            else
+            {
+                Scierror(999, _("%s: Wrong type for input argument #%d: Boolean vector expected.\n"), fname, 1);
+                return FALSE;
+            }
+            break;
+        default:
+            // Call Java xgetmouse
+            // No need to set any option.
+            break;
     }
 
     // Select current figure or create it
     getOrCreateDefaultSubwin();
 
     // Call Java to get mouse information
-    if (selPosition!=0)
+    if (selPosition != 0)
     {
-        GetRhsVar(selPosition,MATRIX_OF_BOOLEAN_DATATYPE, &m, &n, &l1);
-        CheckDims(selPosition,m*n,1,2,1);
-        sel[0]=*istk(l1);
-        sel[1]=*istk(l1+1);
+        GetRhsVar(selPosition, MATRIX_OF_BOOLEAN_DATATYPE, &m, &n, &l1);
+        CheckDims(selPosition, m * n, 1, 2, 1);
+        sel[0] = *istk(l1);
+        sel[1] = *istk(l1 + 1);
 
         // Call Java xgetmouse
         CallJxgetmouseWithOptions(sel[0], sel[1]);
@@ -95,13 +93,13 @@ int sci_xgetmouse( char *fname,unsigned long fname_len )
     pixelCoords[0] = (int) getJxgetmouseXCoordinate();
     pixelCoords[1] = (int) getJxgetmouseYCoordinate();
 
-    CreateVar(Rhs+1,MATRIX_OF_DOUBLE_DATATYPE,&m1,&n1,&l1);
+    CreateVar(Rhs + 1, MATRIX_OF_DOUBLE_DATATYPE, &m1, &n1, &l1);
     // No need to calculate coordinates if callback or close is trapped
     if (mouseButtonNumber == -1000 || mouseButtonNumber == -2)
     {
         *stk(l1) = -1;
-        *stk(l1+1) = -1;
-        *stk(l1+2) = (double) mouseButtonNumber;
+        *stk(l1 + 1) = -1;
+        *stk(l1 + 2) = (double) mouseButtonNumber;
     }
     else
     {
@@ -111,24 +109,24 @@ int sci_xgetmouse( char *fname,unsigned long fname_len )
         sciGet2dViewCoordFromPixel(clickedSubwinUID, pixelCoords, userCoords2D);
 
         *stk(l1) = userCoords2D[0];
-        *stk(l1+1) = userCoords2D[1];
-        *stk(l1+2) = (double) mouseButtonNumber;
+        *stk(l1 + 1) = userCoords2D[1];
+        *stk(l1 + 2) = (double) mouseButtonNumber;
     }
-    LhsVar(1) = Rhs+1;
+    LhsVar(1) = Rhs + 1;
 
     switch (Lhs)
     {
-    case 1:
-               PutLhsVar();
-        return 0;
-    case 2:
-        CreateVar(Rhs+2,MATRIX_OF_DOUBLE_DATATYPE,&m1,&m1,&l2);
-        *stk(l2) = windowsID; /* this is the window number */
-        LhsVar(2) = Rhs+2;
-               PutLhsVar();
-        return 0;
+        case 1:
+            PutLhsVar();
+            return 0;
+        case 2:
+            CreateVar(Rhs + 2, MATRIX_OF_DOUBLE_DATATYPE, &m1, &m1, &l2);
+            *stk(l2) = windowsID; /* this is the window number */
+            LhsVar(2) = Rhs + 2;
+            PutLhsVar();
+            return 0;
     }
-       PutLhsVar();
+    PutLhsVar();
     return -1 ;
 }