On Fri, Jul 15, 2011 at 10:26:01PM +0100, Måns Rullgård wrote: > Diego Biurrun <[email protected]> writes: > > > On Fri, Jul 15, 2011 at 05:14:37PM -0400, Daniel Kang wrote: > >> On Fri, Jul 15, 2011 at 2:57 PM, Diego Biurrun <[email protected]> wrote: > >> > >> > --- > >> > libavcodec/flashsvenc.c | 7 +++---- > >> > 1 files changed, 3 insertions(+), 4 deletions(-) > >> > > >> > diff --git a/libavcodec/flashsvenc.c b/libavcodec/flashsvenc.c > >> > index 6eda26d..bcfd41d 100644 > >> > --- a/libavcodec/flashsvenc.c > >> > +++ b/libavcodec/flashsvenc.c > >> > @@ -230,10 +230,9 @@ static int flashsv_encode_frame(AVCodecContext > >> > *avctx, > >> > uint8_t *buf, > >> > pfptr = s->previous_frame; > >> > > >> > /* Check the placement of keyframes */ > >> > - if (avctx->gop_size > 0) { > >> > - if (avctx->frame_number >= s->last_key_frame + avctx->gop_size) > >> > { > >> > - I_frame = 1; > >> > - } > >> > + if (avctx->gop_size > 0 && > >> > + avctx->frame_number >= s->last_key_frame + avctx->gop_size) { > >> > + I_frame = 1; > >> > > >> > >> Are the extra spaces standard? > > > > It's vertical alignment nonsense, maybe i'll drop it... > > It doesn't really do any good here, IMO.
Yeah, I'll change, rebase and push. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
