[FFmpeg-devel] [PATCH] cmdutils: include threading caps in codecs listing.

[Michael Bradshaw]

On Wed, May 30, 2012 at 1:47 PM, Reimar Döffinger
<Reimar.Doeffinger at gmx.de> wrote:
> On Wed, May 30, 2012 at 09:40:49PM +0200, Nicolas George wrote:
>> -           " D..... = Decoding supported\n"
>> -           " .E.... = Encoding supported\n"
>> -           " ..V... = Video codec\n"
>> -           " ..A... = Audio codec\n"
>> -           " ..S... = Subtitle codec\n"
>> -           " ...S.. = Supports draw_horiz_band\n"
>> -           " ....D. = Supports direct rendering method 1\n"
>> -           " .....T = Supports weird frame truncation\n"
>> -           " ------\n");
>> +           " D....... = Decoding supported\n"
>> +           " .E...... = Encoding supported\n"
>> +           " ..V..... = Video codec\n"
>> +           " ..A..... = Audio codec\n"
>> +           " ..S..... = Subtitle codec\n"
>> +           " ...S.... = Supports draw_horiz_band\n"
>> +           " ....D... = Supports direct rendering method 1\n"
>> +           " .....T.. = Supports weird frame truncation\n"
>> +           " ......F. = Supports frame-based multi-threading\n"
>> +           " .......S = Supports slice-based multi-threading\n"
>> +           " --------\n");
>
> I don't have much of an opinion, but I was wondering if maybe
> F Supports frame-based multi-threading
> S Supports slice-based multi-threading
> B Supports both frame-based and slice-based multi-threading
> would be better

Personally, I like them separate. None of the other fields have a
combined field (i.e. there's no C = supports decoding and encoding),
and plus I don't have to remember what "B" is for.

--Michael