On Sun, Sep 13, 2015 at 11:19:00PM +0200, Carl Eugen Hoyos wrote: > On Sunday 13 September 2015 12:30:20 am Michael Niedermayer wrote: > > On Sat, Sep 12, 2015 at 07:52:42PM +0200, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached is a new patch that should be ready to be applied > > > no matter if we drop compatibility or not. > > > > > > Please comment (quickly if possible), Carl Eugen > > > > i think there are a few changes missing: > > New patch attached.
the entries in avcodec_options are wrong {"b", "set bitrate (in bits/s)", OFFSET(bit_rate), AV_OPT_TYPE_INT, {.i64 = AV_CODEC_DEFAULT_BITRATE }, 0, INT_MAX, A|V|E}, {"ab", "set bitrate (in bits/s)", OFFSET(bit_rate), AV_OPT_TYPE_INT, {.i64 = 128*1000 }, 0, INT_MAX, A|E}, bit_rate is not a AV_OPT_TYPE_INT its a AV_OPT_TYPE_INT64 after this [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The worst form of inequality is to try to make unequal things equal. -- Aristotle
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel