You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hc.apache.org by Oleg Kalnichevski <ol...@apache.org> on 2014/02/20 15:10:12 UTC

Re: svn commit: r1570138

On Thu, 2014-02-20 at 13:28 +0000, sebb wrote:
> >
> >      @Override
> > -    public synchronized void mark(final int readlimit) {
> > +    public void mark(final int readlimit) {
> 
> This is an unrelated change - and is it correct?
> 

Why do you think it is incorrect? Synchronized is completely pointless
here.


> s/wire.input/wire.output/
> 
> Ditto below.
> 

Corrected.

Oleg



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org