From 72827aa44d6848542be7e05d6a56727e0d1f67cc Mon Sep 17 00:00:00 2001
From: Greg Wilkins <gregw@intalio.com>
Date: Thu, 30 Aug 2012 08:44:19 +1000
Subject: [PATCH] jetty-9 TODOs

---
 .../org/eclipse/jetty/spdy/http/HttpTransportOverSPDY.java     | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/jetty-spdy/spdy-jetty-http/src/main/java/org/eclipse/jetty/spdy/http/HttpTransportOverSPDY.java b/jetty-spdy/spdy-jetty-http/src/main/java/org/eclipse/jetty/spdy/http/HttpTransportOverSPDY.java
index 5b95d9dbaa9..824481a5845 100644
--- a/jetty-spdy/spdy-jetty-http/src/main/java/org/eclipse/jetty/spdy/http/HttpTransportOverSPDY.java
+++ b/jetty-spdy/spdy-jetty-http/src/main/java/org/eclipse/jetty/spdy/http/HttpTransportOverSPDY.java
@@ -82,6 +82,7 @@ public class HttpTransportOverSPDY implements HttpTransport
         headers.put(HTTPSPDYHeader.STATUS.name(version), httpStatus.toString());
         LOG.debug("HTTP < {} {}", httpVersion, httpStatus);
 
+        // TODO merge the two Field classes into one
         HttpFields fields = info.getHttpFields();
         if (fields != null)
         {
@@ -107,12 +108,14 @@ public class HttpTransportOverSPDY implements HttpTransport
     public void send(ByteBuffer content, boolean lastContent) throws IOException
     {
         // Guard against a last 0 bytes write
+        // TODO work out if we can avoid double calls for lastContent==true
         if (stream.isClosed() && BufferUtil.isEmpty(content) && lastContent)
             return;
         stream.data(new ByteBufferDataInfo(content, lastContent));
     }
 
     @Override
+    // TODO move to channel ?
     public void completed()
     {
         LOG.debug("completed");