* Bug #14367 fixed - edit_curv failed opening due to a []+1 operation 43/18343/5
Samuel GOUGEON [Sat, 2 Jul 2016 13:10:10 +0000 (15:10 +0200)]
 Simplyfying code, old fix was @ https://codereview.scilab.org/18145

Change-Id: I1e2d1659f4b7809429da461a78c247ca931cc342

scilab/CHANGES.md
scilab/modules/graphics/macros/edit_curv.sci

index 6946f20..efa6489 100644 (file)
@@ -236,6 +236,7 @@ Bug Fixes
 * [Bug #13597](http://bugzilla.scilab.org/show_bug.cgi?id=13597) fixed - `help format` claimed setting a number of digits instead of characters
 * [Bug #13757](http://bugzilla.scilab.org/show_bug.cgi?id=13757) fixed - The toolbox menu dit not load properly ATOMS modules not autoloaded.
 * [Bug #14192](http://bugzilla.scilab.org/show_bug.cgi?id=14192) fixed - `g_margin` error-ed for double integrator.
+* [Bug #14367](http://bugzilla.scilab.org/show_bug.cgi?id=14367) fixed - `edit_curv` failed opening due to a `[]+1` operation.
 * [Bug #14448](http://bugzilla.scilab.org/show_bug.cgi?id=14448) fixed - removed havewindow() was still documented
 * [Bug #14557](http://bugzilla.scilab.org/show_bug.cgi?id=14557) fixed - `csim` failed when the system has no state.
 * [Bug #14586](http://bugzilla.scilab.org/show_bug.cgi?id=14586) fixed - Xcos simulation stop button did not work.
index 2dc70eb..7e6346f 100644 (file)
@@ -109,13 +109,7 @@ function [x,y,ok,gc]=edit_curv(x,y,job,tit,gc)
     w="menus(3)(";rpar=")"
     Data=w(ones(menu_d))+string(1:size(menu_d,"*"))+rpar(ones(menu_d))
 
-    wins = winsid();
-    if ~isempty(wins) then
-        curwin = max(winsid())+1;
-    else
-        curwin = 1;
-    end
-    scf(curwin);
+    curwin = scf().figure_id
 
     // Disable the menus and toolbars
     toolbar(curwin,"off");