[Octave-bug-tracker] [bug #58942] [octave forge] (io) version 2.6.1 read cell range data from worksheet

[Philip Nienhuis]

Follow-up Comment #4, bug #58942 (project octave):

Ping ....

@OP (Carlos):
Can you please explain what bug you saw, and how your patch will fix that bug?
Do you have a reproducible scenario?

AFAIU there are a few intermingled things in your report:
* You got an error message "error: xls2oct: worksheet number 1 > number of
worksheets in file (0)"
That implies that xls2oct.m didn't find a worksheet. AFAICS the only way that
can happen is if you opened a new file, didn't add any data to it and then
tried to read from it.

* Your patch allows to skip specifying a worksheet in xls2oct.m; the same is
done in xlsread.m so there's already a similar mechanism in place. I'm not
fond of having similar measures in different parts of the io package code
base.

I'm afraid that _if_ you hit a real bug your fix is just fixing symptoms but
not a real fix for the underlying issue.

As I couldn't detect any issues in the io package code I intend to close this
report with "invalid" but I wouldn't want to let a real bug pass into a next
io package release.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?58942>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/