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ö <[email protected]> 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
signature.asc
Description: Digital signature
_______________________________________________ FFmpeg-soc mailing list [email protected] https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
