Code review comment for lp:~bac/python-memcached/bug-974632

Revision history for this message
Sean Reifschneider (jafo) wrote :

I think this misses some cases that the current could would also raise an exception in. This is a good start, and helped me understand the problem, thanks for the patch, but I've made some other changes which include restoring the readline() back to not raising an exception in most cases, and the _unsafe_*() functions calling functions which explicitly request that readline() raise the exception.

I'm going to commit a change shortly, would you mind reviewing it and seeing if it fixes the issue?

Thanks,
Sean

review: Disapprove

« Back to merge proposal