[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: new strsplit function
From: |
Ben Abbott |
Subject: |
Re: new strsplit function |
Date: |
Tue, 02 Apr 2013 19:40:10 -0400 |
On Apr 2, 2013, at 6:35 PM, Carnë Draug wrote:
> On 2 April 2013 18:04, Ben Abbott <address@hidden> wrote:
>
>> On Apr 2, 2013, at 12:00 PM, Carnë Draug wrote:
>>
>>> On 2 April 2013 13:02, Ben Abbott <address@hidden> wrote:
>>>> I've added a "conventional" to the possible delimitertypes, and selected
>>>> this as the default when all delimiters are scalar characters. I've also
>>>> enabled 2D character array inputs. The following replacements may be made
>>>> into OF, and Octave core, to use Jaroslav's implementation. A changeset
>>>> to core to use these substitutions will also be needed.
>>>>
>>>> - strsplit (str, del)
>>>> + strsplit (str, del, "collapsedelimiters", false, "delimitertype",
>>>> "conventional")
>>>>
>>>> - strsplit (str, del, false)
>>>> + strsplit (str, del, "collapsedelimiters", false, "delimitertype",
>>>> "conventional")
>>>>
>>>> - strsplit (str, del, true)
>>>> + strsplit (str, del, "collapsedelimiters", true, "delimitertype",
>>>> "conventional")
>>>
>>> That means that OF packages with this change, will only work after 3.8
>>> being released, and the ones without the change will stop working once
>>> 3.8 is released, so the releases would have to be timed.
>>>
>>> Instead, I propose we add the old strsplit in the private directory of
>>> each package. Then, once 3.8 gets released, we can remove them and
>>> make the change as each package gets re-released.
>>
>> That's a good idea!
>
> I have just made the change on revision 11779. I have also created a
> new task on the trackers https://savannah.gnu.org/task/index.php?12561
> so we don't forget what packages got affected.
>
> Finally, geometry and mechanics package need to be fixed some other
> way since they make use of it on PKG_DEL and PKD_ADD scripts.
>
> Carnë
I pushed a slightly modified changeset.
http://hg.savannah.gnu.org/hgweb/octave/rev/5be43435bd5b
Ben
- Re: new strsplit function, Rik, 2013/04/01
- Re: new strsplit function, Ben Abbott, 2013/04/01
- Re: new strsplit function, Philip Nienhuis, 2013/04/01
- Re: new strsplit function, Ben Abbott, 2013/04/02
- Re: new strsplit function, Carnë Draug, 2013/04/02
- Re: new strsplit function, Ben Abbott, 2013/04/02
- Re: new strsplit function, Carnë Draug, 2013/04/02
- Re: new strsplit function,
Ben Abbott <=
- Re: new strsplit function, John W. Eaton, 2013/04/02
- Re: new strsplit function, Ben Abbott, 2013/04/03
- Re: new strsplit function, Ben Abbott, 2013/04/20
- Re: new strsplit function, Philip Nienhuis, 2013/04/20
- Re: new strsplit function, Ben Abbott, 2013/04/20
- Re: new strsplit function, Philip Nienhuis, 2013/04/20
- Re: new strsplit function, Ben Abbott, 2013/04/20
- Re: new strsplit function, Ben Abbott, 2013/04/23
- Re: new strsplit function, Ben Abbott, 2013/04/23
- Re: new strsplit function, Ben Abbott, 2013/04/02