Document potential brokeness.

1 files changed, 6 insertions, 0 deletions

diff --git a/paramiko/channel.py b/paramiko/channel.py
index 583809d5..a7d71212 100644
--- a/paramiko/channel.py
+++ b/paramiko/channel.py
@@ -683,6 +683,9 @@ class Channel (object):
         self.lock.acquire()
         try:
             size = self._wait_for_send_window(size)
+            # Instead of returning 0 here when the channel is closed, we might
+            # want to raise EOFError or similar. Should we just do as in the
+            # send_all method, raise a socket.error?
             if size == 0:
                 # eof or similar
                 return 0
@@ -718,6 +721,9 @@ class Channel (object):
         self.lock.acquire()
         try:
             size = self._wait_for_send_window(size)
+            # Instead of returning 0 here when the channel is closed, we might
+            # want to raise EOFError or similar. Should we just do as in the
+            # send_all method, raise a socket.error?
             if size == 0:
                 # eof or similar
                 return 0