directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Seelmann <seelm...@apache.org>
Subject Re: svn commit: r1000823 - /directory/apacheds/trunk/protocol-shared/src/main/java/org/apache/directory/server/protocol/shared/transport/Transport.java
Date Fri, 24 Sep 2010 12:13:11 GMT
The acceptor is created lazily in TcpTransport and UdpTransport. There
is no way to set it from outside. So let's remove the setAcceptor()
method from the Transport interface.


On Fri, Sep 24, 2010 at 2:04 PM, Stefan Seelmann <seelmann@apache.org> wrote:
> On Fri, Sep 24, 2010 at 1:52 PM,  <felixk@apache.org> wrote:
>> Interesting what one can find when eliminating javadoc formatting issues... And astonishing
enough it's working. That's what I call a stable and fault-tolerant application :-)
>
> Indeed.
>
>> --- directory/apacheds/trunk/protocol-shared/src/main/java/org/apache/directory/server/protocol/shared/transport/Transport.java
(original)
>> +++ directory/apacheds/trunk/protocol-shared/src/main/java/org/apache/directory/server/protocol/shared/transport/Transport.java
Fri Sep 24 11:52:07 2010
>> @@ -66,7 +66,7 @@ public interface Transport
>>     /**
>>      * Set the IoAcceptor
>>      * @param acceptor The IoAcceptor to set
>> -     *
>> +     */
>>     void setAcceptor ( IoAcceptor acceptor );
>
> Hm, maybe that was intended, because now the build fails as the
> classes TcpTransport and UdpTransport don't implement the
> setAcceptor() method. I think we should remove the setAcceptor from
> the interface? Anyone?
>
> Kind Regards,
> Stefan
>

Mime
View raw message