[FFmpeg-devel] [PATCH] avfilter/select: initialize prev_selected_n to NAN

[FFmpeg-devel] [PATCH] avfilter/select: initialize prev_selected_n to NAN

[Gyan Doshi]

As per the doc, prev_selected_n should be NAN at the start.
However, this was never set.
---
 libavfilter/f_select.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libavfilter/f_select.c b/libavfilter/f_select.c
index 47e36f0014..109dae403a 100644
--- a/libavfilter/f_select.c
+++ b/libavfilter/f_select.c
@@ -230,6 +230,7 @@ static int config_input(AVFilterLink *inlink)
 
     select->var_values[VAR_N]          = 0.0;
     select->var_values[VAR_SELECTED_N] = 0.0;
+    select->var_values[VAR_PREV_SELECTED_N] = NAN;
 
     select->var_values[VAR_TB] = av_q2d(inlink->time_base);
 
-- 
2.39.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter/select: initialize prev_selected_n to NAN

[Stefano Sabatini]

On date Friday 2023-12-08 13:05:43 +0530, Gyan Doshi wrote:
> As per the doc, prev_selected_n should be NAN at the start.
> However, this was never set.
> ---
>  libavfilter/f_select.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavfilter/f_select.c b/libavfilter/f_select.c
> index 47e36f0014..109dae403a 100644
> --- a/libavfilter/f_select.c
> +++ b/libavfilter/f_select.c
> @@ -230,6 +230,7 @@ static int config_input(AVFilterLink *inlink)
>  
>      select->var_values[VAR_N]          = 0.0;
>      select->var_values[VAR_SELECTED_N] = 0.0;
> +    select->var_values[VAR_PREV_SELECTED_N] = NAN;

Should be good but move it closer to the other VAR_PREV_* definitions.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter/select: initialize prev_selected_n to NAN

[Gyan Doshi]

On 2023-12-10 06:03 am, Stefano Sabatini wrote:
> On date Friday 2023-12-08 13:05:43 +0530, Gyan Doshi wrote:
>> As per the doc, prev_selected_n should be NAN at the start.
>> However, this was never set.
>> ---
>>   libavfilter/f_select.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/libavfilter/f_select.c b/libavfilter/f_select.c
>> index 47e36f0014..109dae403a 100644
>> --- a/libavfilter/f_select.c
>> +++ b/libavfilter/f_select.c
>> @@ -230,6 +230,7 @@ static int config_input(AVFilterLink *inlink)
>>   
>>       select->var_values[VAR_N]          = 0.0;
>>       select->var_values[VAR_SELECTED_N] = 0.0;
>> +    select->var_values[VAR_PREV_SELECTED_N] = NAN;
> Should be good but move it closer to the other VAR_PREV_* definitions.

Done and pushed as b9fd3e7a88b326f719bd3c47d0e0cc5a435de428

Regards,
Gyan

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".