On Sat, 29 Sep 2012 23:07:42 +0200, Luca Barbato <[email protected]> wrote:
> Mostly cosmetic changes adding some intermediate to shorten the lines.
> ---
>  libavformat/mux.c | 69 
> +++++++++++++++++++++++++++++++++----------------------
>  1 file changed, 42 insertions(+), 27 deletions(-)
> 
> diff --git a/libavformat/mux.c b/libavformat/mux.c
> index f63cffe..ecd2988 100644
> --- a/libavformat/mux.c
> +++ b/libavformat/mux.c
> @@ -140,88 +140,103 @@ int avformat_write_header(AVFormatContext *s, 
> AVDictionary **options)
>      int ret = 0, i;
>      AVStream *st;
>      AVDictionary *tmp = NULL;
> +    AVCodecContext *ctx = NULL;

Please name this 'codec' or something along those lines. ctx is a bit
too generic and unclear here.

Otherwise looks ok.

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

Reply via email to