On 10/03/2011 10:27, Konstantin Kolinko wrote: > Something is wrong with this commit: > > the new SocketWithOptionsProcessor#status field is always null. > > It is assigned by the constructor and is never changed afterwards. > The constructor is called as new SocketWithOptionsProcessor(wrapper, > null), always passing null for the value.
That is me copying and pasting from SocketProcessor. Since the status is always null, the code can be simplified. Once the status is non-null it will switch to using the SocketProcessor. I'll make the changes shortly. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org