On Thu, Oct 15, 2015 at 9:57 AM, Anton Khirnov <[email protected]> wrote:
> Quoting Vittorio Giovara (2015-10-14 15:07:10)
>> ---
>>  cmdutils.c | 29 +++++++++++++++++++++++++++++
>>  1 file changed, 29 insertions(+)
>>
>> diff --git a/cmdutils.c b/cmdutils.c
>> index 065ba49..5cb22f6 100644
>> --- a/cmdutils.c
>> +++ b/cmdutils.c
>> @@ -977,6 +977,35 @@ static void print_codec(const AVCodec *c)
>>      printf("%s %s [%s]:\n", encoder ? "Encoder" : "Decoder", c->name,
>>             c->long_name ? c->long_name : "");
>>
>> +    printf("    General capabilities: ");
>> +    if (c->capabilities & AV_CODEC_CAP_DRAW_HORIZ_BAND)
>> +        printf("horizband ");
>> +    if (c->capabilities & AV_CODEC_CAP_DR1)
>> +        printf("dr1 ");
>> +    if (c->capabilities & AV_CODEC_CAP_TRUNCATED)
>> +        printf("trunc ");
>> +    if (c->capabilities & AV_CODEC_CAP_DELAY)
>> +        printf("delay ");
>> +    if (c->capabilities & AV_CODEC_CAP_SMALL_LAST_FRAME)
>> +        printf("small ");
>> +    if (c->capabilities & AV_CODEC_CAP_SUBFRAMES)
>> +        printf("subframes ");
>> +    if (c->capabilities & AV_CODEC_CAP_EXPERIMENTAL)
>> +        printf("exp ");
>> +    if (c->capabilities & AV_CODEC_CAP_CHANNEL_CONF)
>> +        printf("chconf ");
>> +    if (c->capabilities & AV_CODEC_CAP_PARAM_CHANGE)
>> +        printf("small ");
>> +    if (c->capabilities & AV_CODEC_CAP_PARAM_CHANGE)
>> +        printf("variable ");
>> +    if (c->capabilities & (AV_CODEC_CAP_FRAME_THREADS |
>> +                           AV_CODEC_CAP_SLICE_THREADS |
>> +                           AV_CODEC_CAP_AUTO_THREADS))
>> +        printf("threads ");
>> +    if (!c->capabilities)
>> +        printf("none");
>> +    printf("\n");
>> +
>
> What for?
>
> I specifically did not add those here, because they are of no
> consequence to avconv users. I think avconv output is confusing enough
> as it is, no need to make it even more pointlessly complicated.

It can actually be useful if you can't use libavcodec api (eg due to
missing bindings from another language) or if you want to just parse
avconv output. Also this output is very specific (since you have to
specify -h codec=name), so I don't think it's going to confuse users,
but just provide more details.
-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to