[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Joseph Tate jtate+pyt...@dragonstrider.com added the comment: Thank you, Thomas, for the patch, and Skip, for applying it. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Changes by Raymond Hettinger rhettin...@users.sourceforge.net: -- assignee: - skip.montanaro nosy: +skip.montanaro ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Thomas W. Barr t...@rice.edu added the comment: Patch uploaded to rietveld: http://codereview.appspot.com/96202/show -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
New submission from Joseph Tate jtate+pyt...@dragonstrider.com: Given the attached code, the Sniffer.sniff routine does not set the doublequote property. This results in errors during reader operations. If the doublequote property is set in the dialect, the data is read properly. The data was created using oocalc, forcing it to use ascii quotes rather than u'\u201c\u201d'. -- components: Library (Lib) files: foo.py messages: 91109 nosy: jtate severity: normal status: open title: csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly type: behavior versions: Python 2.4, Python 2.6 Added file: http://bugs.python.org/file14609/foo.py ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Joseph Tate jtate+pyt...@dragonstrider.com added the comment: Note that no exceptions are raised, the reader just returns improperly parsed records. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Thomas W. Barr t...@rice.edu added the comment: The Sniffer.sniff routine doesn't set the doublequote property at all right now. I'm working on a patch to see if I can add this functionality. -- nosy: +twb type: behavior - feature request ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Thomas W. Barr t...@rice.edu added the comment: Test for this issue. -- keywords: +patch Added file: http://bugs.python.org/file14610/test_csv.py.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Thomas W. Barr t...@rice.edu added the comment: Patch for the issue. Looks for extraneous quote inside the quotes by the delimiters. -- Added file: http://bugs.python.org/file14611/csv.py.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Thomas W. Barr t...@rice.edu added the comment: The documentation doesn't actually say what parameters are sniffed, so technically, that doesn't need to be changed. Should this be added? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Changes by Thomas W. Barr t...@rice.edu: Removed file: http://bugs.python.org/file14611/csv.py.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly
Thomas W. Barr t...@rice.edu added the comment: Reformatted line in patch. -- Added file: http://bugs.python.org/file14612/csv.py.diff ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue6606 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com