[Ns-bugs] [Bug 491] It is painful to enable all checksums

[code@nsnam.ece.gatech.edu]

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





--- Comment #4 from Mathieu Lacage <mathieu.lacage at sophia.inria.fr>  2009-04-27 02:58:11 EDT ---
(In reply to comment #2)

> I don't think that adding a calChecksum to InternetStackHelper is the right
> thing to do.  There is no law that says that a script must ever instantiate an
> internet stack helper.  We need something that turns on checksums irrespective
> of what level script one is using.  
> 
> Like I said, I think there should be a global enable-checksums option that
> trumps the individual checksum attributes and just turns on all checksums in
> the system.  All checksum enabling code should respect this global value.
> 
> I would like to fix this tomorrow to get emu-ping working.  How about adding a
> GlobalValue in the internet stack module?

Per your above argument that we should not have an option in
InternetStackHelper, the global value should not be in internet-stack.

Available options:
1) make this an attribute of the Node object
2) make this a global value defined in src/node/node.cc
3) define this global value somewhere else.


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