Hi,

On Tue, Apr 26, 2011 at 4:18 AM, Ronald S. Bultje <rsbul...@gmail.com> wrote:
> Hi,
>
> On Wed, Apr 20, 2011 at 7:45 PM, Anton Khirnov <an...@khirnov.net> wrote:
>> From: Stefano Sabatini <stefano.sabatini-l...@poste.it>
>>
>> The constant values were never set, thus the evaluation of expressions
>> referencing those constants were resulting in an error.
>>
>> Signed-off-by: Stefano Sabatini <stefano.sabatini-l...@poste.it>
>> Signed-off-by: Anton Khirnov <an...@khirnov.net>
>> ---
>>  libavfilter/vf_scale.c |    4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavfilter/vf_scale.c b/libavfilter/vf_scale.c
>> index 77de116..84d97bd 100644
>> --- a/libavfilter/vf_scale.c
>> +++ b/libavfilter/vf_scale.c
>> @@ -165,12 +165,12 @@ static int config_props(AVFilterLink *outlink)
>>     av_expr_parse_and_eval(&res, (expr = scale->w_expr),
>>                            var_names, var_values,
>>                            NULL, NULL, NULL, NULL, NULL, 0, ctx);
>> -    scale->w = var_values[VAR_OW] = res;
>> +    scale->w = var_values[VAR_OUT_W] = var_values[VAR_OW] = res;
>>     if ((ret = av_expr_parse_and_eval(&res, (expr = scale->h_expr),
>>                                       var_names, var_values,
>>                                       NULL, NULL, NULL, NULL, NULL, 0, ctx)) 
>> < 0)
>>         goto fail;
>> -    scale->h = var_values[VAR_OH] = res;
>> +    scale->h = var_values[VAR_OUT_H] = var_values[VAR_OH] = res;
>>     /* evaluate again the width, as it may depend on the output height */
>>     if ((ret = av_expr_parse_and_eval(&res, (expr = scale->w_expr),
>>                                       var_names, var_values,
>
> OK.

Also feel free to squash this with 22.

Ronald
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to