Bump to apply? - dale
On Thu, Jan 30, 2020 at 3:21 PM Dale Curtis <[email protected]> wrote: > On Thu, Jan 30, 2020 at 11:23 AM Dale Curtis <[email protected]> > wrote: > >> On Wed, Jan 29, 2020 at 10:23 PM Michael Niedermayer >> <[email protected]> wrote: >> >>> so i think it works but maybe ive missed something, for which values >>> of e2_pts do you see a problem with e1_pts = INT64_MIN? >>> >> >> For e1_pts = INT64_MIN and e2_pts >= 0 you end up with a negative int64_t >> result for e2_pts - (uint64_t)e1_pts, so it's always < time_tolerance. If >> that's what you intended, then sgtm. >> > > Actually, this was a test construction error on my part. In a conditional > this does evaluate to a uint64_t vs int64_t, so the comparison is valid. > I've attached your recommended patch. Thanks for your patience. > > - dale > > > _______________________________________________ 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".
