Ron Goldman wrote:
> Hi,
> In the file xls2oct.m from io-10.0.14 there is a problem reading dates
> with JXL.
> I have traced a problem to places where strsplit is called with
> arguments in the wrong order. There is another problem with undefined
> variable months which I naively defined as {'JAN','FEB',.. } but I
> suppose a better correction should consider locals. Another problem is a
> reverse condition of isTime.
> I attach my svn diff, can someone look at it and see if it's ok?

Thank you for looking into this.

Your patch looks quite OK. I've edited some style (spaces in months(...) 
declaration, and moved this higher up), added a 'persistent' statement 
for months, and applied the rest verbatim.

I'll commit it later tonight.

Philip

------------------------------------------------------------------------------
Xperia(TM) PLAY
It's a major breakthrough. An authentic gaming
smartphone on the nation's most reliable network.
And it wants your games.
http://p.sf.net/sfu/verizon-sfdev
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to