Completed a couple of TODOs.

This commit is contained in:
Simone Bordet 2013-07-25 16:40:32 +02:00
parent 1d77a139fd
commit 89977cdab1
1 changed files with 2 additions and 5 deletions

View File

@ -112,10 +112,7 @@ public class HttpReceiverOverHTTP extends HttpReceiver implements HttpParser.Res
parser.atEOF(); parser.atEOF();
parser.parseNext(BufferUtil.EMPTY_BUFFER); parser.parseNext(BufferUtil.EMPTY_BUFFER);
if (!responseFailure(new EOFException())) if (!responseFailure(new EOFException()))
{
// TODO: just shutdown here, or full close ?
getHttpChannel().getHttpConnection().close(); getHttpChannel().getHttpConnection().close();
}
} }
@Override @Override
@ -177,10 +174,10 @@ public class HttpReceiverOverHTTP extends HttpReceiver implements HttpParser.Res
{ {
HttpExchange exchange = getHttpExchange(); HttpExchange exchange = getHttpExchange();
if (exchange == null) if (exchange == null)
return false; // TODO: is it correct to return false here ? return false;
responseSuccess(exchange); responseSuccess(exchange);
return true; return false;
} }
@Override @Override