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".