[FFmpeg-devel] [PATCH v3 0/3] s/RUNTIME/POST_INIT_SETTABLE/

[Andrew Sayers <ffmpeg-devel@pileofstuff.org>]

AV_OPT_FLAG_POST_INIT_SETTABLE_PARAM is fine by me, here's a patch.
I've added a "@deprecated" comment for the old name, but would this
need to be queued up for 8.0?  Technically this is a backwards-incompatible
change to the existing API, even though it doesn't change the ABI or generate
warnings when compiling code.

My vote is always going to be for putting documentation in the first place
people look, even at the expense of redundancy.  But I can live without the
extra comments so long as the flag is renamed.  This patch moves the extra
documentation to an optional commit - I'm fine with just applying #1 and #3
if people prefer, but it's there if the conversation goes the other way.

Also, I think this is better, but can also live with the v2 patch,
so long as the other notes remain in.

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".