
All, The review of Phoenix V3 ended on March 2nd. We've got quite some discussion, even if we hoped for more. Overall, 3 people have confirmed that the Phoenix rewrite addresses all issues raised during the original review back in 2008. Nobody raised any concerns or serious issues. So here is the result of the mini-review: Boost.Phoenix is ready to be moved to the Boost SVN trunk! All other discussions were about minor inconsistencies, which Thomas has already fixed by now. There are a couple of suggestions related to the documentation, which he agreed to address as soon as possible: 1) The Phoenix documentation should have a "What's New" section 2) Compile time is often a concern with libraries that make heavy use of expression templates. Adding a compile time test would be ideal. This should be easily invoked and results from various compilers reported within the documentation. The only thing, which needs to be decided (and which has not been discussed yet) is how the migration from Boost.Bind, Boost.Lambda, and Phoenix V2 has to be organized, ensuring minimal disruption for the users of those libraries. I see several possible ways of doing so, but would like to open up the discussion. Any suggestions and comments are welcome. With high likelihood this migration process will be implemented using forwarding headers. Therefore, this point has not to be resolved before moving the main code base to trunk. On the other hand, this migration strategy needs to be in place before merging to the release branch. -------------------- This mini-review wraps up the review process of Boost.Phoenix. Congratulations to Thomas for this version of the excellent library, which will contribute to the unification of the functional landscape in Boost. At the same time I would like to mention a couple of names which are related to the development of Phoenix since its inception, and without those Boost.Phoenix wouldn't have happened. - Joel de Guzman, who wrote the initial V1 and since then was actively involved and driving the development of Boost.Phoenix - Dan Marsden, who is the main developer (with Joel) of Phoenix V2 - Angus Leeming and Daniel Wallin who contributed to V2 - Eric Niebler, the author of Boost.Proto, who contributed to V2 and was actively driving and supporting the development of V3, he also mentored Thomas Heller's GSoC project. Regards Hartmut Review Manager --------------- http://boost-spirit.com

On 3/9/2011 8:36 PM, Hartmut Kaiser wrote:
All,
The review of Phoenix V3 ended on March 2nd. We've got quite some discussion, even if we hoped for more. Overall, 3 people have confirmed that the Phoenix rewrite addresses all issues raised during the original review back in 2008. Nobody raised any concerns or serious issues.
So here is the result of the mini-review: Boost.Phoenix is ready to be moved to the Boost SVN trunk! <snip>
Woo-hoo! Thanks to everybody who participated in the review. And congrats to Thomas! -- Eric Niebler BoostPro Computing http://www.boostpro.com

On 03/09/2011 05:36 AM, Hartmut Kaiser wrote:
So here is the result of the mini-review: Boost.Phoenix is ready to be moved to the Boost SVN trunk!
Amazing job Thomas! To all the Phoenix contributors and especially Joel de Guzman, thank you for this great library. -- Michael Caisse Object Modeling Designs www.objectmodelingdesigns.com

On 3/9/2011 11:47 PM, Michael Caisse wrote:
On 03/09/2011 05:36 AM, Hartmut Kaiser wrote:
So here is the result of the mini-review: Boost.Phoenix is ready to be moved to the Boost SVN trunk!
Amazing job Thomas!
To all the Phoenix contributors and especially Joel de Guzman, thank you for this great library.
Thank you, Michael. Of course a big thanks to Hartmut Kaiser who volunteered to be review manager for this mini-review and the previous (2008) Phoenix review. Thank you, Hartmut! Regards, -- Joel de Guzman http://www.boostpro.com http://boost-spirit.com

On Wed, Mar 9, 2011 at 4:30 PM, Joel de Guzman <joel@boost-consulting.com> wrote:
On 3/9/2011 11:47 PM, Michael Caisse wrote:
On 03/09/2011 05:36 AM, Hartmut Kaiser wrote:
So here is the result of the mini-review: Boost.Phoenix is ready to be moved to the Boost SVN trunk!
Amazing job Thomas!
To all the Phoenix contributors and especially Joel de Guzman, thank you for this great library.
Thank you, Michael. Of course a big thanks to Hartmut Kaiser who volunteered to be review manager for this mini-review and the previous (2008) Phoenix review. Thank you, Hartmut!
Congratulations Thomas on such a fantastic job getting Phoenix 3 out the door. And thanks to all the Phoenix contributors, Hartmut as review manager, and all the reviewers. Cheers Dan

On Wed, Mar 9, 2011 at 2:36 PM, Hartmut Kaiser <hartmut.kaiser@gmail.com> wrote:
This mini-review wraps up the review process of Boost.Phoenix. Congratulations to Thomas for this version of the excellent library, which will contribute to the unification of the functional landscape in Boost. At the same time I would like to mention a couple of names which are related to the development of Phoenix since its inception, and without those Boost.Phoenix wouldn't have happened.
- Joel de Guzman, who wrote the initial V1 and since then was actively involved and driving the development of Boost.Phoenix - Dan Marsden, who is the main developer (with Joel) of Phoenix V2 - Angus Leeming and Daniel Wallin who contributed to V2 - Eric Niebler, the author of Boost.Proto, who contributed to V2 and was actively driving and supporting the development of V3, he also mentored Thomas Heller's GSoC project.
Thanks! I am very happy :) I thank everyone who was/is involved in the development! It wouldn't have been possible without you :)
participants (6)
-
Dan Marsden
-
Eric Niebler
-
Hartmut Kaiser
-
Joel de Guzman
-
Michael Caisse
-
Thomas Heller