[Octave-patch-tracker] [patch #8593] Faster version of dlmread(), supporting escaped delimiters

[Philip Nienhuis]

Update of patch #8593 (project octave):

                  Status:                    None => Wont Do                
             Open/Closed:                    Open => Closed                 

    _______________________________________________________

Follow-up Comment #3:

I was afraid of that :-(

But I really do appreciate your hard work. Your code didn't look bad at all
(but I'm no c++ coder).

The 4092 row size limit has been discussed in the help ML a while back; for me
it's just a parameter, could be 32 Kb as well, it was just there when
csv2cell/cell2csv were moved from the miscellaneous package to io some years
back.


If you still have appetite to help develop Octave, there are some close
alternatives you might be interested in:

- As regards the column selector (and optionally row selector), that would be
a great addition; could you implement that for csv2cell?

- On the maintainers ML at irregular times the discussion about a binary
textscan() pops up (it's now an m-file and the work horse strread,m became so
complex that it is almost unmaintanable - I know because I rewrote large parts
of it). 
I figured csv2cell could be a start for a binary textscan. 2 or 3 years ago
the lead dev supplied a __xtextscan__.cc skeleton that could already do some
input parsing, the actual read code was the big missing piece.


Once again, thanks for your efforts.
Closing report, but you can still post here if you like.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?8593>

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