[FFmpeg-devel] [PATCH 2/2] Set AV_PKT_FLAG_KEY based on nupplevideo header

Joakim Plate elupus at ecce.se
Sun Sep 18 18:34:34 CEST 2011


On Sat, 17 Sep 2011 16:03:41 +0200, Reimar Döffinger wrote:

> On Sun, Sep 11, 2011 at 06:11:27PM +0200, Joakim Plate wrote:
>> [22 quoted lines suppressed]
> 
> Two suggestions (do as you consider best to do):
> 1) Drop the pointless {} (maybe even use ?: for consistency with your other
> patch)
> 2) Change the order, ->pts is hdr[4], this uses hdr[2], so I think it
> would be good if they are close together with the flags first.
> 
> Otherwise I think we should try it.
> If there's a problem we could also try to force a parser in to generate
> the KEY flag if it is necessary after all.

Pushed



More information about the ffmpeg-devel mailing list