[FFmpeg-devel] [PATCH 1/4] libavformat/movenc: add dnxhr compatibility for apple players
michael at niedermayer.cc
Tue Jul 19 03:11:13 EEST 2016
On Mon, Jul 18, 2016 at 04:37:49PM -0700, Mark Reid wrote:
> On Sun, Jul 17, 2016 at 4:35 AM, Michael Niedermayer
> <michael at niedermayer.cc> wrote:
> > On Sat, Jul 16, 2016 at 07:37:36PM -0700, Mark Reid wrote:
> >> ---
> >> libavcodec/dnxhddec.c | 3 +++
> >> libavformat/movenc.c | 19 ++++++++++++++-----
> >> 2 files changed, 17 insertions(+), 5 deletions(-)
> > this should be split
> > a single patch changing both libs is a bit suspicous API wise
> >> diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c
> >> index 5b60326..4c6f708 100644
> >> --- a/libavcodec/dnxhddec.c
> >> +++ b/libavcodec/dnxhddec.c
> >> @@ -204,6 +204,9 @@ static int dnxhd_decode_header(DNXHDContext *ctx, AVFrame *frame,
> >> }
> >> cid = AV_RB32(buf + 0x28);
> >> + if (cid >= 1270 && cid <= 1274)
> >> + ctx->avctx->codec_tag = MKTAG('A','V','d','h');
> > codec_tag is documented as:
> > * - decoding: Set by user, will be converted to uppercase by libavcodec during init.
> > this sets it in the decoder by libavcodec though
> okay, in that case I'll have to remove the same thing from the encoder patch.
> I saw some other codecs doing it so I assumed it was okay.
several encoders set codec_tag, and the docs are a bit unclear about
* - encoding: Set by user, if not then the default based on codec_id will be used.
but i think no decoder sets it
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
You can kill me, but you cannot change the truth.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel