On 2011-07-05 14:26, jdrewsen wrote: > Hi, > > I believe the curl wrapper is ready for a review now. Is there > anything else in the review queue or can we schedule the curl wrapper now?
Robert Jacques has some changes to std.variant which are required for his changes to std.json which he was going to put up for review, but I guess that he hasn't had the time to finish that up yet, so we can't review them yet. std.path and std.log are on hold because their authors were going on vacation, and apparently std.benchmark needs a bit of further work on it now that it's become its own module, and Andrei hasn't had time to complete that yet. So, nothing is actually up for review just yet. That being the case, I think that we can review the curl wrapper. I guess that I can be the review manager for it, since someone needs to step up and take that role for it. How about we review it until the 25th and then reassess where it stands? That's a little over 2 weeks. If it needs more review at that point, we can extend the review time (or put it on the backburner if it turns out that you need more time to iron out changes and something else is ready for review by then). Or if it actually looks like it's been amply reviewed and updated, then we can set up a vote. Does that sound good to you? - Jonathan M Davis
