Re: [boost] [Fit] formal review ends today, 13th March.

Subject: Re: [boost] [Fit] formal review ends today, 13th March.
From: Paul Fultz II (pfultz2_at_[hidden])
Date: 2016-03-13 17:35:54

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..._at_[hidden] <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
>
>