On Sun, Nov 20, 2016 at 08:53:19PM +0100, Andreas Cadhalpun wrote: > On 20.11.2016 12:57, Michael Niedermayer wrote: > > Found-by: Andreas > > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > > --- > > libavutil/opt.c | 8 +++++--- > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/libavutil/opt.c b/libavutil/opt.c > > index cd16bd1..6669356 100644 > > --- a/libavutil/opt.c > > +++ b/libavutil/opt.c > > @@ -126,9 +126,11 @@ static int write_number(void *obj, const AVOption *o, > > void *dst, double num, int > > break; > > case AV_OPT_TYPE_DURATION: > > case AV_OPT_TYPE_CHANNEL_LAYOUT: > > - case AV_OPT_TYPE_INT64: > > - *(int64_t *)dst = llrint(num / den) * intnum; > > - break; > > + case AV_OPT_TYPE_INT64:{ > > + double d = num / den; > > + if (intnum == 1 && d == (double)INT64_MAX) *(int64_t *)dst = > > INT64_MAX; > > + else *(int64_t *)dst = > > llrint(d) * intnum; > > + break;} > > case AV_OPT_TYPE_FLOAT: > > *(float *)dst = num * intnum / den; > > break; > > > > LGTM. Thanks for finding a better fix for the problem!
applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No snowflake in an avalanche ever feels responsible. -- Voltaire
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel