[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