On Wed, Sep 21, 2016 at 12:26:48PM +0200, Michael Niedermayer wrote: > On Tue, Sep 20, 2016 at 11:20:48PM +0200, Clément Bœsch wrote: > > From: Clément Bœsch <clem...@stupeflix.com> > > > > This commit is largely based on commit 15e84ed3 from Anton Khirnov > > <an...@khirnov.net> which was previously skipped in bbf5ef9d. > > > > There are still a bunch of things raising codecpar related warnings that > > need fixing, such as: > > - the use of codec->debug in the interactive debug mode > > - read_ffserver_streams(): it's probably broken now but there is no test > > - lowres stuff > > - codec copy apparently required by bitstream filters > > > > The matroska references are updated because they now properly forward > > the field_order (previously unknown, now progressive). > > > > Thanks to James Almer for fixing a bunch of FATE issues in this commit. > > > > Signed-off-by: Clément Bœsch <clem...@stupeflix.com> > > Signed-off-by: James Almer <jamr...@gmail.com> > > --- > > Changes since previous patch: > > - restore the usage of avctx instead of ost->enc_ctx in the extradata stuff > > (which is going away in later commits) > > - unconditionnally set pkt_timebase (as suggested by nevcairiel) with a new > > simpler comment > > - restored the frame_size copy which i discarded by mistake > > --- > > cmdutils.c | 4 +- > > ffmpeg.c | 234 > > +++++++++++++++++++++++++---------------------- > > ffmpeg.h | 1 + > > ffmpeg_filter.c | 24 ++--- > > ffmpeg_opt.c | 140 ++++++++++++++++------------ > > > libavformat/utils.c | 2 +- > > This should go in a seperate patch possibly with a version bump > so ffmpeg.c can properly "depend" on that avformat version >
would it be OK to replace that condition with (ost->codecpar->codec_tag == AV_RL32("tmcd") || enc_ctx->codec_tag == AV_RL32("tmcd")) within that commit? -- Clément B. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel