On Sun, Nov 8, 2015 at 7:24 AM, Derek Buitenhuis <derek.buitenh...@gmail.com> wrote: > On 11/7/2015 10:42 PM, Ganesh Ajjanagadde wrote: >> This is what I felt - it is an internal struct, and there are explicit >> comments in the header saying that the struct's fields are meant to be >> manipulated via the options API unlike e.g libavcodec. The types were >> IMHO not part of the ABI/API since the struct was marked opaque. >> Anyway, this was my reasoning while submitting the patch. > > I think the patch is OK, but needs a micro bump.
Modulo the version bump, I will wait for an explicit ack from Michael as he is the maintainer for this. > > - Derek > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
