[FFmpeg-devel] [PATCH 3/5] avfilter/drawutils: simplify xyz format check

[epirat07 at gmail.com]

On 26 Oct 2023, at 16:05, Niklas Haas wrote:

> On Thu, 26 Oct 2023 10:13:03 -0300 James Almer <jamrial at gmail.com> wrote:
>> On 10/26/2023 9:23 AM, Nicolas George wrote:
>>> Niklas Haas (12023-10-26):
>>>> From: Niklas Haas <git at haasn.dev>
>>>>
>>>> ---
>>>>   libavfilter/drawutils.c | 3 ---
>>>>   1 file changed, 3 deletions(-)
>>>
>>> Please merge this with patch 1.
>>
>> This patch reverts patch 1, so merging is basically just dropping both.
>> I assume the reason he did things this way is to prevent patch 2 (an
>> avutil patch adding new API) from changing the output filter-pixfmts-pad.
>
> Yes, exactly. But if you want, I can merge the FATE ref change into
> patch 2. I just thought this way separates the logic change (bugfix in
> drawutils) from the new API addition, which is IMO cleaner.

IMO it’s easier to understand with the patches separated.
Maybe clarify the commit message a bit though about this being
replaced by the new flag in a follow-up commit?

Also I am confused about this patch (simplify xyz format check),
as the message says „simplify“ but you just remove it? Is it covered
already by an existing check in this file not touched by the patch?
Then the message should probably be „remove redundant check“ and
clarify why it is redundant now?

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".