[Ns-bugs] [Bug 148] Replace "std::string context" parameter with a more useful Config::Path parameter

bugzilla-daemon@nsnam-www.ece.gatech.edu bugzilla-daemon at nsnam-www.ece.gatech.edu
Wed Mar 26 07:33:54 PDT 2008


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





------- Comment #2 from mathieu.lacage at sophia.inria.fr  2008-03-26 10:33 -------
I discussed this at length with gustavo on irc: while I do understand why he
wants to do this, I would personally go another route:

1) keep the current std::string context string in callback signatures
2) provide a Config::Path class which can be constructed from a string (cannot
be converted back to a string) and which performs the string parsing for you to
allow you to call simple methods such as GetElementAsString (1) or
GetElementAsUint32 (2)

Basically, I would take gustavo's proposed code but I would not use it as the
default context in callback signatures.

So, to summarize, I think that gustavo's code is useful. The only question is
how we want to integrate it and although I disagree with gustavo on how it
should be integrated, I don't think it is that important so, if anyone agrees
with gustavo, feel free to apply the patch as-is.


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


More information about the Ns-bugs mailing list