[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-09-28 Thread Joseph Tate
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 ___

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-09-24 Thread Raymond Hettinger
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 ___

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-07-31 Thread Thomas W. Barr
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 ___

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-07-30 Thread Joseph Tate
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

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-07-30 Thread Joseph Tate
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

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-07-30 Thread Thomas W. Barr
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

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-07-30 Thread Thomas W. Barr
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

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-07-30 Thread Thomas W. Barr
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

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-07-30 Thread Thomas W. Barr
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

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-07-30 Thread Thomas W. Barr
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 ___

[issue6606] csv.Sniffer.sniff on data with doublequotes doesn't set up the dialect properly

2009-07-30 Thread Thomas W. Barr
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 ___