On Tue, Nov 16, 2021 at 1:57 PM Anton Khirnov <[email protected]> wrote:

> Quoting [email protected] (2021-11-15 07:22:20)
> > From: Mark Reid <[email protected]>
> >
> > Changes av_clipf to return amin if a is nan.
> > Before if a is nan av_clipf_c returned nan and
> > av_clipf_sse would return amax. Now the both
> > should behave the same.
> >
> > This works because nan > amin is false.
> > The max(nan, amin) will be amin.
>
> This seems like it goes against the point of nan. I would expect any
> float operation involving nan to return nan.
>

fmaxf,fminf does not return nan, unless both values are nan.
So to me patch is fine.


>
> --
> Anton Khirnov
> _______________________________________________
> ffmpeg-devel mailing list
> [email protected]
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> [email protected] with subject "unsubscribe".
>
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".

Reply via email to