Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Comma in string #17

Closed
ghost opened this issue Jan 29, 2014 · 0 comments
Closed

Comma in string #17

ghost opened this issue Jan 29, 2014 · 0 comments

Comments

@ghost
Copy link

ghost commented Jan 29, 2014

Part of my file is

[plot]
title = [
"Client: XXXX, Job: XXXX",
"Code: XXXX"
]

when parsed gives the error
ValueError: invalid literal for int() with base 10: 'Job XXXX"'

C:\Anaconda>pip show toml
Name: toml
Version: 0.8.1
Location: c:\anaconda\lib\site-packages
Requires:

The comma in the first string of the array causes a parsing error, it is expecting that the Job part be an integer.
@uiri uiri mentioned this issue Jan 29, 2014
Closed
uiri added a commit that referenced this issue Jan 29, 2014
@uiri
Fix issue #17 with comma inside string in array
6bcd44d 
The array is split on commas and then each result of the split is
parsed as a value. However, with commas inside the string, string
fragments were being passed to the value parser. This change detects
if the array is an array of strings and if so, it looks for string
fragments and concatenates them.
@uiri uiri closed this as completed in 1fa0338 Jan 29, 2014
uiri added a commit to uiri/toml-test that referenced this issue Jan 27, 2018
@uiri
Add regression test for uiri/toml#17
4e9162d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
0 participants