Skip to content

accept unicode strings in parse_string() under Python 2#49

Merged
timlnx merged 1 commit intotimlnx:masterfrom
drewbrew:support-unicode-in-parse-string
May 31, 2015
Merged

accept unicode strings in parse_string() under Python 2#49
timlnx merged 1 commit intotimlnx:masterfrom
drewbrew:support-unicode-in-parse-string

Conversation

@drewbrew
Copy link
Copy Markdown
Contributor

In Python 3, strings are unicode, so this problem does not exist, but in Python 2, if you call parse_string(u"20 GiB"), it'll raise a ValueError.

In Python 3, str == unicode, so this problem does not exist, but in
Python 2, if you call `parse_string(u"20 GiB")`, it'll raise a
ValueError.
@drewbrew drewbrew force-pushed the support-unicode-in-parse-string branch from 9e78dbc to 05e9729 Compare May 28, 2015 18:19
@timlnx timlnx self-assigned this May 29, 2015
@timlnx
Copy link
Copy Markdown
Owner

timlnx commented May 31, 2015

Reviewed. Looks good!

Thanks @drewbrew merged!

timlnx added a commit that referenced this pull request May 31, 2015
accept unicode strings in parse_string() under Python 2
@timlnx timlnx merged commit b568258 into timlnx:master May 31, 2015
@drewbrew drewbrew deleted the support-unicode-in-parse-string branch June 1, 2015 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants