Graphics GetCommandArg macros removed 45/18345/5
Dhruv Khattar [Sat, 2 Jul 2016 20:04:30 +0000 (01:04 +0530)]
 * Expliciting these macros will enable dealing with the leaks provoked when they errored out

Change-Id: Ia0cad5627cb5ef08b2c314ce89438dfd62719ee0

scilab/modules/graphics/includes/GetCommandArg.h
scilab/modules/graphics/sci_gateway/c/sci_champ.c
scilab/modules/graphics/sci_gateway/c/sci_fec.c
scilab/modules/graphics/sci_gateway/c/sci_grayplot.c
scilab/modules/graphics/sci_gateway/c/sci_param3d.c
scilab/modules/graphics/sci_gateway/c/sci_param3d1.c
scilab/modules/graphics/sci_gateway/c/sci_plot2d.c
scilab/modules/graphics/sci_gateway/c/sci_plot2d1.c
scilab/modules/graphics/sci_gateway/c/sci_plot3d.c

index 972313f..5791234 100644 (file)
@@ -175,20 +175,5 @@ GRAPHICS_IMPEXP int get_optional_double_arg(void* _pvCtx, char * fname, int pos,
  * @return <ReturnValue>
  */
 GRAPHICS_IMPEXP int get_optional_int_arg(void* _pvCtx, char * fname, int pos, char * name, int ** value, int sz, rhs_opts opts[]);
-/*------------------------------------------------------------------------*/
-#define sciGetStyle(PVCTX,fname,pos,n1,opts,style) { if (get_style_arg(PVCTX,fname,pos,n1,opts,style) == 0) { return 0 ; } }
-#define GetRect(PVCTX,fname,pos,opts,rect) { if (get_rect_arg(PVCTX,fname,pos,opts,rect) == 0) { return 0 ; } }
-#define GetStrf(PVCTX,fname,pos,opts,strf) { if (get_strf_arg(PVCTX,fname,pos,opts,strf) == 0) { return 0 ; } }
-#define GetLegend(PVCTX,fname,pos,opts,legend) { if (get_legend_arg(PVCTX,fname,pos,opts,legend) == 0) { return 0 ; } }
-#define GetLabels(PVCTX,fname,pos,opts,labels) { if (get_labels_arg(PVCTX,fname,pos,opts,labels) == 0) { return 0 ; } }
-#define GetNax(PVCTX,pos,opts,nax,flagNax) { if (get_nax_arg(PVCTX,pos,opts,nax,flagNax)==0) { return 0 ; } }
-#define GetZminmax(PVCTX,fname,pos,opts,zminmax) { if (get_zminmax_arg(PVCTX,fname,pos,opts,zminmax) == 0) { return 0 ; } }
-#define GetColminmax(PVCTX,fname,pos,opts,colminmax) { if (get_colminmax_arg(PVCTX,fname,pos,opts,colminmax)==0) { return 0 ; } }
-#define GetColOut(PVCTX,fname,pos,opts,colout) { if (get_colout_arg(PVCTX,fname,pos,opts,colout)==0) { return 0 ; } }
-#define GetWithMesh(PVCTX,fname,pos,opts,withMesh) { if (get_with_mesh_arg(PVCTX,fname,pos,opts,withMesh)==0) { return 0 ; } }
-#define GetLogflags(PVCTX, fname,pos,opts,logFlags) { if (get_logflags_arg(PVCTX, fname,pos,opts,logFlags) == 0) { return 0 ; } }
-#define GetOptionalDoubleArg(PVCTX,fname,pos,name,value,sz,opts) { if (get_optional_double_arg(PVCTX,fname,pos,name,value,sz,opts) == 0) { return 0 ; } }
-#define GetOptionalIntArg(PVCTX,fname,pos,name,value,sz,opts) { if (get_optional_int_arg(PVCTX,fname,pos,name,value,sz,opts) == 0) { return 0 ; } }
-/*------------------------------------------------------------------------*/
 
 #endif /* _GET_COMMAND_ARG_H_ */
index b5aa066..f4312b9 100644 (file)
@@ -190,9 +190,18 @@ int sci_champ_G(char *fname,
         return 0;
     }
 
-    GetOptionalDoubleArg(pvApiCtx, fname, 5, "arfact", &arfact, 1, opts);
-    GetRect(pvApiCtx, fname, 6, opts, &rect);
-    GetStrf(pvApiCtx, fname, 7, opts, &strf);
+    if (get_optional_double_arg(pvApiCtx, fname, 5, "arfact", &arfact, 1, opts) == 0)
+    {
+        return 0;
+    }
+    if (get_rect_arg(pvApiCtx, fname, 6, opts, &rect) == 0)
+    {
+        return 0;
+    }
+    if (get_strf_arg(pvApiCtx, fname, 7, opts, &strf) == 0)
+    {
+        return 0;
+    }
 
     getOrCreateDefaultSubwin();
 
index 90cb86c..0a721cc 100644 (file)
@@ -181,14 +181,38 @@ int sci_fec(char *fname, void *pvApiCtx)
         return 0;
     }
 
-    GetStrf(pvApiCtx, fname, 5, opts, &strf);
-    GetLegend(pvApiCtx, fname, 6, opts, &legend);
-    GetRect(pvApiCtx, fname, 7, opts, &rect);
-    GetNax(pvApiCtx, 8, opts, &nax, &flagNax);
-    GetZminmax(pvApiCtx, fname, 9, opts, &zminmax);
-    GetColminmax(pvApiCtx, fname, 10, opts, &colminmax);
-    GetColOut(pvApiCtx, fname, 11, opts, &colOut);
-    GetWithMesh(pvApiCtx, fname, 12, opts, &withMesh);
+    if (get_strf_arg(pvApiCtx, fname, 5, opts, &strf) == 0)
+    {
+        return 0;
+    }
+    if (get_legend_arg(pvApiCtx, fname, 6, opts, &legend) == 0)
+    {
+        return 0;
+    }
+    if (get_rect_arg(pvApiCtx, fname, 7, opts, &rect) == 0)
+    {
+        return 0;
+    }
+    if (get_nax_arg(pvApiCtx, 8, opts, &nax, &flagNax)==0)
+    {
+        return 0;
+    }
+    if (get_zminmax_arg(pvApiCtx, fname, 9, opts, &zminmax) == 0)
+    {
+        return 0;
+    }
+    if (get_colminmax_arg(pvApiCtx, fname, 10, opts, &colminmax)==0)
+    {
+        return 0;
+    }
+    if (get_colout_arg(pvApiCtx, fname, 11, opts, &colOut)==0)
+    {
+        return 0;
+    }
+    if (get_with_mesh_arg(pvApiCtx, fname, 12, opts, &withMesh)==0)
+    {
+        return 0;
+    }
 
     getOrCreateDefaultSubwin();
 
index 626d197..96744db 100644 (file)
@@ -170,10 +170,22 @@ int sci_grayplot(char *fname, void *pvApiCtx)
     }
 
 
-    GetStrf(pvApiCtx, fname, 4, opts, &strf);
-    GetRect(pvApiCtx, fname, 5, opts, &rect);
-    GetNax(pvApiCtx, 6, opts, &nax, &flagNax);
-    GetLogflags(pvApiCtx, fname, 7, opts, &logFlags);
+    if (get_strf_arg(pvApiCtx, fname, 4, opts, &strf) == 0)
+    {
+        return 0;
+    }
+    if (get_rect_arg(pvApiCtx, fname, 5, opts, &rect) == 0)
+    {
+        return 0;
+    }
+    if (get_nax_arg(pvApiCtx, 6, opts, &nax, &flagNax)==0)
+    {
+        return 0;
+    }
+    if (get_logflags_arg(pvApiCtx, fname, 7, opts, &logFlags) == 0)
+    {
+        return 0;
+    }
 
     getOrCreateDefaultSubwin();
 
@@ -187,12 +199,18 @@ int sci_grayplot(char *fname, void *pvApiCtx)
             strfl[1] = '7';
         }
 
-        GetOptionalIntArg(pvApiCtx, fname, 7, "frameflag", &frame, 1, opts);
+        if (get_optional_int_arg(pvApiCtx, fname, 7, "frameflag", &frame, 1, opts) == 0)
+        {
+            return 0;
+        }
         if (frame != &frame_def)
         {
             strfl[1] = (char)(*frame + 48);
         }
-        GetOptionalIntArg(pvApiCtx, fname, 7, "axesflag", &axes, 1, opts);
+        if (get_optional_int_arg(pvApiCtx, fname, 7, "axesflag", &axes, 1, opts) == 0)
+        {
+            return 0;
+        }
         if (axes != &axes_def)
         {
             strfl[2] = (char)(*axes + 48);
index 03bc36f..8feded7 100644 (file)
@@ -151,17 +151,32 @@ int sci_param3d(char * fname, void *pvApiCtx)
     }
 
 
-    GetOptionalDoubleArg(pvApiCtx, fname, 4, "theta", &theta, 1, opts);
-    GetOptionalDoubleArg(pvApiCtx, fname, 5, "alpha", &alpha, 1, opts);
-    GetLabels(pvApiCtx, fname, 6, opts, &labels);
+    if (get_optional_double_arg(pvApiCtx, fname, 4, "theta", &theta, 1, opts) == 0)
+    {
+        return 0;
+    }
+    if (get_optional_double_arg(pvApiCtx, fname, 5, "alpha", &alpha, 1, opts) == 0)
+    {
+        return 0;
+    }
+    if (get_labels_arg(pvApiCtx, fname, 6, opts, &labels) == 0)
+    {
+        return 0;
+    }
 
     iflag_def[1] = 8;
     ifl = &(iflag_def[1]);
-    GetOptionalIntArg(pvApiCtx, fname, 7, "flag", &ifl, 2, opts);
+    if (get_optional_int_arg(pvApiCtx, fname, 7, "flag", &ifl, 2, opts) == 0)
+    {
+        return 0;
+    }
     iflag[0] = iflag_def[0];
     iflag[1] = ifl[0];
     iflag[2] = ifl[1];
-    GetOptionalDoubleArg(pvApiCtx, fname, 8, "ebox", &ebox, 6, opts);
+    if (get_optional_double_arg(pvApiCtx, fname, 8, "ebox", &ebox, 6, opts) == 0)
+    {
+        return 0;
+    }
 
     getOrCreateDefaultSubwin();
 
index 58db6cb..af60708 100644 (file)
@@ -240,17 +240,32 @@ int sci_param3d1(char *fname, void *pvApiCtx)
     }
 
 
-    GetOptionalDoubleArg(pvApiCtx, fname, 4, "theta", &theta, 1, opts);
-    GetOptionalDoubleArg(pvApiCtx, fname, 5, "alpha", &alpha, 1, opts);
-    GetLabels(pvApiCtx, fname, 6, opts, &labels);
+    if (get_optional_double_arg(pvApiCtx, fname, 4, "theta", &theta, 1, opts) == 0)
+    {
+        return 0;
+    }
+    if (get_optional_double_arg(pvApiCtx, fname, 5, "alpha", &alpha, 1, opts) == 0)
+    {
+        return 0;
+    }
+    if (get_labels_arg(pvApiCtx, fname, 6, opts, &labels) == 0)
+    {
+        return 0;
+    }
     iflag_def[1] = 8;
     ifl = &(iflag_def[1]);
-    GetOptionalIntArg(pvApiCtx, fname, 7, "flag", &ifl, 2, opts);
+    if (get_optional_int_arg(pvApiCtx, fname, 7, "flag", &ifl, 2, opts) == 0)
+    {
+        return 0;
+    }
     iflag[0] = iflag_def[0];
     iflag[1] = ifl[0];
     iflag[2] = ifl[1];
 
-    GetOptionalDoubleArg(pvApiCtx, fname, 8, "ebox", &ebox, 6, opts);
+    if (get_optional_double_arg(pvApiCtx, fname, 8, "ebox", &ebox, 6, opts) == 0)
+    {
+        return 0;
+    }
 
     if (m1 == 1 && n1 > 1)
     {
index bcfb322..35adba4 100644 (file)
@@ -98,7 +98,10 @@ int sci_plot2d(char* fname, void *pvApiCtx)
     if (checkInputArgumentType(pvApiCtx, 1, sci_strings))
     {
         /* logflags */
-        GetLogflags(pvApiCtx, fname, 1, opts, &logFlags);
+        if (get_logflags_arg(pvApiCtx, fname, 1, opts, &logFlags) == 0)
+        {
+            return 0;
+        }
         iskip = 1;
     }
 
@@ -339,15 +342,33 @@ int sci_plot2d(char* fname, void *pvApiCtx)
         return 0;
     }
 
-    sciGetStyle(pvApiCtx, fname, 3 + iskip, n1, opts, &style);
-    GetStrf(pvApiCtx, fname, 4 + iskip, opts, &strf);
-    GetLegend(pvApiCtx, fname, 5 + iskip, opts, &legend);
-    GetRect(pvApiCtx, fname, 6 + iskip, opts, &rect);
-    GetNax(pvApiCtx, 7 + iskip, opts, &nax, &flagNax);
+    if (get_style_arg(pvApiCtx, fname, 3 + iskip, n1, opts, &style) == 0)
+    {
+        return 0;
+    }
+    if (get_strf_arg(pvApiCtx, fname, 4 + iskip, opts, &strf) == 0)
+    {
+        return 0;
+    }
+    if (get_legend_arg(pvApiCtx, fname, 5 + iskip, opts, &legend) == 0)
+    {
+        return 0;
+    }
+    if (get_rect_arg(pvApiCtx, fname, 6 + iskip, opts, &rect) == 0)
+    {
+        return 0;
+    }
+    if (get_nax_arg(pvApiCtx, 7 + iskip, opts, &nax, &flagNax)==0)
+    {
+        return 0;
+    }
 
     if (iskip == 0)
     {
-        GetLogflags(pvApiCtx, fname, 8, opts, &logFlags);
+        if (get_logflags_arg(pvApiCtx, fname, 8, opts, &logFlags) == 0)
+        {
+            return 0;
+        }
     }
 
     freeStrf = !isDefStrf(strf);
@@ -377,7 +398,10 @@ int sci_plot2d(char* fname, void *pvApiCtx)
             strfl[0] = '1';
         }
 
-        GetOptionalIntArg(pvApiCtx, fname, 9, "frameflag", &frame, 1, opts);
+        if (get_optional_int_arg(pvApiCtx, fname, 9, "frameflag", &frame, 1, opts) == 0)
+        {
+            return 0;
+        }
         if (frame != &frame_def)
         {
             if (*frame >= 0 && *frame <= 8)
@@ -395,7 +419,10 @@ int sci_plot2d(char* fname, void *pvApiCtx)
             }
         }
 
-        GetOptionalIntArg(pvApiCtx, fname, 9, "axesflag", &axes, 1, opts);
+        if (get_optional_int_arg(pvApiCtx, fname, 9, "axesflag", &axes, 1, opts) == 0)
+        {
+            return 0;
+        }
         if (axes != &axes_def)
         {
             if ((*axes >= 0 && *axes <= 5) || *axes == 9)
index a7415ec..c478ed5 100644 (file)
@@ -101,7 +101,10 @@ int sci_plot2d1_G(char * fname, int ptype, void *pvApiCtx)
     if (checkInputArgumentType(pvApiCtx, 1, sci_strings))
     {
         /* logflags */
-        GetLogflags(pvApiCtx, fname, 1, opts, &logFlags);
+        if (get_logflags_arg(pvApiCtx, fname, 1, opts, &logFlags) == 0)
+        {
+            return 0;
+        }
         iskip = 1;
     }
 
@@ -309,14 +312,33 @@ int sci_plot2d1_G(char * fname, int ptype, void *pvApiCtx)
         }
     }
 
-    sciGetStyle(pvApiCtx, fname, 3 + iskip, n1, opts, &style);
-    GetStrf(pvApiCtx, fname, 4 + iskip, opts, &strf);
-    GetLegend(pvApiCtx, fname, 5 + iskip, opts, &legend);
-    GetRect(pvApiCtx, fname, 6 + iskip, opts, &rect);
-    GetNax(pvApiCtx, 7 + iskip, opts, &nax, &flagNax);
+    if (get_style_arg(pvApiCtx, fname, 3 + iskip, n1, opts, &style) == 0)
+    {
+        return 0;
+    }
+    if (get_strf_arg(pvApiCtx, fname, 4 + iskip, opts, &strf) == 0)
+    {
+        return 0;
+    }
+    if (get_legend_arg(pvApiCtx, fname, 5 + iskip, opts, &legend) == 0)
+    {
+        return 0;
+    }
+    if (get_rect_arg(pvApiCtx, fname, 6 + iskip, opts, &rect) == 0)
+    {
+        return 0;
+    }
+    if (get_nax_arg(pvApiCtx, 7 + iskip, opts, &nax, &flagNax)==0)
+    {
+        return 0;
+    }
+
     if (iskip == 0)
     {
-        GetLogflags(pvApiCtx, fname, 8, opts, &logFlags);
+        if (get_logflags_arg(pvApiCtx, fname, 8, opts, &logFlags) == 0)
+        {
+            return 0;
+        }
     }
 
     if (isDefStrf(strf))
@@ -332,12 +354,18 @@ int sci_plot2d1_G(char * fname, int ptype, void *pvApiCtx)
         {
             strfl[0] = '1';
         }
-        GetOptionalIntArg(pvApiCtx, fname, 9, "frameflag", &frame, 1, opts);
+        if (get_optional_int_arg(pvApiCtx, fname, 9, "frameflag", &frame, 1, opts) == 0)
+        {
+            return 0;
+        }
         if (frame != &frame_def)
         {
             strfl[1] = (char)(*frame + 48);
         }
-        GetOptionalIntArg(pvApiCtx, fname, 9, "axesflag", &axes, 1, opts);
+        if (get_optional_int_arg(pvApiCtx, fname, 9, "axesflag", &axes, 1, opts) == 0)
+        {
+            return 0;
+        }
         if (axes != &axes_def)
         {
             strfl[2] = (char)(*axes + 48);
index 9a0af0f..7f25253 100644 (file)
@@ -322,11 +322,26 @@ int sci_plot3d(char * fname, void *pvApiCtx)
 
     iflag_def[1] = 8;
 
-    GetOptionalDoubleArg(pvApiCtx, fname, 4, "theta", &theta, 1, opts);
-    GetOptionalDoubleArg(pvApiCtx, fname, 5, "alpha", &alpha, 1, opts);
-    GetLabels(pvApiCtx, fname, 6, opts, &legend);
-    GetOptionalIntArg(pvApiCtx, fname, 7, "flag", &iflag, 3, opts);
-    GetOptionalDoubleArg(pvApiCtx, fname, 8, "ebox", &ebox, 6, opts);
+    if (get_optional_double_arg(pvApiCtx, fname, 4, "theta", &theta, 1, opts) == 0)
+    {
+        return 0;
+    }
+    if (get_optional_double_arg(pvApiCtx, fname, 5, "alpha", &alpha, 1, opts) == 0)
+    {
+        return 0;
+    }
+    if (get_labels_arg(pvApiCtx, fname, 6, opts, &legend) == 0)
+    {
+        return 0;
+    }
+    if (get_optional_int_arg(pvApiCtx, fname, 7, "flag", &iflag, 3, opts) == 0)
+    {
+        return 0;
+    }
+    if (get_optional_double_arg(pvApiCtx, fname, 8, "ebox", &ebox, 6, opts) == 0)
+    {
+        return 0;
+    }
 
 
     getOrCreateDefaultSubwin();