octave-patch-tracker
[Top][All Lists]
Advanced

[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/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]