Michael Niedermayer: > On Sun, Jun 08, 2025 at 01:37:32AM +0200, Andreas Rheinhardt wrote: >> Michael Niedermayer: >>> Hi >>> >>> On Wed, Jun 04, 2025 at 03:05:12AM +0200, Michael Niedermayer wrote: >>> [...] >>>>>> #define FFSWAP(type,a,b) do{type SWAP_tmp= b; b= a; a= >>>>>> SWAP_tmp;}while(0) >>>>> >>>>> 1. I don't like that you change the common case (namely the integer one) >>>>> and thereby making the name longer (and making this patch huge). >>>> >>>> I can just rename FFMINI/MAXI to FFMIN/MAX if preferred >>> >>> are there any objections to FFMIND / FFMINF / FFMAXD / FFMAXF ? >>> if not i intend to apply these in the next days (unless i forget) >>> >> >> I don't want them to be public. > > in which header file do you want them ? >
lavu/internal.h seems to be the least bad option. - Andreas _______________________________________________ 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".