[FFmpeg-soc] [PATCH] rtsp tunneling

Michael Niedermayer michaelni at gmx.at
Mon Jun 14 18:47:32 CEST 2010


On Mon, Jun 14, 2010 at 11:36:07AM +0300, Martin Storsjö wrote:
> On Fri, 11 Jun 2010, Michael Niedermayer wrote:
> 
> > On Wed, Jun 09, 2010 at 01:53:51PM -0400, Ronald S. Bultje wrote:
> > > 
> > > 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
> 
> In general, that sounds like a good solution.
> 
> For this particular case, the options that we would like to set are http 
> specific, and (with the current design) are to be set within HTTPContext, 
> that is allocated and set to URLContext->priv_data only in http_open (and 
> its size isn't even known outside of http.c. So that would require some 

add a URLProtocol.priv_data_size
this also might simplify existing code by moving allocation of priv_data
to a common place

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- 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/20100614/2f154049/attachment.pgp>


More information about the FFmpeg-soc mailing list