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. - Derek _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel