[Ns-bugs] [Bug 167] helper API alignment patch

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

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





------- Comment #3 from tomh at tomh.org  2008-04-13 23:31 -------
(In reply to comment #2)
> The patch looks good. Reviewing it made me think about another the InstallAll
> methods you have added. 

I did not add this method (was previously "EnableAll").

> Maybe we should consider using the All postfix
> consistently across this API. For example, CsmaHelper::EnableAscii (void) could
> be renamed...
> 
> Do you want me to report this in another bug report ?
> 

If you mean EnableAllAscii (), yes, that would seem to be bring more alignment
with InstallAll().


-- 
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.