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 suppose
the 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 ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel