[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-29 Thread R. David Murray
R. David Murray added the comment: Committed, with slight modifications to the tests. Thanks Vajrasky. -- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker __

[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-29 Thread Roundup Robot
Roundup Robot added the comment: New changeset 68ff68f9a0d5 by R David Murray in branch '3.3': #18155: Regex-escape delimiter, in case it is a regex special char. http://hg.python.org/cpython/rev/68ff68f9a0d5 New changeset acaf73e3d882 by R David Murray in branch 'default': Merge #18155: Regex-e

[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-14 Thread A.M. Kuchling
Changes by A.M. Kuchling : -- nosy: +akuchling ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyth

[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-07 Thread Vajrasky Kok
Vajrasky Kok added the comment: Added test to sniffer double quote. -- Added file: http://bugs.python.org/file30501/csv_has_header.diff ___ Python tracker ___ ___

[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-07 Thread Vajrasky Kok
Changes by Vajrasky Kok : Removed file: http://bugs.python.org/file30499/csv_has_header.diff ___ Python tracker ___ ___ Python-bugs-list maili

[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-07 Thread Vajrasky Kok
Vajrasky Kok added the comment: Sorry. This one is correct. Attached the patch to fix the problem. -- Added file: http://bugs.python.org/file30499/csv_has_header.diff ___ Python tracker

[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-07 Thread Vajrasky Kok
Vajrasky Kok added the comment: Attached the patch to fix the problem. -- keywords: +patch nosy: +vajrasky Added file: http://bugs.python.org/file30498/csv_has_header.diff ___ Python tracker ___

[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-07 Thread Vajrasky Kok
Changes by Vajrasky Kok : Removed file: http://bugs.python.org/file30498/csv_has_header.diff ___ Python tracker ___ ___ Python-bugs-list maili

[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-07 Thread R. David Murray
R. David Murray added the comment: I doubt this is a regression, so I'm marking the others versions as well without actually testing it. Should be an easy fix. -- keywords: +easy nosy: +r.david.murray stage: -> needs patch versions: +Python 2.7, Python 3.4 ___

[issue18155] csv.Sniffer.has_header doesn't escape characters used in regex

2013-06-07 Thread Dave Challis
New submission from Dave Challis: When attempting to detect the presence of CSV headers, delimiters are passed to a regex function without escaping, which causes an exception if a delimiter which has meaning in a regex (e.g. '+', '*' etc.) is used. Code to reproduce: import csv s = csv.Sniffer