[Ffmpeg-devel] Re: [PATCH/RFC] http seeking

Ronald S. Bultje rbultje
Fri Jan 5 14:14:17 CET 2007


Hi Michael,

On Fri, 2007-01-05, Michael Niedermayer wrote:
> On Thu, Jan 04, 2007 at 08:10:33PM -0500, Ronald S. Bultje wrote:
> > On Fri, 2007-01-05, Michael Niedermayer wrote:
> > > On Thu, Jan 04, 2007 at 07:24:39PM -0500, Ronald S. Bultje wrote:
> > > > +static void http_close_cnx(URLContext *h)
> > > > +{
> > > > +    HTTPContext *s = h->priv_data;
> > > > +    url_close(s->hd);
> > > > +}
> > > 
> > > hmm, why not call url_close() everywhere instead? it would also be clearer
> > > IMHO
> > 
> > As opposite of http_open_cnx(). Basically just in case something new
> > ever gets added that needs to be undone in all places (open and seek)
> > closing the connection. I can remove it for now if you wish.
> 
> i do

New patch attached.

Cheers,
Ronald
-------------- next part --------------
A non-text attachment was scrubbed...
Name: d
Type: text/x-patch
Size: 7108 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070105/87876e09/attachment.bin>



More information about the ffmpeg-devel mailing list