[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: |
Tue, 15 Mar 2016 23:55:50 +0000 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:41.0) Gecko/20100101 Firefox/41.0 SeaMonkey/2.38 |
Follow-up Comment #86, patch #8783 (project octave):
In the first FAIL it is really just a wrong test. Matlab and Octave behave
exactly the same. I just added that comma to the text string and the FAIL was
gone (also Matlab gave the "right" answer).
I'm afraid that the second FAIL is a case of "bug-for-bug" Matlab
compatibility. Would you agree that a line like
",,,,10" does have a missing (empty) field before the first comma?
If multipledelims = 0, Octave and Matlab both recognize the empty field.
If multipledelimsasone = 1, suddenly that empty field isn't recognized
anymore. IMO that is inconsistent, even if it is the supposedly "proper"
behavior.
Well, let JWE have the final word here.
I will mention this "bug" on the Matlab prerelease feedback web page; I'm
curious as to what TMW will say about it (if they react at all).
My mxe cross-build errors out in the end with:
libtool: install: /usr/bin/install -c libinterp/.libs/liboctinterp-3.dll
/home/philip/devel/octdev/mxe/mxe_w64_20160308/tmp-default-octave-install/home/philip/devel/octdev/mxe/mxe_w64_20160308/usr/x86_64-w64-mingw32/lib/octave/4.1.0+/../../../bin/liboctinterp-3.dll
/home/philip/devel/octdev/mxe/mxe_w64_20160308/tmp-default-octave/octave-4.1.0+/.build/libtool:
line 10095:
/home/philip/devel/octdev/mxe/mxe_w64_20160308/tmp-default-octave/octave-4.1.0+/.build/src/octave:
No such file or directory
Makefile:27022: recipe for target 'doc/interpreter/doc-cache' failed
make[5]: *** [doc/interpreter/doc-cache] Error 127
make[5]: *** Waiting for unfinished jobs....
All binaries in usr/mingw..... were compiled so I just gambled and copied them
over but once in Windows Octave didn't run.
Something to sort out tomorrow evening, it is past midnight here.
_______________________________________________________
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, Philip Nienhuis, 2016/03/14
- [Octave-patch-tracker] [patch #8783] C++ implementation of textscan, Philip Nienhuis, 2016/03/14
- [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 <=
- [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/16
- [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