Re: Add a constant to reflect --with-curlwrappers
| From: | Julien Pauli | Date: | Fri, 05 Apr 2013 14:14:54 +0000 |
| Subject: | Re: Add a constant to reflect --with-curlwrappers | ||
| References: | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-66986@lists.php.net to get a copy of this message | ||
On Fri, Apr 5, 2013 at 12:51 PM, Johannes Schlüter
<johannes@schlueters.de>wrote:
> On Fri, 2013-04-05 at 08:01 +0200, Pierre Joye wrote:
> > > stream_wrapper_unregister("http");
> > > stream_wrapper_register("http", "CurlStreamWrapper");
> > > and then stream_wrapper_restore("http") to go back to the core
> > streams.
> > >
> >
> > I wonder what one will do with open streams during the switches. That
> > can't go well.
>
> For open streams there should be no issue - they hold the pointer to
> their respective implementation.
>
> The issue I see is that libraries might change that for whatever reasons
> and not fix it up before passing control to some other library, thus
> creating a hardly debugable mess.
>
I'm feeling like we wont be able to make it stable for 5.5 final.
Why not extract the code to pecl, make it stable if possible, then use it
as a pecl extension ?
It could also be merged to ext/curl in some way , that could be another
option.
Julien.Pauli