Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1217)

Issue 11916045: encoding/csv: always allow trailing commas (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 9 months ago by rsc
Modified:
10 years, 9 months ago
Reviewers:
Visibility:
Public.

Description

encoding/csv: always allow trailing commas The motivation for disallowing them was RFC 4180 saying "The last field in the record must not be followed by a comma." I believe this is an admonition to CSV generators, not readers. When reading, anything followed by a comma is not the last field.

Patch Set 1 #

Patch Set 2 : diff -r ed08d206a05c https://code.google.com/p/go/ #

Patch Set 3 : diff -r ed08d206a05c https://code.google.com/p/go/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -41 lines) Patch
M src/pkg/encoding/csv/reader.go View 1 3 chunks +7 lines, -29 lines 0 comments Download
M src/pkg/encoding/csv/reader_test.go View 1 2 1 chunk +12 lines, -12 lines 0 comments Download

Messages

Total messages: 1
rsc
10 years, 9 months ago (2013-08-02 16:29:15 UTC) #1
*** Abandoned ***
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b