[FFmpeg-devel] Fixes #10509

[Marton Balint]

On Fri, 29 Mar 2024, Poorva wrote:

>>
>
> On Tue, Mar 26, 2024 at 2:36 AM Poorva <2003gaikarpoorva at gmail.com> wrote:
>>>
>>>
>>
>>
>> Thank you for your feedback on the Git patch I submitted for review.
>> I have rectified the problem by adding the necessary changes .
>> The updated patch file is attached for your review.
>>>
>>>
> I wanted to follow up on the patch titled
> "v3-0001-avfilter-f_select.c - Add Support for IW and IH" that I
> submitted earlier and provide an update based on the feedback
> received.
>
> In response to your suggestion about the switch block, I have
> integrated the changes into the existing switch block for
> AVMEDIA_TYPE_VIDEO. Additionally, I have removed an unnecessary new
> line that was added at the end of the file.
>
> Despite these modifications, I have not received any further feedback
> or comments on the patch. Therefore, I kindly request the community to
> review the updated patch attached to this email.

[..]

> @@ -371,6 +383,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
>          break;
>      }
> 
> +
>      select->select = res = av_expr_eval(select->expr, select->var_values, NULL);
>      av_log(inlink->dst, AV_LOG_DEBUG,
>             "n:%f pts:%f t:%f key:%d",
> @@ -545,4 +558,4 @@ const AVFilter ff_vf_select = {
>      FILTER_QUERY_FUNC(query_formats),
>      .flags         = AVFILTER_FLAG_DYNAMIC_OUTPUTS | AVFILTER_FLAG_METADATA_ONLY,
>  };
> -#endif /* CONFIG_SELECT_FILTER */
> +#endif /* CONFIG_SELECT_FILTER */
> \ No newline at end of file
> -- 
> 2.43.0.windows.1
>

These two whitespace changes are still unnecessary. Please check your 
patch before sending.

Thanks,
Marton