On Tue, Nov 3, 2015 at 9:02 PM, Michael Niedermayer <mich...@niedermayer.cc> wrote:
> On Mon, Nov 02, 2015 at 11:45:05AM +0100, Matthieu Bouron wrote: > > From: Matthieu Bouron <matthieu.bou...@stupeflix.com> > > > > Codec supporting FF_CODEC_SKIP_FRAME_FILL must still extract and fill > > their parameters into AVCodecContext while honoring the skip_frame > > flag. > > --- > > libavcodec/internal.h | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/libavcodec/internal.h b/libavcodec/internal.h > > index 0abe17f..a023d1a 100644 > > --- a/libavcodec/internal.h > > +++ b/libavcodec/internal.h > > @@ -53,6 +53,11 @@ > > * from the input AVPacket. > > */ > > #define FF_CODEC_CAP_SETS_PKT_DTS (1 << 2) > > +/** > > + * Codec still extract and fill its parameters even if the skip_frame > > + * parameter is honored. > > iam not sure about the english grammer here sounds a bit odd but > otherwise the patch LGTM > If you have a better sentence or someone else does, I would be happy to update the current one before i push the patch. Matthieu _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
