curl-library
[PATCH] always-multi
Date: Wed, 26 Dec 2012 00:25:34 +0100 (CET)
Hi friends,
I'm happy to announce my first always-multi patch that run all test cases
error-free (well, to be honest I modified a few of them! ;-)). I am especially
pleased with the diffstat summary from this:
50 files changed, 418 insertions(+), 828 deletions(-)
... and then I've still added a file header. Also, what's not visualized in
these numbers is the simplified logic in several places due to the single
option being used.
There are a few downsides with this approach that I think aren't blocking it,
but are diffinately things we should work on post-merge. The primary one I can
think of is that since each easy perform now will use a multi handle
internally, we will use (much?) more memory. I'm considering some kind of
(internal) shortcut or something that will allow us to make a multi handle
that is somehow more targeted to use only one or very few easy handles. Could
possibly turn out useful for external use too...
My brief plan on the future of this:
- I'll run the test more and see that I don't spot any obvious problems that
will hinder a merge. If so, fix it.
- Merge this as one single commit, and then never look back again to the time
before this new order. This then makes us use this in the upcoming release,
which will use a bumped minor number to celebrate this (and possibly some
other things too but we'll see about them later...). The merge should be
possible to do before the end of the year, but if not it shouldn't be too
long into the new year until it'll go live.
- Release 7.29.0 on February 6th 2013!
Questions, comments and suggestions are welcome!
-- / daniel.haxx.se
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
- TEXT/x-diff attachment: always-multi.diff