Re: [Pharo-users] NeoCSV change proposal

2017-07-23 Thread Peter Uhnak
Thank you Sven! Peter On Sun, Jul 23, 2017 at 01:59:41PM +0200, Sven Van Caekenberghe wrote: > > > On 23 Jul 2017, at 09:55, Peter Uhnak wrote: > > > > Ah, ByteArrayMap, I was trying ByteArray. > > ByteArrayMap is not a class, it is still a ByteArray, but of size 256 used

Re: [Pharo-users] NeoCSV change proposal

2017-07-23 Thread Sven Van Caekenberghe
> On 23 Jul 2017, at 09:55, Peter Uhnak wrote: > > Ah, ByteArrayMap, I was trying ByteArray. ByteArrayMap is not a class, it is still a ByteArray, but of size 256 used as an inclusion map for characters that fit a byte. BTW, using CharacterSet as an abstraction makes the

Re: [Pharo-users] NeoCSV change proposal

2017-07-23 Thread Stephane Ducasse
thanks for the explanation. this is cool. On Sat, Jul 22, 2017 at 10:06 PM, Sven Van Caekenberghe wrote: > >> On 22 Jul 2017, at 21:51, Stephane Ducasse wrote: >> >> Hi peter >> Do you have an example that I understand? > > In its most simple form, CSV

Re: [Pharo-users] NeoCSV change proposal

2017-07-23 Thread Peter Uhnak
Ah, ByteArrayMap, I was trying ByteArray. Not sure what the next step is here: should I add it and send you mczs, or will you do it yourself? (it is a simple change). Peter On Sat, Jul 22, 2017 at 10:51:42PM +0200, Sven Van Caekenberghe wrote: > Peter, > > > On 22 Jul 2017, at 22:27, Peter

Re: [Pharo-users] NeoCSV change proposal

2017-07-22 Thread Sven Van Caekenberghe
Peter, > On 22 Jul 2017, at 22:27, Peter Uhnak wrote: > > Hi Sven, > > my use case was hand-edited CSVs (therefore the quotes are extra clutter), > which imples that I would be hand-viewing/editing only small CSVs (no > performance issues). > > I agree that we should err

Re: [Pharo-users] NeoCSV change proposal

2017-07-22 Thread Peter Uhnak
Hi Sven, my use case was hand-edited CSVs (therefore the quotes are extra clutter), which imples that I would be hand-viewing/editing only small CSVs (no performance issues). I agree that we should err on the safe side with CR & LF (e.g. tools may sometimes autoconvert line endings).

Re: [Pharo-users] NeoCSV change proposal

2017-07-22 Thread Sven Van Caekenberghe
> On 22 Jul 2017, at 21:51, Stephane Ducasse wrote: > > Hi peter > Do you have an example that I understand? In its most simple form, CSV looks like this: a,b,c,d However, if a field is text, it might contain a comma itself, or other problematic characters like line

Re: [Pharo-users] NeoCSV change proposal

2017-07-22 Thread Stephane Ducasse
Hi peter Do you have an example that I understand? @Svn Once the change is integrated could you add a little paragraph to the chapter? Stef On Sat, Jul 22, 2017 at 6:51 PM, Sven Van Caekenberghe wrote: > Hi Peter, > >> On 22 Jul 2017, at 14:12, Peter Uhnak

Re: [Pharo-users] NeoCSV change proposal

2017-07-22 Thread Sven Van Caekenberghe
Hi Peter, > On 22 Jul 2017, at 14:12, Peter Uhnak wrote: > > Hi, > > this is a continuation of an older thread about quoting fields only when > necessary. ( > http://forum.world.st/NeoCSVWriter-automatic-quotes-td4924781.html ) > > I've attached fileouts of NeoCSV

Re: [Pharo-users] NeoCSV change proposal

2017-07-22 Thread Sven Van Caekenberghe
Peter, > On 22 Jul 2017, at 14:12, Peter Uhnak wrote: > > Hi, > > this is a continuation of an older thread about quoting fields only when > necessary. ( > http://forum.world.st/NeoCSVWriter-automatic-quotes-td4924781.html ) > > I've attached fileouts of NeoCSV packages

Re: [Pharo-users] NeoCSV change proposal

2017-07-22 Thread Peter Uhnak
attached On Sat, Jul 22, 2017 at 02:12:10PM +0200, Peter Uhnak wrote: > Hi, > > this is a continuation of an older thread about quoting fields only when > necessary. ( > http://forum.world.st/NeoCSVWriter-automatic-quotes-td4924781.html ) > > I've attached fileouts of NeoCSV packages with the

[Pharo-users] NeoCSV change proposal

2017-07-22 Thread Peter Uhnak
Hi, this is a continuation of an older thread about quoting fields only when necessary. ( http://forum.world.st/NeoCSVWriter-automatic-quotes-td4924781.html ) I've attached fileouts of NeoCSV packages with the addition (I don't know if Metacello can file-out only changesets). The change