[FFmpeg-devel] [PATCH 5/6] libavformat/mxf: add dnxhr codec ul

[Tomas Härdin]

On Mon, 2016-07-04 at 18:07 -0700, Mark Reid wrote:
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 0affca9..8f2f10a 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -1098,6 +1098,10 @@ static int mxf_match_uid(const UID key, const UID uid, int len)
>  static const MXFCodecUL *mxf_get_codec_ul(const MXFCodecUL *uls, UID *uid)
>  {
>      while (uls->uid[0]) {
> +        /* match version byte for dnxhr */
> +        if (uls->id == AV_CODEC_ID_DNXHR && !memcmp(uls->uid, *uid, uls->matching_len))
> +            break;
> +

Looks OK

I wonder if other codecs need this kind of special treatment. The
version byte isn't supposed to matter.. If any more pop up then we
could generalize this - for now this is good enough I suppose

/Tomas