Skip to content
This repository has been archived by the owner on Feb 5, 2022. It is now read-only.

fix string in tuple for python 3 #11

Merged
merged 3 commits into from Jan 13, 2016
Merged

fix string in tuple for python 3 #11

merged 3 commits into from Jan 13, 2016

Conversation

pmuston
Copy link
Contributor

@pmuston pmuston commented Jan 13, 2016

I got an exception when decoding a string in a tuple in Python 3.
try:
int_str_int = Tuple(Integer, String, Integer)
encoded = int_str_int.encode((123, 'abc\ndef', 456,))
decoded = int_str_int.decode(encoded)

I fixed it and added a test.

@@ -0,0 +1 @@
{}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you exclude this file from the commit?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good suggestions.
I'll work out how to do that.
This is my first pull request.

On 13 January 2016 at 06:07, Hong Minhee notifications@github.com wrote:

In .cache/v/cache/lastfailed
#11 (comment):

@@ -0,0 +1 @@
+{}

Could you exclude this file from commit?


Reply to this email directly or view it on GitHub
https://github.com/dahlia/sider/pull/11/files#r49549811.

@dahlia
Copy link
Owner

dahlia commented Jan 13, 2016

Thanks for your effort!

dahlia added a commit that referenced this pull request Jan 13, 2016
fix string in tuple for python 3
@dahlia dahlia merged commit 7db8c82 into dahlia:master Jan 13, 2016
dahlia added a commit that referenced this pull request Jan 13, 2016
@pmuston
Copy link
Contributor Author

pmuston commented Jan 13, 2016

Thanks for your module!!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants