On Fri, May 1, 2020 at 10:32 AM James Almer <jamr...@gmail.com> wrote:
> On 5/1/2020 2:23 PM, Dale Curtis wrote:
> > On Fri, May 1, 2020 at 6:12 AM James Almer <jamr...@gmail.com> wrote:
> >
> >> On 5/1/2020 6:36 AM, Carl Eugen Hoyos wrote:
> >>>
> >>> The macro exists to avoid separate patches?
> >>
> >> No, it exists to not require configure checks just to enable a path for
> >> gcc/clang and another for other compilers.
> >>
> >
> > Since consensus seems to have landed on splitting the patches, I've done
> > so. This thread now contains just the default implementation.
>
> That wasn't the consensus. Neither Nicholas or I thought it was
> required, but i don't have strong feelings about it.
>
Ah sorry for misunderstanding, your response to Carl above only seemed to
refute the necessity of a configure patch. I also don't care either way.
> > +static int64_t av_sat_add64_c(int64_t a, int64_t b) {
>
> Missing av_always_inline attribute
>
Done.
>
> > +static int64_t av_sat_sub64_c(int64_t a, int64_t b) {
>
> Ditto
>
Done.
- dale
sat_math_v3.patch
Description: Binary data
_______________________________________________ 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".
