This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author andrewmcnamara
Recipients Mario.Fasold, amaury.forgeotdarc, andrewmcnamara, eric.araujo, iain_haslam, orsenthil, pitrou, rhettinger, skip.montanaro
Date 2010-02-04.02:10:21
SpamBayes Score 1.2925994e-10
Marked as misclassified No
Message-id <1265249424.27.0.173536144843.issue1225769@psf.upfronthosting.co.za>
In-reply-to
Content
Okay, while I am sympathetic to the points raised by the people asking for this enhancement, I'm persuaded to reject it by the arguments that the potential benefit is outweighed by the increase in complexity (code and documentation).

While the attached patch from Iain requires relatively minor and innocuous changes to the state machine, it only satisfies a limited subset of users, and the effect of complexity is geometric. In other words, to understand the state machine, you need to understand the relation of each state to every other state, etc.

As to why the core of the module is implemented in C, the sort of stateful parsing done by the module is one area where python does not excel (Ha!). Stripping comments, on the other hand, CAN be done with reasonable efficiency from python (and considerably more flexibility). Also, it has been my experience that CSV files that use comments typically have other non-standard features as well, requiring additional pre- and post-processing in any case.
History
Date User Action Args
2010-02-04 02:10:24andrewmcnamarasetrecipients: + andrewmcnamara, skip.montanaro, rhettinger, amaury.forgeotdarc, orsenthil, pitrou, iain_haslam, eric.araujo, Mario.Fasold
2010-02-04 02:10:24andrewmcnamarasetmessageid: <1265249424.27.0.173536144843.issue1225769@psf.upfronthosting.co.za>
2010-02-04 02:10:22andrewmcnamaralinkissue1225769 messages
2010-02-04 02:10:21andrewmcnamaracreate