Bump to apply? - dale
On Thu, Jan 30, 2020 at 3:21 PM Dale Curtis <dalecur...@google.com> wrote: > On Thu, Jan 30, 2020 at 11:23 AM Dale Curtis <dalecur...@chromium.org> > wrote: > >> On Wed, Jan 29, 2020 at 10:23 PM Michael Niedermayer >> <mich...@niedermayer.cc> 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 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".
