[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #8783] C++ implementation of textscan
From: |
Philip Nienhuis |
Subject: |
[Octave-patch-tracker] [patch #8783] C++ implementation of textscan |
Date: |
Wed, 16 Mar 2016 07:55:30 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux i686 on x86_64; rv:41.0) Gecko/20100101 Firefox/41.0 SeaMonkey/2.38 |
Follow-up Comment #88, patch #8783 (project octave):
OK I yield....
But then something must be done about the 2nd FAIL, either the test or the
code. Your pick?
Maybe it is time (for me) to run all the textscan/strread tests through Matlab
again. After all, only now that the limitations of the m-code have vanished we
can strive for max compatibility. I'm not pessimistic BTW, I think at most
we'll only catch rare corner cases like bug #44750.
Trying the example there in Matlab r2016a learns that w/o
multipledelimsasone=1 we get {'', 'home','foo'} and with it we get {'home
'foo'}. Intriguing but I suppose your reasoning is better than mine. Your
observation that no design choice can be 100 % consistent is absolutely
right.
Looking back I've learned a lot about Octave's m-code figuring out how to
reliably parse complicated repeated text patterns. But m-code was never the
best means for what textscan is to get together. I'm happy with leaving the
role of strread/textread/textscan custodian behind me :-)
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?8783>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, (continued)
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Ben Abbott, 2016/03/14
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Lachlan Andrew, 2016/03/15
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/15
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, John W. Eaton, 2016/03/15
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/15
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/15
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/15
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Lachlan Andrew, 2016/03/15
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/15
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Lachlan Andrew, 2016/03/15
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan,
Philip Nienhuis <=
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Ben Abbott, 2016/03/16
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/17
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/17
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, John W. Eaton, 2016/03/17
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Mike Miller, 2016/03/17
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/17
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Ben Abbott, 2016/03/17
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Lachlan Andrew, 2016/03/17
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Lachlan Andrew, 2016/03/17
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/18