<html><head><meta http-equiv="Content-Type" content="text/html; charset=UTF-8" /></head><body style='font-size: 10pt; font-family: Verdana,Geneva,sans-serif'>
<p>However, the findobj() fix that has been recently merged by Clément restores only the speed of tags search. findbobj() won't be further fixed as it provides many useful features which were missing to its (fast but poor) previous version. </p>
<p>Indeed, using findobj to get the handle of a figure knowing its id is the most complicated that one can imagine. It sounds to me like the affirmation: « why make it simple when it can be made complicated »... I mean, close() has to be fixed to use scf() as proposed in this thread.</p>
<p>Sometimes adding useful and needed features can degrade the speed of a function (y<span>ou can't make an omelette without breaking eggs). </span> Finding handles given their tag has its original performance with this patch. But I repeat that h=findobj('figure_id",id) should be avoided in favour of h=scf(id), until findobj() is completely rewritten in C++ (which is planned).</p>
<p><span>Moreover </span>nobody was complaining about xdel() and making it deprecated was never discussed by the community. Many people don't use Scilab any more because it misses features. So Scilab needs features, not absurde/arbitrary cleaning . A word to the wise...</p>
<p>S. </p>
<p id="reply-intro">Le 2022-01-22 14:24, Stéphane Mottelet a écrit :</p>
<blockquote type="cite" style="padding: 0 0.4em; border-left: #1010ff 2px solid; margin: 0">
<div id="replybody1">
<div dir="auto">
<div dir="ltr"> </div>
<div dir="ltr">Hi,</div>
<div dir="ltr"> </div>
<div dir="ltr">This is fixed for the next release:</div>
<div dir="ltr"> </div>
<div dir="ltr"><a href="https://antispam.utc.fr/proxy/v3?i=SGI0YVJGNmxZNE90Z2thMFYLWSxJOfIERJocpmb73Vs&r=SW5LV3JodE9QZkRVZ3JEYbCxfBpXCzNXFJI9eyEBCB_E8EQzXar_oWBGCSukLx6I&f=bnJjU3hQT3pQSmNQZVE3aPQ56ZBjot0Lu_H1dlHvp1727w_z78BVJc295PQlt99Z&u=https%3A//codereview.scilab.org/%23/c/21803/&k=dFBm" target="_blank" rel="noopener noreferrer">https://codereview.scilab.org/#/c/21803/</a></div>
<div dir="ltr"> </div>
<div dir="ltr">In the meantime, if you really need it you can download the patched version of findobj.</div>
<div dir="ltr"> </div>
<div dir="ltr">S.</div>
<div dir="ltr"><br />
<blockquote type="cite" style="padding: 0 0.4em; border-left: #1010ff 2px solid; margin: 0">Le 21 janv. 2022 à 23:54, Samuel Gougeon <sgougeon@free.fr> a écrit :<br /><br /></blockquote>
</div>
<blockquote type="cite" style="padding: 0 0.4em; border-left: #1010ff 2px solid; margin: 0">
<div dir="ltr">
<div class="v1moz-cite-prefix">Hello Claus,<br /><br />Le 21/01/2022 à 20:58, Claus Futtrup a écrit :</div>
<blockquote type="cite" style="padding: 0 0.4em; border-left: #1010ff 2px solid; margin: 0">Hi Scilabers <br /><br />Today I tried to replace xdel(winsid()) in one of my scripts with close(winsid()) because Scilab shows a warning in the console: <br /><br />Warning: Feature xdel(...) is obsolete and will be permanently removed in Scilab 6.2 <br /><br />Warning: Please use close(...) instead. <br /><br />I am negatively surprised that the close() command takes a long time to execute with 5-10 windows open (a few seconds per window). The xdel() command can do it in a split second. <br /><br /></blockquote>
<br />The issue comes not from close() but from the change of<a href="https://antispam.utc.fr/proxy/v3?i=SGI0YVJGNmxZNE90Z2thMFYLWSxJOfIERJocpmb73Vs&r=SW5LV3JodE9QZkRVZ3JEYbCxfBpXCzNXFJI9eyEBCB_E8EQzXar_oWBGCSukLx6I&f=bnJjU3hQT3pQSmNQZVE3aPQ56ZBjot0Lu_H1dlHvp1727w_z78BVJc295PQlt99Z&u=http%3A//bugzilla.scilab.org/show_bug.cgi%3Fid%3D13738%23c4&k=dFBm" target="_blank" rel="noopener noreferrer"> findobj() that it calls</a>, noticeably to perform findobj("figure_id",id) in 6.1.1.<br />It does <a href="https://antispam.utc.fr/proxy/v3?i=SGI0YVJGNmxZNE90Z2thMFYLWSxJOfIERJocpmb73Vs&r=SW5LV3JodE9QZkRVZ3JEYbCxfBpXCzNXFJI9eyEBCB_E8EQzXar_oWBGCSukLx6I&f=bnJjU3hQT3pQSmNQZVE3aPQ56ZBjot0Lu_H1dlHvp1727w_z78BVJc295PQlt99Z&u=http%3A//bugzilla.scilab.org/16734&k=dFBm" target="_blank" rel="noopener noreferrer">not impact only</a> close() for this case, that was processed <a href="https://antispam.utc.fr/proxy/v3?i=SGI0YVJGNmxZNE90Z2thMFYLWSxJOfIERJocpmb73Vs&r=SW5LV3JodE9QZkRVZ3JEYbCxfBpXCzNXFJI9eyEBCB_E8EQzXar_oWBGCSukLx6I&f=bnJjU3hQT3pQSmNQZVE3aPQ56ZBjot0Lu_H1dlHvp1727w_z78BVJc295PQlt99Z&u=https%3A//codereview.scilab.org/%23/c/20953/6/scilab/modules/gui/macros/findobj.sci&k=dFBm" target="_blank" rel="noopener noreferrer">as a special accelerated case</a> in the former findobj().<br />
<p>This downgrading<a href="https://antispam.utc.fr/proxy/v3?i=SGI0YVJGNmxZNE90Z2thMFYLWSxJOfIERJocpmb73Vs&r=SW5LV3JodE9QZkRVZ3JEYbCxfBpXCzNXFJI9eyEBCB_E8EQzXar_oWBGCSukLx6I&f=bnJjU3hQT3pQSmNQZVE3aPQ56ZBjot0Lu_H1dlHvp1727w_z78BVJc295PQlt99Z&u=http%3A//bugzilla.scilab.org/show_bug.cgi%3Fid%3D7117%23c6&k=dFBm" target="_blank" rel="noopener noreferrer"> was reported</a> 7 months before releasing 6.1.1, and knowingly ignored, while it is perfectly fixable by restoring the fast special case.</p>
<br />
<blockquote type="cite" style="padding: 0 0.4em; border-left: #1010ff 2px solid; margin: 0"><br />What is the motivation for replacing xdel with close? </blockquote>
<br />Both do the same thing. Scilab does not need duplicates.<br /><br />Regards<br />Samuel<br /><br /><span>_______________________________________________</span><br /><span>users mailing list</span><br /><span>users@lists.scilab.org</span><br /><span>http://lists.scilab.org/mailman/listinfo/users</span></div>
</blockquote>
</div>
</div>
<br />
<div class="pre" style="margin: 0; padding: 0; font-family: monospace">_______________________________________________<br />users mailing list<br /><a href="mailto:users@lists.scilab.org">users@lists.scilab.org</a><br /><a href="https://antispam.utc.fr/proxy/v3?i=SGI0YVJGNmxZNE90Z2thMFYLWSxJOfIERJocpmb73Vs&r=SW5LV3JodE9QZkRVZ3JEYbCxfBpXCzNXFJI9eyEBCB_E8EQzXar_oWBGCSukLx6I&f=bnJjU3hQT3pQSmNQZVE3aPQ56ZBjot0Lu_H1dlHvp1727w_z78BVJc295PQlt99Z&u=http%3A//lists.scilab.org/mailman/listinfo/users&k=dFBm" target="_blank" rel="noopener noreferrer">https://antispam.utc.fr/proxy/v3?i=SGI0YVJGNmxZNE90Z2thMFYLWSxJOfIERJocpmb73Vs&r=SW5LV3JodE9QZkRVZ3JEYbCxfBpXCzNXFJI9eyEBCB_E8EQzXar_oWBGCSukLx6I&f=bnJjU3hQT3pQSmNQZVE3aPQ56ZBjot0Lu_H1dlHvp1727w_z78BVJc295PQlt99Z&u=http%3A//lists.scilab.org/mailman/listinfo/users&k=dFBm</a></div>
</blockquote>
</body></html>