On Sun, Jan 01, 2012 at 09:41:13PM +0530, Aneesh Dogra wrote:
>
> --- a/libavcodec/bethsoftvideo.c
> +++ b/libavcodec/bethsoftvideo.c
> @@ -83,37 +84,40 @@ static int bethsoftvid_decode_frame(AVCodecContext *avctx,
>
> - switch(block_type = *buf++){
> + switch (block_type = *buf++) {
> case PALETTE_BLOCK:
> return set_palette(&vid->frame, buf, buf_size);
> case VIDEO_YOFF_P_FRAME:
> yoffset = bytestream_get_le16(&buf);
> - if(yoffset >= avctx->height)
> + if (yoffset >= avctx->height)
> return -1;
> dst += vid->frame.linesize[0] * yoffset;
> }
http://www.libav.org/developer.html#Code-formatting-conventions
There's something in there about switch/case.
You also missed prettyprinting the AVCodec declaration at the bottom
of the file.
I have the nagging feeling to have told you this and what Mashiat said
in the previous review before.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
