[Octave-bug-tracker] [bug #43098] "if (array)" should raise a warning, and give the same result for dense and sparse

[Lachlan Andrew]

Follow-up Comment #11, bug #43098 (project octave):

I couldn't reproduce the failure with the natural syntax of the BIST, so the
patch now uses the natural syntax.

The previous attempt to make the range case match  all(r!=0)  wouldn't have
worked, so now it just reverts to the old inefficient code for cases that
cross zero.

Finally, the new patch shows the warning each time instead of only the first
time.  I still think it is useful to have a "once only" mode for warnings. 
For example, if the code that does "if(array)" is third party code, the user
will probably not want to modify it -- especially if it is updated frequently.
 Still, it would be better to be part of a framework rather than a kludge for
this particular warning.

(file #37563)
    _______________________________________________________

Additional Item Attachment:

File name: bug_43098_warn_if_array.v4.cset Size:11 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?43098>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/