[Octave-bug-tracker] [bug #59273] [octave forge] (io) can't reliably load XLSX files in release 2.6.2

[Sébastien Villemot]

Follow-up Comment #18, bug #59273 (project octave):


[comment #17 commentaire #17 :]
> Please try with attached __OCT_spsh_open__.m and __ods_get_sheet_dims__.m,
bth in private/ subdir.
> As usual, on top of the other latest file versions in this bug report :-)

Thanks, it now works, all the tests pass. I am going to fix the Debian package
using those patches.

> I'm a bit surprised you got this error. This code apparently hasn't changed
since > 7 years; I can only surmise it is due to the test.ods spreadsheet in
the test directory. That spreadsheet *is* kosher, that is, it gives no errors
when reading with LibreOffice and any of the other spreadsheet interfaces, so
the bug must be real.
> Which makes me wonder if the debian tests have changed the last year, IOW,
why hasn't this been catched before.
> But then again, maybe maybe something changed in Octave-6+ that provoked the
bug. File encoding handling?

As said before, the problem with ODS/OCT started to appear when I applied the
patches that you added in comment #10.

The Debian testsuite has not changed for years. Also note that I use Octave
5.2.0 for all my tests, not Octave 6+. And I just verified that io 2.6.1,
using the same tests and the same environment, passes all the tests.

I have no idea why your recent changes triggered the bug in ODS/OCT. But there
is clearly a causality relationship.

As far as I’m concerned, this bug is now fixed. Thanks for your help.


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  Message posté via Savannah
  https://savannah.gnu.org/