[FFmpeg-soc] [PATCH] rtsp tunneling

[Michael Niedermayer]

On Wed, Jun 09, 2010 at 01:53:51PM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On Wed, Jun 9, 2010 at 5:28 AM, Martin Storsjö <martin at martin.st> wrote:
> > Ronald: There's still an arcitectural regression introduced by the http
> > delay open, that I'm a bit unsure about how to fix. url_fdopen (which is
> > invoked directly by url_fopen) copies the value of is_streamed to the
> > ByteIOContext directly after opening. In the http protocol, is_streamed is
> > 1 initially to indicate that seeking isn't supported, but is set to 0 if a
> > proper reply header is found. This used to be done already within
> > url_open(), but isn't anymore since the connection hasn't been opened yet.
> > So this effectively removes support for seeking in http sources for now.
> >
> > How do you suggest we fix this?
> 
> This, and many other regressions, are all caused because URL* combines
> allocation with opening, something that AVFormatContext/AVCodecContext
> separate. The proper thing to do is to separate it also. That breaks
> or adds API, which is bad, but maybe we just have to byte the bullet.
> It also means we have to separate close and free.
> 
> Michael, that OK with you, if it doesn't break existing API/apps?

the only thing needed for url_open() / url_open_protocol() is to
pass a allocated context and add a flag that indicates that the
context has already been allocated

then we need a function like avformat_alloc_context() for allocation

spliting free/close does not seem required

this leaves url_fopen() which iam not sure how to handle best

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20100611/c7ee42d1/attachment.pgp>