On 26 Oct 2023, at 16:05, Niklas Haas wrote:
> On Thu, 26 Oct 2023 10:13:03 -0300 James Almer <jamr...@gmail.com> wrote: >> On 10/26/2023 9:23 AM, Nicolas George wrote: >>> Niklas Haas (12023-10-26): >>>> From: Niklas Haas <g...@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@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
