Question about souce codes "TcpEndpoint::acceptor"

[OrNot]

hi,there,
In the TcpEndpoint.cpp file,

AcceptorPtr
IceInternal::TcpEndpoint::acceptor(EndpointPtr& endp) const
{
TcpAcceptor* p = new TcpAcceptor(_instance, _host, _port);

------------- ?????-------------------------------------------
endp = new TcpEndpoint(_instance, _host, p->effectivePort(), _timeout, _compress);
---------------------------------------------------------


return p;
}


I can't understand the lines with "new" . why need such codes? Consideration for reference number of smart pointer?

sorry for this trivial question.
Thanks in advance.

[benoit]

Hi,

The "endp" argument is passed by reference and is used as a return value for the accept() method. The use of a smart pointer ensures that the caller won't have to take care of the deallocation of the TcpEndpoint object.

I hope this is clearer!

Benoit.

[OrNot]

hi, Benoit,
Sorry for my poor expression about my question. I mean, why need to allocate a new TcpEndpoint here? I traced into the code and found that the members of the new TcpEndpoint seem to have the same values as the one passed in by "EndpointPtr& endp" except the address and the ref. Why not use the original TcpEndoint in the accept() method ?
There must be some thing out of my understanding. Hope getting your help.

Best Regards---OrNot

[benoit]

I don't think it's exactly the same -- the port might be different for instance.

Benoit.

[OrNot]

Thank you Benoit.
Maybe the method "effectivePort()" is the key. I will test it further.


OrNot

[marc]

If you do not specify a port number, the operating system will choose one. endp returns an endpoint with that port number.

[OrNot]

Oh yeah!