Re: [FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

2014-07-14 Thread Lukasz Marek
On 14 July 2014 14:00, Nicolas George wrote: > Le sextidi 26 messidor, an CCXXII, Lukasz Marek a écrit : > > Nicolas, do you need more time or I can send pull request? > > Sorry, I am in the middle of preparing luggages. Can you give me 24 more > hours. > No problem, no hurry, just let me know w

Re: [FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

2014-07-14 Thread Nicolas George
Le sextidi 26 messidor, an CCXXII, Lukasz Marek a écrit : > Nicolas, do you need more time or I can send pull request? Sorry, I am in the middle of preparing luggages. Can you give me 24 more hours. Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

2014-07-14 Thread Lukasz Marek
On 11 July 2014 20:34, Nicolas George wrote: > Le tridi 23 messidor, an CCXXII, Lukasz Marek a écrit : > > Hi, any comments, if not I will sent pull request tomorrow including > > libssh and ftp implementations (with minor changes) > > Sorry, I did not follow the thread carefully enough. Could yo

Re: [FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

2014-07-11 Thread Lukasz Marek
On 11.07.2014 20:34, Nicolas George wrote: Le tridi 23 messidor, an CCXXII, Lukasz Marek a écrit : Hi, any comments, if not I will sent pull request tomorrow including libssh and ftp implementations (with minor changes) Sorry, I did not follow the thread carefully enough. Could you resend the

Re: [FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

2014-07-11 Thread Nicolas George
Le tridi 23 messidor, an CCXXII, Lukasz Marek a écrit : > Hi, any comments, if not I will sent pull request tomorrow including > libssh and ftp implementations (with minor changes) Sorry, I did not follow the thread carefully enough. Could you resend the updated series as a coherent whole once aga

Re: [FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

2014-07-11 Thread Lukasz Marek
On 11 July 2014 14:01, wm4 wrote: > On Fri, 11 Jul 2014 03:03:53 +0200 > Lukasz Marek wrote: > > > On 10.07.2014 01:36, Lukasz Marek wrote: > > > On 08.07.2014 14:49, Lukasz Marek wrote: > > >> On 7 July 2014 23:59, wm4 > >> All symbols in public headers should have the AV prefix. (I guess

Re: [FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

2014-07-11 Thread wm4
On Fri, 11 Jul 2014 03:03:53 +0200 Lukasz Marek wrote: > On 10.07.2014 01:36, Lukasz Marek wrote: > > On 08.07.2014 14:49, Lukasz Marek wrote: > >> On 7 July 2014 23:59, wm4 >> All symbols in public headers should have the AV prefix. (I guess > >> this > >> is why you wanted to use void*

Re: [FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

2014-07-10 Thread Lukasz Marek
On 10.07.2014 01:36, Lukasz Marek wrote: On 08.07.2014 14:49, Lukasz Marek wrote: On 7 July 2014 23:59, wm4 new patch attached. Hi, any comments, if not I will sent pull request tomorrow including libssh and ftp implementations (with minor changes) ___

Re: [FFmpeg-devel] [PATCH 1/6] lavf: add directory listing API

2014-07-09 Thread Lukasz Marek
On 08.07.2014 14:49, Lukasz Marek wrote: On 7 July 2014 23:59, wm4 new patch attached. TBH I cannot tell which patch I prefer the most. Last option I see would be to create separate context for listing dir, but it would just have one void* variable for URLContext. I will resent all other pac