On Fri, Jul 21, 2017 at 9:59 AM, VinÃcius dos Santos Oliveira via
Boost
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