Author vajrasky
Recipients Thibault.Kruse, ezio.melotti, r.david.murray, serhiy.storchaka, vajrasky
Date 2013-09-08.15:10:16
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1378653016.86.0.490493070032.issue18829@psf.upfronthosting.co.za>
In-reply-to
Content
David R. Murray said, '"delimiter must be a 1 character string" would cover it.'

You mean

$ ./python -c 'import csv; reader = csv.reader("foo", delimiter="")' 

should give this error '"delimiter" must be a 1 character string'?

Attached the patch to accommodate your request.

I found out that:

$ ./python -c 'import csv; reader = csv.reader("foo", quotechar="")'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
TypeError: quotechar must be set if quoting enabled

This is not consistent with the cure. But since this ticket is about delimiter, we keep the status-quo about quotechar for now. After the patch is committed, we can open the ticket about the quotechar inconsistency.

The fix for this quotechar is not straightforward, though, because of the quoting condition.
History
Date User Action Args
2013-09-08 15:10:16vajraskysetrecipients: + vajrasky, ezio.melotti, r.david.murray, serhiy.storchaka, Thibault.Kruse
2013-09-08 15:10:16vajraskysetmessageid: <1378653016.86.0.490493070032.issue18829@psf.upfronthosting.co.za>
2013-09-08 15:10:16vajraskylinkissue18829 messages
2013-09-08 15:10:16vajraskycreate