Skip to content

Conversation

@hongquan
Copy link
Contributor

@hongquan hongquan commented Aug 8, 2019

Per Python 3 documentation, the second parameter of recv_into is optional.

@hongquan hongquan changed the title Fix compliant with original recv_into() Fix compliance with original recv_into() Aug 8, 2019
@mindflayer
Copy link
Owner

mindflayer commented Aug 8, 2019

Good job, I noticed that a couple of days ago when I was investigating the problem with redis.
Thank you.

@mindflayer mindflayer merged commit cc0dd4c into mindflayer:master Aug 8, 2019
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 97.642% when pulling 0176f16 on sunshine-tech:fix/recv_into into 6dbfdec on mindflayer:master.

@mindflayer
Copy link
Owner

@hongquan
Copy link
Contributor Author

hongquan commented Aug 8, 2019

Thanks, @mindflayer

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.

3 participants