[FFmpeg-devel] [PATCH] lavc/libvpxenc: add -row-mt option

Kagami Hiiragi kagami at genshiken.org
Fri Mar 3 00:30:18 EET 2017


On 03/03/17 01:16, Moritz Barsnick wrote:
> On Thu, Mar 02, 2017 at 22:00:36 +0300, Kagami Hiiragi wrote:
>> +    {"row-mt", "Row based multi-threading", OFFSET(row_mt), AV_OPT_TYPE_INT, {.i64 = -1}, -1, 1, VE},
>                                                                            ^
> Woudn't a _BOOL type accept exactly the same ranges and defaults, with
> the same behavior, but document itself more nicely?
> 
> It would result in
>  -row-mt  <boolean>    E....... Row based multi-threading (default auto)
> instead of
>  -row-mt  <int>        E....... Row based multi-threading (from -1 to 1) (default -1)
> 
> (Guessing, untested.)
> 
> Moritz

I copied description of "lossless" flag, but "frame-parallel" and other
encoders seems to prefer BOOL, you are right.

I'm leaving it up to commiter, it's just s/_INT/_BOOL/.


More information about the ffmpeg-devel mailing list