On Mon, Jan 07, 2013 at 01:23:27PM +0100, Anton Khirnov wrote:
> --- a/libavcodec/eatgq.c
> +++ b/libavcodec/eatgq.c
> @@ -47,23 +47,24 @@ typedef struct TgqContext {
>  
> -static av_cold int tgq_decode_init(AVCodecContext *avctx){
> +static av_cold int tgq_decode_init(AVCodecContext *avctx)
> +{
>      TgqContext *s = avctx->priv_data;
>      uint8_t idct_permutation[64];
>      s->avctx = avctx;
>      ff_init_scantable_permutation(idct_permutation, FF_NO_IDCT_PERM);
>      ff_init_scantable(idct_permutation, &s->scantable, ff_zigzag_direct);
>      avctx->time_base = (AVRational){1, 15};

avctx->time_base = (AVRational) { 1, 15 };

>  static void tgq_decode_block(TgqContext *s, DCTELEM block[64], GetBitContext 
> *gb){

{ on the next line

>      uint8_t *perm = s->scantable.permutated;
> -    int i,j,value;
> +    int i, j, value;
>      block[0] = get_sbits(gb,8) * s->qtable[0];
> -    for(i=1; i<64; ) {
> -        switch(show_bits(gb,3)) {
> +    for (i = 1; i < 64;) {
> +        switch (show_bits(gb,3)) {

space after comma here and in many other places

> @@ -90,79 +91,84 @@ static void tgq_decode_block(TgqContext *s, DCTELEM 
> block[64], GetBitContext *gb
>  
>  static void tgq_idct_put_mb_dconly(TgqContext *s, int mb_x, int mb_y, const 
> int8_t *dc)
>  {
> -    int linesize= s->frame.linesize[0];
> -    uint8_t *dest_y  = s->frame.data[0] + (mb_y * 16* linesize            ) 
> + mb_x * 16;
> -    uint8_t *dest_cb = s->frame.data[1] + (mb_y * 8 * s->frame.linesize[1]) 
> + mb_x * 8;
> -    uint8_t *dest_cr = s->frame.data[2] + (mb_y * 8 * s->frame.linesize[2]) 
> + mb_x * 8;
> +    int linesize = s->frame.linesize[0];
> +    uint8_t *dest_y  = s->frame.data[0] + (mb_y * 16 * linesize)             
> + mb_x * 16;
> +    uint8_t *dest_cb = s->frame.data[1] + (mb_y * 8  * s->frame.linesize[1]) 
> + mb_x * 8;
> +    uint8_t *dest_cr = s->frame.data[2] + (mb_y * 8  * s->frame.linesize[2]) 
> + mb_x * 8;
>      tgq_dconly(s,dest_y                 , linesize, dc[0]);
>      tgq_dconly(s,dest_y              + 8, linesize, dc[1]);
>      tgq_dconly(s,dest_y + 8*linesize    , linesize, dc[2]);
>      tgq_dconly(s,dest_y + 8*linesize + 8, linesize, dc[3]);

far too many spaces before comma

>              tgq_decode_block(s, s->block[i], &gb);
>          tgq_idct_put_mb(s, s->block, mb_x, mb_y);
>          bytestream2_skip(&s->gb, mode);
>      }else{
> -        if (mode==3) {
> +        if (mode == 3) {

} else {

>              memset(dc, bytestream2_get_byte(&s->gb), 4);
>              dc[4] = bytestream2_get_byte(&s->gb);
>              dc[5] = bytestream2_get_byte(&s->gb);
> -        }else if (mode==6) {
> +        }else if (mode == 6) {
>              bytestream2_get_buffer(&s->gb, dc, 6);
> -        }else if (mode==12) {
> +        }else if (mode == 12) {

space before else

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to