[Review Results] Named Parameters Library
The Named Parameters Library has been accepted into Boost.
Congratulations to the authors, David Abrahams and Daniel Wallin.
The vast majority of reviews were positive, although most reviewers
requested additional documentation. Here is a list of the most
important/commonly-requested changes or additions to the library:
- More documentation! Specifically,
+ Automatic overload generation should go into the tutorial
+ Document operator,
+ Performance data (what's the cost?)
+ Reference section
+ "Best practices" section.
- Add a metafunction for getting the type of a parameter, e.g.,
param_type
On Sun, 5 Dec 2004 15:10:36 -0500, Doug Gregor
The Named Parameters Library has been accepted into Boost. Congratulations to the authors, David Abrahams and Daniel Wallin.
The vast majority of reviews were positive, although most reviewers requested additional documentation. Here is a list of the most important/commonly-requested changes or additions to the library:
- More documentation! Specifically, + Automatic overload generation should go into the tutorial + Document operator, + Performance data (what's the cost?) + Reference section + "Best practices" section.
- Add a metafunction for getting the type of a parameter, e.g., param_type
::type. - Add test cases for the automatic overload generation macros.
- Fix the BOOST_NAMED_PARAMS_MAX_ARITY bug; Cromwell Enage provided a solution.
There was also some debate about the syntax and implementation of the optional parameters. See for example this post from Dave: http://article.gmane.org/gmane.comp.lib.boost.devel/113589 What is the final word on this? -- Caleb Epstein caleb dot epstein at gmail dot com
On Dec 6, 2004, at 8:57 AM, Caleb Epstein wrote:
On Sun, 5 Dec 2004 15:10:36 -0500, Doug Gregor
wrote: The Named Parameters Library has been accepted into Boost. Congratulations to the authors, David Abrahams and Daniel Wallin.
The vast majority of reviews were positive, although most reviewers requested additional documentation. Here is a list of the most important/commonly-requested changes or additions to the library:
- More documentation! Specifically, + Automatic overload generation should go into the tutorial + Document operator, + Performance data (what's the cost?) + Reference section + "Best practices" section.
- Add a metafunction for getting the type of a parameter, e.g., param_type
::type. - Add test cases for the automatic overload generation macros.
- Fix the BOOST_NAMED_PARAMS_MAX_ARITY bug; Cromwell Enage provided a solution.
There was also some debate about the syntax and implementation of the optional parameters. See for example this post from Dave:
http://article.gmane.org/gmane.comp.lib.boost.devel/113589
What is the final word on this?
The syntax will stay as it is. Some of the options proposed by Joel are not workable (e.g., they cannot result in a compile-time error for params[name] when a "name" argument was not passed) and the "superior" option from Dave doesn't handle the common case where the type of the argument passed by the user differs from the type of the default value. The only unresolved syntax argument is the "f(foo = "wibble", bar = 17)" vs. "f((foo = "wibble", bar = 17))" question, where the former requires lots of overloads (some people hate that) but the latter requires the odd double-paren syntax (other people hate that). We won't reach a decision on this, so there is no need to require changes to the library. Doug
Caleb Epstein wrote:
On Sun, 5 Dec 2004 15:10:36 -0500, Doug Gregor
wrote: The Named Parameters Library has been accepted into Boost. Congratulations to the authors, David Abrahams and Daniel Wallin.
The vast majority of reviews were positive, although most reviewers requested additional documentation. Here is a list of the most important/commonly-requested changes or additions to the library:
- More documentation! Specifically, + Automatic overload generation should go into the tutorial + Document operator, + Performance data (what's the cost?) + Reference section + "Best practices" section.
- Add a metafunction for getting the type of a parameter, e.g., param_type
::type. - Add test cases for the automatic overload generation macros.
- Fix the BOOST_NAMED_PARAMS_MAX_ARITY bug; Cromwell Enage provided a solution.
There was also some debate about the syntax and implementation of the optional parameters. See for example this post from Dave:
http://article.gmane.org/gmane.comp.lib.boost.devel/113589
What is the final word on this?
http://lists.boost.org/MailArchives/boost/msg74967.php http://lists.boost.org/MailArchives/boost/msg74977.php I don't think I've seen any syntax suggestions that improve on what we've implemented. -- Dave Abrahams Boost Consulting http://www.boost-consulting.com
participants (3)
-
Caleb Epstein
-
David Abrahams
-
Doug Gregor