Code review comment for lp:~jameinel/bzr/2.5-conn-reset-socket-pipe-1047325

Revision history for this message
Richard Wilbur (richard-wilbur) wrote :

This looks good but I agree the integration with your other patch is very important. Should the other patch also treat a single 0 return value as a ConnectionReset?

+1

review: Approve

« Back to merge proposal