[Octave-bug-tracker] [bug #45171] isequal fails for large sparse matrices (subscript indixes too large)

[Carlo de Falco]

Follow-up Comment #2, bug #45171 (project octave):

This patch was being discussed on IRC yesterday,
when my connection went suddenly off, I'll
continue the discussion here.

Just before communication was interrupted jwe asked:


<jwe> cdf: in that last one, is length the right thing to apply to x?  That's
a strange function because it returns the larger of the number of rows or
columns.


My answer would have been:

I would say *length* is equivalent to *numel* in this context because *find*
should always return vectors, as noted in its
documentation
<https://www.gnu.org/software/octave/doc/interpreter/Finding-Elements-and-Checking-Conditions.html#XREFfind>.

But if you prefer it I see no problem in replacing *length* by *numel* in the
patch.

Also note that the use of *length* was there before this change.



    _______________________________________________________

Reply to this item at:

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

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