[FFmpeg-devel] [PATCH] avformat/framehash: print channel layout as a string instead of a hex value

[Michael Niedermayer]

On Wed, Sep 28, 2016 at 08:18:20PM -0300, James Almer wrote:
> On 9/28/2016 8:09 PM, James Almer wrote:
> > On 9/28/2016 7:35 PM, Michael Niedermayer wrote:
> >> On Wed, Sep 28, 2016 at 07:23:37PM -0300, James Almer wrote:
> >>> Numerical values for channel layout can be confusing and unintuitive, especially
> >>> when no channel count is printed.
> >>
> >> doesnt this break the format in libavformat/hashenc.c
> >> for format_version=2 ?
> > 
> > Yeah, it would break both versions 1 and 2, assuming anything tries to
> > parse the hex value for some purpose, as both are expected to have it.
> > And for that matter it would also break parsing framecrc in a similar
> > fashion.
> 
> Then again, framecrc doesn't report any kind of version, so a standardized
> output shouldn't be expected.
> 
> Ok if i make this version 3 for framehash/framemd5 then? I can also keep
> the hex value, so something like
> 
> #channel_layout 0: 0x3 (stereo)
> 
> would be printed.

maybe a
 #channel_layout 0: 0x3
+#channel_layout_name 0: stereo

would be within the existing version syntax


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160929/24f73826/attachment.sig>