[FFmpeg-soc] [soc]: r3884 - amr/amrnbdec.c

Robert Swain robert.swain at gmail.com
Mon Dec 15 12:06:56 CET 2008


2008/12/15 diego <subversion at mplayerhq.hu>:
> Author: diego
> Date: Mon Dec 15 11:13:50 2008
> New Revision: 3884
>
> Log:
> K&R function declaration and whitespace cosmetics
>
>
> Modified:
>   amr/amrnbdec.c
>
> Modified: amr/amrnbdec.c
> ==============================================================================
> --- amr/amrnbdec.c      (original)
> +++ amr/amrnbdec.c      Mon Dec 15 11:13:50 2008
> @@ -76,8 +76,8 @@ typedef struct AMRContext {
>  } AMRContext;
>
>
> -static int amrnb_decode_init(AVCodecContext *avctx) {
> -
> +static int amrnb_decode_init(AVCodecContext *avctx)
> +{
>     AMRContext *p = avctx->priv_data;
>
>     // variables needed for cos table generation
> @@ -126,8 +126,9 @@ static int amrnb_decode_init(AVCodecCont
>  * @return Returns the frame mode
>  */
>
> -enum Mode decode_bitstream(AVCodecContext *avctx, uint8_t *buf, int buf_size, enum Mode *speech_mode) {
> -
> +enum Mode decode_bitstream(AVCodecContext *avctx, uint8_t *buf, int buf_size,
> +                           enum Mode *speech_mode)
> +{

Urgh. I'm happy with the line breaks but I don't tend to like the
opening { on a new line. I thought that was a GNU thing not a K&R
thing.

Rob



More information about the FFmpeg-soc mailing list