On Sun, Aug 07, 2016 at 03:49:34PM +0200, Tomas Härdin wrote:
> 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 supposethe patch adding AV_CODEC_ID_DNXHR (which this one depends upon) has been superseded and it uses the normal DNXHD codec id unless iam mixing something up here [...] -- 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
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
