Message48501
Logged In: YES
user_id=1301296
> I'm not inclined to clutter the C code with further
> complications.
Sorry - I haven't been keeping up with the existing
complications! Don't forget that one man's clutter is
another man's functionality. It doesn't actually require
much of a change to the code, although I was slightly
suprised to discover that this module was in C in the first
place...
Basically, I noticed that the csv module has a bias towards
Excel-generated csv files, but most of the time I've come
across csv files, they were hand-edited, and I've often seen
comment fields as described in the submission.
My submission was intended in the "batteries included"
spirit (I do understand that you stop short of the kitchen
sink), and also seemed in-keeping with the
'skipinitialspace' option within the existing csv module.
> Why can't you implement this on an as-needed basis
> with a file object wrapper [other options]
True, I could do any of those things, but it would be
simpler / clearer not to have to. Of course, if you took
your argument further, you could cut chunks out of several
modules; the argument comes down to whether the benefits
outweigh the additional complexity. I was suprised to
discover the option wasn't already there, but maybe that's
just me.
In any case, if your vote goes from your apparent -0 to -1,
that's your choice, and you're better placed to make it than
I am.
Cheers,
Iain. |
|
Date |
User |
Action |
Args |
2007-08-23 15:43:21 | admin | link | issue1225769 messages |
2007-08-23 15:43:21 | admin | create | |
|