Re: [FFmpeg-devel] [PATCH] avformat/framehash: also print channel layout as a string

2016-11-05 Thread James Almer
On 11/5/2016 8:55 PM, Michael Niedermayer wrote: > On Wed, Sep 28, 2016 at 09:05:07PM -0300, James Almer wrote: >> This should be more useful for users since numerical values for channel >> layout can be confusing and unintuitive. >> >> Signed-off-by: James Almer > > LGTM > >

Re: [FFmpeg-devel] [PATCH] avformat/framehash: also print channel layout as a string

2016-11-05 Thread Michael Niedermayer
On Wed, Sep 28, 2016 at 09:05:07PM -0300, James Almer wrote: > This should be more useful for users since numerical values for channel > layout can be confusing and unintuitive. > > Signed-off-by: James Almer LGTM thx [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH] avformat/framehash: also print channel layout as a string

2016-09-28 Thread James Almer
This should be more useful for users since numerical values for channel layout can be confusing and unintuitive. Signed-off-by: James Almer --- libavformat/framehash.c | 3 +++ tests/ref/fate/8bps | 1 +