[Ns-bugs] [Bug 592] pointer to packet likely needed for RouteOutput() call

code@nsnam.ece.gatech.edu code at nsnam.ece.gatech.edu
Sun Jun 21 23:28:36 PDT 2009


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





--- Comment #3 from Tom Henderson <tomh at tomh.org>  2009-06-22 02:28:36 EDT ---
(In reply to comment #1)
> I have not looked at the details but it would be safer to make it Ptr<const
> Packet> to avoid weird interactions between the caller and callee. The callee
> can call Packet::Copy to get a non-const version if needed.
> 
> Feel free to ignore the above is you envision cases where it's important that
> the caller and callee share the same packet.
> 

I prefer to change this to Ptr<Packet>; callee is allowed to modify the packet
(such as an explicit source routing header).  If caller doesn't like this
behavior, can call Packet::Copy itself.  


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


More information about the Ns-bugs mailing list