[Octave-bug-tracker] [bug #59277] [octave-forge](io) xls2oct is slow when a spreadsheet contains many text cells

[Philip Nienhuis]

Follow-up Comment #25, bug #59277 (project octave):

> So, replacing lines 204 and 205 as follows may speed up things further:
> idx.num = vi.row;
> idx.alph = vi.alph; 
Good catch!

A little further down, 
* that 2nd assignment idx.alph = vi.alph seems a bit useless, vi.alph isn't
referenced afterwards;
and
* vi.col turns out to be identical to idx.alph, so even idx.alph isn't
referenced anymore.

This is original code that AFAICR I haven't ever touched except for column and
row limits. I need to look better to not screw up things, the program flow
isn't exactly straightforward.
But some more optimization is due, absolutely.


    _______________________________________________________

Reply to this item at:

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

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