On Thu, 2011-09-22 at 17:54 -0700, Alex Converse wrote: > On Thu, Sep 22, 2011 at 5:51 PM, Justin Ruggles > <[email protected]> wrote: > > On 09/22/2011 08:33 PM, Måns Rullgård wrote: > > > >> Benjamin Larsson <[email protected]> writes: > >> > >>> On 09/23/2011 12:10 AM, Diego Biurrun wrote: > >>>> --- > >>>> libavformat/utils.c | 1 + > >>>> 1 files changed, 1 insertions(+), 0 deletions(-) > >>>> > >>>> diff --git a/libavformat/utils.c b/libavformat/utils.c > >>>> index 05d4fda..17b342e 100644 > >>>> --- a/libavformat/utils.c > >>>> +++ b/libavformat/utils.c > >>>> @@ -852,6 +852,7 @@ static int is_intra_only(AVCodecContext *enc){ > >>>> case CODEC_ID_MJPEG: > >>>> case CODEC_ID_MJPEGB: > >>>> case CODEC_ID_LJPEG: > >>>> + case CODEC_ID_PRORES: > >>>> case CODEC_ID_RAWVIDEO: > >>>> case CODEC_ID_DVVIDEO: > >>>> case CODEC_ID_HUFFYUV: > >>> > >>> How about a codec capability instead ? > >> > >> WTF is that used for at all? > > > > > > looks to me like a shortcut to not require all the demuxers set the key > > frame flag like they're supposed to. > > > > For mov files the keyframe box is often missing for intra only > streams. I'm sure some other formats are similar.
Maybe add a comment explaining that? /Tomas _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
