[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