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