Re: [FFmpeg-devel] [PATCH] Replace old cookies with new cookies of the same name

2014-08-18 Thread Nicolas George
Le primidi 1er fructidor, an CCXXII, Micah Galizia a écrit : > Yes & no. I agree its not an ideal implementation (it actually was > mine to begin with) to just use a string full of cookies. But we can't > pass around complex structures through avopts, which is where we would > really see the benefi

Re: [FFmpeg-devel] [PATCH] Replace old cookies with new cookies of the same name

2014-08-18 Thread Michael Niedermayer
On Mon, Aug 18, 2014 at 07:59:11PM +1000, Micah Galizia wrote: > On Sun, Aug 17, 2014 at 9:29 PM, Nicolas George wrote: > > Le decadi 30 thermidor, an CCXXII, Micah Galizia a écrit : > >> When new cookie values (with the same name as an existing cookie) are > >> returned in an HLS stream, the curr

Re: [FFmpeg-devel] [PATCH] Replace old cookies with new cookies of the same name

2014-08-18 Thread Micah Galizia
On Sun, Aug 17, 2014 at 9:29 PM, Nicolas George wrote: > Le decadi 30 thermidor, an CCXXII, Micah Galizia a écrit : >> When new cookie values (with the same name as an existing cookie) are >> returned in an HLS stream, the current implementation will append the >> new cookie to the list of cookies

Re: [FFmpeg-devel] [PATCH] Replace old cookies with new cookies of the same name

2014-08-17 Thread Nicolas George
Le decadi 30 thermidor, an CCXXII, wm4 a écrit : > AFAIK this cookie string is exposed by AVOption API, and I use it for > setting cookies. And I don't feel like rewriting this code at all. I do not propose to break user-visible API. You confuse it with internal data structures. Regards, -- N

Re: [FFmpeg-devel] [PATCH] Replace old cookies with new cookies of the same name

2014-08-17 Thread wm4
On Sun, 17 Aug 2014 13:29:15 +0200 Nicolas George wrote: > Le decadi 30 thermidor, an CCXXII, Micah Galizia a écrit : > > When new cookie values (with the same name as an existing cookie) are > > returned in an HLS stream, the current implementation will append the > > new cookie to the list of c

Re: [FFmpeg-devel] [PATCH] Replace old cookies with new cookies of the same name

2014-08-17 Thread Nicolas George
Le decadi 30 thermidor, an CCXXII, Micah Galizia a écrit : > When new cookie values (with the same name as an existing cookie) are > returned in an HLS stream, the current implementation will append the > new cookie to the list of cookies. This causes FFMPEG to send multiple > cookie values for the

[FFmpeg-devel] [PATCH] Replace old cookies with new cookies of the same name

2014-08-17 Thread Micah Galizia
When new cookie values (with the same name as an existing cookie) are returned in an HLS stream, the current implementation will append the new cookie to the list of cookies. This causes FFMPEG to send multiple cookie values for the same cookie and in some cases exceed the http header size in some