From 69646f624b5888cddefa2b6a63495bc02f8ae2ea Mon Sep 17 00:00:00 2001 From: Mike Thomas Date: Thu, 6 Mar 2008 21:43:08 +0000 Subject: [PATCH] Roll this back - calling interrupt() in this way caused the other end's writer to get fairly upset. git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@4961 542714f4-19e9-0310-aa3c-eee0fc999fb1 --- .../threerings/presents/client/BlockingCommunicator.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/java/com/threerings/presents/client/BlockingCommunicator.java b/src/java/com/threerings/presents/client/BlockingCommunicator.java index 593b6067e..5ee61f763 100644 --- a/src/java/com/threerings/presents/client/BlockingCommunicator.java +++ b/src/java/com/threerings/presents/client/BlockingCommunicator.java @@ -501,7 +501,11 @@ public class BlockingCommunicator extends Communicator { // we want to interrupt the reader thread as it may be blocked listening to the socket; // this is only called if the reader thread doesn't shut itself down - interrupt(); + + // While it would be nice to be able to handle wacky cases requiring reader-side + // shutdown, doing so causes consternation on the other end's writer which suddenly + // loses its connection. So, we rely on the writer side to take us down. + // interrupt(); } }