Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread Michael Niedermayer
On Thu, Feb 08, 2018 at 06:21:06PM +0100, wm4 wrote: > On Thu, 8 Feb 2018 12:34:07 -0300 > James Almer wrote: > > > On 2/8/2018 7:25 AM, Rostislav Pehlivanov wrote: > > > On 8 February 2018 at 10:06, Nicolas George wrote: > > > > > >> Michael Niedermayer (2018-02-08): > > >>> +1 > > >> >

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread wm4
On Thu, 8 Feb 2018 12:34:07 -0300 James Almer wrote: > On 2/8/2018 7:25 AM, Rostislav Pehlivanov wrote: > > On 8 February 2018 at 10:06, Nicolas George wrote: > > > >> Michael Niedermayer (2018-02-08): > >>> +1 > >> > >> I agree too. > >> > >> And maybe, since we are reverting something,

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread James Almer
On 2/8/2018 1:16 PM, Rostislav Pehlivanov wrote: > On 8 February 2018 at 13:19, James Almer wrote: > >> On 2/8/2018 9:52 AM, wm4 wrote: >>> On Thu, 8 Feb 2018 10:29:11 + >>> Rostislav Pehlivanov wrote: >>> On 8 February 2018 at 01:08, Muhammad Faiz wrote: > On Thu, Feb 8, 2018

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread Rostislav Pehlivanov
On 8 February 2018 at 13:19, James Almer wrote: > On 2/8/2018 9:52 AM, wm4 wrote: > > On Thu, 8 Feb 2018 10:29:11 + > > Rostislav Pehlivanov wrote: > > > >> On 8 February 2018 at 01:08, Muhammad Faiz wrote: > >> > >>> On Thu, Feb 8, 2018 at 7:35 AM, James Almer wrote: > On 2/7/2018 9:

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread James Almer
On 2/8/2018 7:25 AM, Rostislav Pehlivanov wrote: > On 8 February 2018 at 10:06, Nicolas George wrote: > >> Michael Niedermayer (2018-02-08): >>> +1 >> >> I agree too. >> >> And maybe, since we are reverting something, revert the whole series. >> >> Regards, >> >> -- >> Nicolas George >> >>

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread James Almer
On 2/8/2018 9:52 AM, wm4 wrote: > On Thu, 8 Feb 2018 10:29:11 + > Rostislav Pehlivanov wrote: > >> On 8 February 2018 at 01:08, Muhammad Faiz wrote: >> >>> On Thu, Feb 8, 2018 at 7:35 AM, James Almer wrote: On 2/7/2018 9:32 PM, Michael Niedermayer wrote: > On Wed, Feb 07, 2018

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread wm4
On Thu, 8 Feb 2018 12:03:22 + Rostislav Pehlivanov wrote: > On 8 February 2018 at 10:35, Nicolas George wrote: > > > Rostislav Pehlivanov (2018-02-08): > > > We should fix what we have right now. The API looks good too except some > > > people here feel like they haven't bikeshedded enoug

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread wm4
On Thu, 8 Feb 2018 10:29:11 + Rostislav Pehlivanov wrote: > On 8 February 2018 at 01:08, Muhammad Faiz wrote: > > > On Thu, Feb 8, 2018 at 7:35 AM, James Almer wrote: > > > On 2/7/2018 9:32 PM, Michael Niedermayer wrote: > > >> On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wro

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread James Almer
On 2/8/2018 7:29 AM, Rostislav Pehlivanov wrote: > On 8 February 2018 at 01:08, Muhammad Faiz wrote: > >> On Thu, Feb 8, 2018 at 7:35 AM, James Almer wrote: >>> On 2/7/2018 9:32 PM, Michael Niedermayer wrote: On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: > Yes, my bad. I

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread Rostislav Pehlivanov
On 8 February 2018 at 10:35, Nicolas George wrote: > Rostislav Pehlivanov (2018-02-08): > > We should fix what we have right now. The API looks good too except some > > people here feel like they haven't bikeshedded enough and want to cause > > even more chaos. > > Do you realize that sentence ac

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread Nicolas George
Rostislav Pehlivanov (2018-02-08): > We should fix what we have right now. The API looks good too except some > people here feel like they haven't bikeshedded enough and want to cause > even more chaos. Do you realize that sentence accuses Michael, Muhammad and me of pursuing a personal power trip

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread Rostislav Pehlivanov
On 8 February 2018 at 10:06, Nicolas George wrote: > Michael Niedermayer (2018-02-08): > > +1 > > I agree too. > > And maybe, since we are reverting something, revert the whole series. > > Regards, > > -- > Nicolas George > > ___ > ffmpeg-devel mailin

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread Rostislav Pehlivanov
On 8 February 2018 at 01:08, Muhammad Faiz wrote: > On Thu, Feb 8, 2018 at 7:35 AM, James Almer wrote: > > On 2/7/2018 9:32 PM, Michael Niedermayer wrote: > >> On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: > >>> Yes, my bad. It looked like it worked initially (was more > >>> worr

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-08 Thread Nicolas George
Michael Niedermayer (2018-02-08): > +1 I agree too. And maybe, since we are reverting something, revert the whole series. Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.or

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Michael Niedermayer
On Wed, Feb 07, 2018 at 09:35:24PM -0300, James Almer wrote: > On 2/7/2018 9:32 PM, Michael Niedermayer wrote: > > On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: > >> Yes, my bad. It looked like it worked initially (was more > >> worried about getting a fix out quickly), but it didnt

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Muhammad Faiz
On Thu, Feb 8, 2018 at 7:35 AM, James Almer wrote: > On 2/7/2018 9:32 PM, Michael Niedermayer wrote: >> On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: >>> Yes, my bad. It looked like it worked initially (was more >>> worried about getting a fix out quickly), but it didnt. Try this o

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Michael Niedermayer
On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: > Yes, my bad. It looked like it worked initially (was more > worried about getting a fix out quickly), but it didnt. Try this one. > > --- > fftools/cmdutils.c | 104 > + > 1 file c

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread James Almer
On 2/7/2018 9:32 PM, Michael Niedermayer wrote: > On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: >> Yes, my bad. It looked like it worked initially (was more >> worried about getting a fix out quickly), but it didnt. Try this one. >> >> --- >> fftools/cmdutils.c | 104 >> ++

Re: [FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Michael Niedermayer
On Wed, Feb 07, 2018 at 09:07:39PM +, Josh de Kock wrote: > Yes, my bad. It looked like it worked initially (was more > worried about getting a fix out quickly), but it didnt. Try this one. > > --- > fftools/cmdutils.c | 104 > + > 1 file c

[FFmpeg-devel] [PATCH] cmdutils: fix printing of codecs

2018-02-07 Thread Josh de Kock
Yes, my bad. It looked like it worked initially (was more worried about getting a fix out quickly), but it didnt. Try this one. --- fftools/cmdutils.c | 104 + 1 file changed, 58 insertions(+), 46 deletions(-) diff --git a/fftools/cmdutils.c b/