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 2008/07/05 14:37:17 UTC
Re: svn commit: r674186
On Sat, 2008-07-05 at 13:29 +0100, sebb wrote:
> On 05/07/2008, olegk@apache.org <ol...@apache.org> wrote:
> > Author: olegk
> > Date: Sat Jul 5 05:18:54 2008
> > New Revision: 674186
> >
...
> >
> > Modified: httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/conn/tsccm/BasicPoolEntryRef.java
> > URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/conn/tsccm/BasicPoolEntryRef.java?rev=674186&r1=674185&r2=674186&view=diff
> > ==============================================================================
> > --- httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/conn/tsccm/BasicPoolEntryRef.java (original)
> > +++ httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/conn/tsccm/BasicPoolEntryRef.java Sat Jul 5 05:18:54 2008
> > @@ -76,18 +76,5 @@
> > return this.route;
> > }
> >
> > -
> > - /* *
> > - * Indicates whether this reference is still valid.
> > - *
> > - * @return <code>true</code> if the pool entry is still referenced,
> > - * <code>false</code> otherwise
> > - * /
> > - public final boolean isValid() {
> > - //@@@ better sematics: allow explicit invalidation
> > - return (super.get() != null);
> > - }
> > - */
> > -
>
> Was the removal of that method intentional?
>
> It's not mentioned in the log message...
>
Sebastian,
This method was commented out. I thought it was pointless to keep it.
Oleg
> > } // class BasicPoolEntryRef
> >
> >
> >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> For additional commands, e-mail: dev-help@hc.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org
Re: svn commit: r674186
Posted by sebb <se...@gmail.com>.
On 05/07/2008, Oleg Kalnichevski <ol...@apache.org> wrote:
> On Sat, 2008-07-05 at 13:29 +0100, sebb wrote:
> > On 05/07/2008, olegk@apache.org <ol...@apache.org> wrote:
> > > Author: olegk
> > > Date: Sat Jul 5 05:18:54 2008
> > > New Revision: 674186
> > >
>
> ...
>
> > >
> > > Modified: httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/conn/tsccm/BasicPoolEntryRef.java
> > > URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/conn/tsccm/BasicPoolEntryRef.java?rev=674186&r1=674185&r2=674186&view=diff
> > > ==============================================================================
> > > --- httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/conn/tsccm/BasicPoolEntryRef.java (original)
> > > +++ httpcomponents/httpclient/trunk/module-client/src/main/java/org/apache/http/impl/conn/tsccm/BasicPoolEntryRef.java Sat Jul 5 05:18:54 2008
> > > @@ -76,18 +76,5 @@
> > > return this.route;
> > > }
> > >
> > > -
> > > - /* *
> > > - * Indicates whether this reference is still valid.
> > > - *
> > > - * @return <code>true</code> if the pool entry is still referenced,
> > > - * <code>false</code> otherwise
> > > - * /
> > > - public final boolean isValid() {
> > > - //@@@ better sematics: allow explicit invalidation
> > > - return (super.get() != null);
> > > - }
> > > - */
> > > -
> >
> > Was the removal of that method intentional?
> >
> > It's not mentioned in the log message...
> >
>
> Sebastian,
>
> This method was commented out. I thought it was pointless to keep it.
Sorry, my mistake.
I'd not noticed the subtle commenting out.
> Oleg
>
>
>
> > > } // class BasicPoolEntryRef
> > >
> > >
> > >
> > >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> > For additional commands, e-mail: dev-help@hc.apache.org
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> For additional commands, e-mail: dev-help@hc.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org