FYI,
looks like the changes when it, but svn seems to only see the
documentation changes. Anyway take a look at the class for the simple
change.
-- Jeanfrancois
jfarcand_at_dev.java.net wrote:
> Author: jfarcand
> Date: 2008-05-21 21:45:45+0000
> New Revision: 1169
>
> Modified:
> trunk/modules/grizzly/src/main/java/com/sun/grizzly/filter/ReadFilter.java
>
> Log:
> Fix for https://grizzly.dev.java.net/issues/show_bug.cgi?id=146
> ("ReadFillter should discard OP_WRITE by default")
>
> Discard OP_WRITE as they shouldn't not be handled by the ProtocolFilter
>
>
>
> Modified: trunk/modules/grizzly/src/main/java/com/sun/grizzly/filter/ReadFilter.java
> Url: https://grizzly.dev.java.net/source/browse/grizzly/trunk/modules/grizzly/src/main/java/com/sun/grizzly/filter/ReadFilter.java?view=diff&rev=1169&p1=trunk/modules/grizzly/src/main/java/com/sun/grizzly/filter/ReadFilter.java&p2=trunk/modules/grizzly/src/main/java/com/sun/grizzly/filter/ReadFilter.java&r1=1168&r2=1169
> ==============================================================================
> --- trunk/modules/grizzly/src/main/java/com/sun/grizzly/filter/ReadFilter.java (original)
> +++ trunk/modules/grizzly/src/main/java/com/sun/grizzly/filter/ReadFilter.java 2008-05-21 21:45:45+0000
> @@ -58,11 +58,13 @@
> import com.sun.grizzly.SelectorHandler;
>
> /**
> - * Simple ProtocolFilter implementation which read the available bytes
> - * and delegate the processing to the next ProtocolFilter in the ProtocolChain.
> - * If no bytes are available, no new ProtocolHandler will be a invoked and
> + * Simple {_at_link ProtocolFilter} implementation which read the available bytes
> + * and delegate the processing to the next {_at_link ProtocolFilter} in the {_at_link ProtocolChain}.
> + * If no bytes are available, no new {_at_link ProtocolFilter} will be a invoked and
> * the connection (SelectionKey) will be cancelled. This filter can be used
> * for both UDP (reveive) and TCP (read).
> + *
> + * Note that all ready OP_WRITE operations will be ignored.
> *
> * @author Jeanfrancois Arcand
> */
> @@ -86,7 +88,7 @@
>
> /**
> * Read available bytes and delegate the processing of them to the next
> - * ProtocolFilter in the ProtocolChain.
> + * {_at_link ProtocolFilter} in the {_at_link ProtocolChain}.
> * @return <tt>true</tt> if the next ProtocolFilter on the ProtocolChain
> * need to bve invoked.
> */
> @@ -234,7 +236,7 @@
> * this property is useful for protocol that needs to support pipelined
> * message requests as the ProtocolFilter are automatically re-executed,
> * avoiding the overhead of releasing the current Thread, registering
> - * back the SelectionKey to the SelectorHandler and waiting for a new
> + * back the SelectionKey to the {_at_link SelectorHandler} and waiting for a new
> * NIO event.
> *
> * Some protocols (like http) can get the http headers in one
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commits-unsubscribe_at_grizzly.dev.java.net
> For additional commands, e-mail: commits-help_at_grizzly.dev.java.net
>