On Sat, Jan 2, 2016 at 6:25 PM, Anton Khirnov <[email protected]> wrote:
> Quoting Luca Barbato (2016-01-02 15:34:25)
>> Subject: Re: [libav-devel] [PATCH] avf: Dump the cpb side data information
>                                     ^
> missing l
>
>> ---
>>  libavformat/dump.c | 20 ++++++++++++++++++++
>>  1 file changed, 20 insertions(+)
>>
>> diff --git a/libavformat/dump.c b/libavformat/dump.c
>> index 878eae8..de361b5 100644
>> --- a/libavformat/dump.c
>> +++ b/libavformat/dump.c
>> @@ -318,6 +318,22 @@ static void dump_audioservicetype(void *ctx, 
>> AVPacketSideData *sd)
>>      }
>>  }
>>
>> +static void dump_cpb(void *ctx, AVPacketSideData *sd)
>> +{
>> +    AVCPBProperties *cpb = (AVCPBProperties *)sd->data;
>> +
>> +    if (sd->size < sizeof(*cpb)) {
>> +        av_log(ctx, AV_LOG_INFO, "invalid data");
>> +        return;
>> +    }
>> +
>> +    av_log(ctx, AV_LOG_INFO,
>> +           "bitrate: %d/%d/%d buffer size: %d vbv_delay: %"PRId64,
>
> prepending "max/min/avg" would make it easier to read IMO

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

Reply via email to