Author: markt
Date: Thu Jan  4 21:51:15 2018
New Revision: 1820229

URL: http://svn.apache.org/viewvc?rev=1820229&view=rev
Log:
Deprecate unused code that has been removed in 9.0.x

Modified:
    tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/AbstractStream.java
    tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java
    tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Stream.java

Modified: tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/AbstractStream.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/AbstractStream.java?rev=1820229&r1=1820228&r2=1820229&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/AbstractStream.java 
(original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/AbstractStream.java Thu 
Jan  4 21:51:15 2018
@@ -147,5 +147,6 @@ abstract class AbstractStream {
 
     protected abstract int getWeight();
 
+    @Deprecated // Unused
     protected abstract void doNotifyAll();
 }

Modified: 
tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java?rev=1820229&r1=1820228&r2=1820229&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java 
(original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Http2UpgradeHandler.java 
Thu Jan  4 21:51:15 2018
@@ -816,8 +816,9 @@ public class Http2UpgradeHandler extends
 
 
     @Override
+    @Deprecated
     protected synchronized void doNotifyAll() {
-        this.notifyAll();
+        // NO-OP. Unused.
     }
 
 

Modified: tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Stream.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Stream.java?rev=1820229&r1=1820228&r2=1820229&view=diff
==============================================================================
--- tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Stream.java (original)
+++ tomcat/tc8.5.x/trunk/java/org/apache/coyote/http2/Stream.java Thu Jan  4 
21:51:15 2018
@@ -230,16 +230,9 @@ public class Stream extends AbstractStre
 
 
     @Override
+    @Deprecated
     protected synchronized void doNotifyAll() {
-        if (coyoteResponse.getWriteListener() == null) {
-            // Blocking IO so thread will be waiting. Release it.
-            // Use notifyAll() to be safe (should be unnecessary)
-            this.notifyAll();
-        } else {
-            if (streamOutputBuffer.isRegisteredForWrite()) {
-                coyoteResponse.action(ActionCode.DISPATCH_WRITE, null);
-            }
-        }
+        // NO-OP. Unused.
     }
 
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to