[Ns-bugs] [Bug 271] PacketSocketFactory should be added in the Node base class, not in InternetStackHelper

[bugzilla-daemon@nsnam-www.ece.gatech.edu]

http://www.nsnam.org/bugzilla/show_bug.cgi?id=271





------- Comment #6 from mathieu.lacage at sophia.inria.fr  2008-08-05 13:58 -------
(In reply to comment #5)
> (In reply to comment #3)
> [...]
> > We don't care about any kind of compatibility with pre-1.0 releases. It is sad
> > that you were surprised but, clearly, we are not going to change this _again_
> > to surprise more people who now rely on an official release.
> 
> It is not about compatibility, it is about being logical.
> [...]

I understand that what currently happens is not what you expected but I
explained the rationale for that behavior so, please, don't try to argue for
"logic". There is logic. It just happens that you want another one.

> > 
> > I fail to see the kind of internal complexity you are talking about. If you
> > don't want to use the helpers, you have to perform object aggregation yourself. 
> 
> The above sentence is the kind of complexity I am talking about.
> 
> If I want to use a UDP socket, I don't need to manually add a UdpSocketFactory.
>  It is provided in the "internet stack".  Like wise, if I wanted to use a
> packet socket, it should be provided by the "node stack".  It just turns out
> that "node stack" == Node.

I understand what you want and I explained why it is not so right now. Let me
repeat: Node == a standalone object defined by src/node.h. Helper == class to
aggregate functionality to Node. PacketSocketFactory == functionality which can
be aggregated to a node. Hence, PacketSocketHelper == class to aggregate
PacketSocketFactory to node. 


-- 
Configure bugmail: http://www.nsnam.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.