[review] BOOST_SCOPE_EXIT review results
Subject: [review] BOOST_SCOPE_EXIT review results
Newsgroups: gmane.comp.lib.boost.announce
Date: 2008-05-07 19:22:58 GMT
Here are the review results for the Scope Exit library, submitted by Alexander Nasonov. I would be remiss if I didn't begin these results with an apology and a thank you. The review wizards are sincerely sorry that the result of this review took so long to be presented, and very grateful that Alexander has been so patient and according to a post on the forum has even been working to improve the library in the interim when time allowed. The following people contributed to the review. Kim Barrett, Steven Wantanabe, Oleg Abrosimov, Andrey Semashev, Peder Holt, Johan Nilsson, Maxim Yanchenko, Pavel Vozenilek, Aaron LaFramboise, Ben Kuppers, Goran Mitrovic, Tom Brinkman, Martin Wille, Joseph Gauterin, Matt Gruenke, Dave Abrahams, Christian Holmquist, Mathais Gaunard, Michael Marcin, Daniel Wallin, Zach Laine, Sid Sacek, Wang Yun, and Ilya Sokolov After carefully considering the review comments, I'm pleased to announce that the Scope Exit library is accepted into Boost. As is almost always the case, there are some notes for improvement for this library, and in this case there is even a suggestion for a substantial future restructuring. Let's start with the big one first. As it is currently implemented, SCOPE_EXIT suggests a method for creating a general closure mechanism as a library. This is a useful enough idea that many of the reviewers strongly encouraged Alexander to do it, and provided ideas for implementation details. Alexander should continue to work on this, and when it is ready for(Continue reading)
To avoid further delays, here is the short version of the review result:
Proto got 16 reviews (That's well above average! Thanks to all for
participating), out of which 15 have been positive. The one 'No' essentially
stated that the library, even if doubtless very useful, shouldn't be
accepted yet, mainly because there has to be done more work in the area of
formalizing the problem domain.
In my opinion this is an overwhelming result and I'm whole heartily agreeing
with the majority of the voters.
Proto is accepted as a Boost library.
The discussions have been very detailed and there have been raised a lot of
(mostly minor) issues, but Eric is aware of the problems and is working on
solving them. I'll try to put together a summary as soon as possible.
Regards Hartmut
Review Manager
_______________________________________________
Unsubscribe & other changes:
4) John Phillips raised the issue of finding these tools - making
sure that
they don't get lost amongst all the other special functions and
"stuff" in
Boost.Math. JM and Johan have agreed to work together to ensure this
doesn't happen.
5) There were a couple of suggestions for a convenience function for
installing the facets in a locale/iostream: Johan agreed to add this.
6) There was +1 vote for manipulators rather than flags to the facet
constructors - further investigation suggested this was problematic in a
header only library - so the suggestion was dropped.
7) There was one comment about the library not using quickbook for
documentation: ideally it would be nice to see this fixed so that the
docs
integrate seamlessly with the rest of Boost.Math - but see #4 as well.
Many thanks to Johan for making these utilities available.
John Maddock.
Review Manager for the Floating Point Utilities.
_______________________________________________
Unsubscribe & other changes:
RSS Feed