[FFmpeg-devel] [rfc] merge the applehttp-urlprotocol into main
Luca Barbato
lu_zero
Fri Mar 4 11:40:27 CET 2011
On 02/28/2011 02:57 PM, Martin Storsj? wrote:
> On Mon, 28 Feb 2011, Martin Storsj? wrote:
>
>> On Mon, 28 Feb 2011, Luca Barbato wrote:
>>
>>> On 02/28/2011 12:54 PM, Martin Storsj? wrote:
>>>> That might indeed be a better route. Although I don't think we currently
>>>> can handle registering a protocol that will catch any
>>>> applehttp+<whatever>://, at the moment we'd have to register the full
>>>> applehttp+<whatever> as protocol name.
>>>>
>>>> What would be a good way of handling this? Adding a flag to URLProtocol,
>>>> where a protocol can signal that it can work as a base protocol? That is,
>>>> if this flag is set, we'd try to match only the part of the URL scheme up
>>>> to the first + with this protocol name.
>>>
>>> might be useful go to this route and change the url splitter accordingly
>>> in the long run.
>>
>> I'm not sure the url splitter in itself should be changed - there's no
>> inherent meaning in using + as a separator. I'll try to prototype the
>> thing I suggested above, though.
>
> Implemented this in the attached patches. Do you think this looks like a
> sensible approach?
I'd queue them up and push tomorrow, after that I might merge my
protos[1] branch once cleaned up enough and switch rtsp to use the same
format for it.
Anybody against it? (I let a canonical week pass to hear more comments
on it)
lu
--
Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero
More information about the ffmpeg-devel
mailing list