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... Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
