On Sunday, March 13, 2016 at 2:38:05 PM UTC-5, Manu Sánchez wrote:
Hi,
I have been checking both the library (I have not used Fit for eight months or so, so I had to be sure I didn't miss anything) and the review the whole week, but I haven't had enough time to formalize a review.
My opinion is, as others stated, that Fit is useful (function composition, lifting, tools, etc) and has quality enough to be incorporated into Boost. My only concern, first noticed by Louis Dionne IIRC, is that constexpr lambda objects are not constexpr-evaluated. This may be obvious for experts like us, but might be a source of misconception for newcomers. Until C++17 fixes this with constexpr lambdas a documentation note clarifying it should be enough. Of course such a detail is not a reason for rejection, my opinion is that Fit should be accepted unconditionally.
Thanks Manu for the support, and I'm glad you've found the library useful. Paul
Hope it helps.
El dom., 13 de marzo de 2016 17:38, Vicente J. Botet Escriba < vicent...@wanadoo.fr javascript:> escribió:
Le 13/03/2016 15:24, Peter Dimov a écrit :
Vicente J. Botet Escriba wrote:
The formal review of Paul Fultz II's Fit library ends today, 13th March. If you had not time to do the review and you plan to do it, please let us know so that we see if the review can be extended.
I do not have time at the moment for a full review, but I would like to cast my vote in favor of acceptance.
I looked at Fit at the time the review was announced as forthcoming (a few weeks ago) and it gave me the impression of a potentially very useful library that would be a worthwhile addition to Boost.
In my opinion, none of the changes that have been requested so far, and which Paul have committed to address, require the library to be rejected and re-reviewed later.
Thanks Peter for sharing your point of view.
Vicente
_______________________________________________ Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
_______________________________________________ Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost