On 09/30/2012 02:40 PM, Anton Khirnov wrote:
>
> 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.
>
Done
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
