[FFmpeg-devel] [PATCH 1/2] Replace FFMIN/FFMAX by type specific macros

[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 ?

thx


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20250608/4bc7050a/attachment.sig>