On Wed, Aug 31, 2011 at 10:15 AM, Anton Khirnov <[email protected]> wrote:
>
> On Wed, 31 Aug 2011 10:00:34 -0700, Jason Garrett-Glaser <[email protected]> 
> wrote:
>> On Wed, Aug 31, 2011 at 5:04 AM, Anton Khirnov <[email protected]> wrote:
>> > ---
>> >  libavcodec/libx264.c |    1 +
>> >  1 files changed, 1 insertions(+), 0 deletions(-)
>> >
>> > diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c
>> > index 8614ec5..bff29d8 100644
>> > --- a/libavcodec/libx264.c
>> > +++ b/libavcodec/libx264.c
>> > @@ -476,6 +476,7 @@ static const AVClass class = {
>> >
>> >  static const AVCodecDefault x264_defaults[] = {
>> >     { "b",                "0" },
>> > +    { "threads",          "0" },
>> >     { NULL },
>> >  };
>> >
>> > --
>> > 1.7.5.4
>>
>> X264_THREADS_AUTO exists for a reason, you should use it.
>>
>
> That would be a little nontrivial to do, our current per-codec
> defaults "system" is just pairs of key:value strings.
>
> Extending it just for this is :effort:.

What in the world are you on?  X264_THREADS_AUTO is an integer defined
in a header file that you've already included, and is known on compile
time.  What in the world is stopping you from using it?

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

Reply via email to