Bug 62 - naming of methods in DefaultValue
naming of methods in DefaultValue
Status: RESOLVED INVALID
Product: ns-3
Classification: Unclassified
Component: core
pre-release
Mac Intel All
: P5 minor
Assigned To: ns-bugs
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-08-06 00:48 EDT by Tom Henderson
Modified: 2008-04-15 11:39 EDT (History)
0 users

See Also:


Attachments
rename Bind to DefaultValue::Bind (13.17 KB, application/octet-stream)
2007-08-08 08:05 EDT, Mathieu Lacage
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Henderson 2007-08-06 00:48:09 EDT
This is a carryover from the global routing discussion.
>
> >> - along the same lines, Bind() is kind of an oddball in the present
> >> system.  We do NodeList::Begin (), Simulator::Run (),
> >> CommandLine::Parse (); but simply Bind ().  This isn't consistent.
> >> 
> >> (note:  the choice of "bind()" was due to ns-2's methods of the same name)
> >> 

> >> Perhaps it would be better to do something like,
> >> 
> >> Configurator::Set ("DoStaticRouting", "true");
> >> 
> >> if (Configurator::IsEqual ("DoStaticRouting", "true"))
> >>    {
> >>    }
>
>Maybe it would make sense to try to stick the ns-2 naming scheme and use
>DefaultValue::Bind and DefaultValue::IsBound ?
Comment 1 Mathieu Lacage 2007-08-08 07:18:23 EDT
Any opposition to DefaultValue::Bind and DefaultValue::IsBound ?
Comment 2 Mathieu Lacage 2007-08-08 08:05:00 EDT
Created attachment 35 [details]
rename Bind to DefaultValue::Bind
Comment 3 Mathieu Lacage 2007-08-08 08:05:36 EDT
I did not implement IsBound since this would have required a few changes in a lot of places and we do not need it right away.
Comment 4 Mathieu Lacage 2008-04-15 11:39:43 EDT
code not there anymore.