[FFmpeg-devel] [PATCH] avformat/dv : change audio format endianess to BE

Michael Niedermayer michael at niedermayer.cc
Sat Nov 4 13:38:21 EET 2017

On Fri, Nov 03, 2017 at 05:44:28PM -0800, Lou Logan wrote:
> On Fri, Nov 3, 2017, at 04:49 PM, Michael Niedermayer wrote:
> > This does not explain why this change is done.
> > does it fix a bug?
> > does it make the code faster?
> ffmbc uses BE too, but I'm ignorant as to why it does.

If BE is the expected output by most DV users (is it ?) then this
should be mentioned in the commit message.

Otherwise most people looking at this commit in the future will not
have a clue why it was changed

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- 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/20171104/16259d2a/attachment.sig>

More information about the ffmpeg-devel mailing list