Re: [boost] [review][beast] Beast Review Results

Subject: Re: [boost] [review][beast] Beast Review Results
From: Vinnie Falco (vinnie.falco_at_[hidden])
Date: 2017-07-21 17:03:04

On Fri, Jul 21, 2017 at 9:59 AM, Vinícius dos Santos Oliveira via
Boost <boost_at_[hidden]> wrote:
> People review frozen targets, not moving targets.
>
> Unfortunate, indeed, but I wanted to add this small note (it's not like
> it's reviewer's fault for not having added feedback on the Beast v2 which
> wasn't the thing being reviewed).

That is definitely true, but I was asking not in the context of the
formal review but in the context of simply getting feedback from a
stakeholder. Even if Beast was not accepted, Bjorn's feedback is still
valuable because existing users benefit from library improvements.

I would still like interested parties to review the design of
chunking. Although Beast has already been ACCEPTED to Boost without
conditions, the earliest version in which Beast can be merged is
1.66.0 in December, so there is plenty of time for me to make
adjustments - I consider all feedback and as mentioned before I am
responsive to issues. If you want to make a difference in Beast, or if
you want to make sure Beast doesn't have an interface design flaw that
would hinder its usage in your Boost.Http library, then your
participation is vital.

Regards