From 9e1d10b5f1250e33e3a4e9caa85e96307fdaea2b Mon Sep 17 00:00:00 2001 From: Allan Odgaard Date: Wed, 15 Aug 2012 15:58:38 +0200 Subject: [PATCH] Remove debug output MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit I was initially under the impression that no data should be sent past EOF but from testing it is clear that this assumption is incorrect. It’s unimportant though as we do handle the case (though we didn’t initially, and that’s why I put in the fprintf, to verify the situation I guarded for actually was happening). --- Frameworks/OakSystem/src/reader.cc | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/Frameworks/OakSystem/src/reader.cc b/Frameworks/OakSystem/src/reader.cc index 153cf672..e5051098 100644 --- a/Frameworks/OakSystem/src/reader.cc +++ b/Frameworks/OakSystem/src/reader.cc @@ -111,6 +111,7 @@ namespace io iterate(it, tmp) { + // required check as we may receive a post-EOF “bytes available” with ‘it->length == 0’ if(clients_consumed_eof.find(it->client_key) == clients_consumed_eof.end()) { std::map::iterator client = client_to_callback.find(it->client_key); @@ -119,10 +120,6 @@ namespace io if(it->length == 0) clients_consumed_eof.insert(it->client_key); } - else - { - fprintf(stderr, "*** got %zu bytes for client which already received EOF\n", it->length); - } delete it->bytes; } }