Re: Package Proposal: Text_TSV

On Sunday, Jul 13, 2003, at 12:52 US/Central, Tomas V.V.Cox wrote:

I love the well presented class

Thanks ...

but I guess that there is already this functionality in PEAR and missing features should be implemented there. At least the points you mention in your page can be easily added. I'm -1.

...well, maybe not *easily*; if it was easy it would already be there. ;-)

Btw I think that File_CSV should have its own package, nowadays the class is very hidden inside the PEAR CVS.

Agreed -- it'd be nice to have a little "higher up" in the hierarchy. Or perhaps (if this is allowed) split File_CSV off from File entirely and create a new package (Text_CSV) to parse the results of reading a file. That logically separates the functions of reading a file from parsing its contents. (Or, move the parsing functionality out of File_CSV into Text_CSV and user File_CSV just to read specific lines and then handle parsing with Text_CSV.) Just a suggestion -- any comments on the splitting-up idea? I'll reply to the "combine functionality" argument in Martin's email. -- pmj