On Fri, Jul 15, 2011 at 5:23 PM, Diego Biurrun <[email protected]> wrote:
> 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... I generally only keep that when the conditions (themselves) are related. They don't seem to be in this case.
_______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
