On 10/9/19, Michael Niedermayer <mich...@niedermayer.cc> wrote: > On Wed, Oct 09, 2019 at 12:16:52PM +0200, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol <one...@gmail.com> >> --- >> libavutil/opt.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/libavutil/opt.h b/libavutil/opt.h >> index 39f4a8dda0..693a170bcc 100644 >> --- a/libavutil/opt.h >> +++ b/libavutil/opt.h >> @@ -288,6 +288,7 @@ typedef struct AVOption { >> */ >> #define AV_OPT_FLAG_READONLY 128 >> #define AV_OPT_FLAG_BSF_PARAM (1<<8) ///< a generic parameter which >> can be set by the user for bit stream filtering >> +#define AV_OPT_FLAG_COMMAND_PARAM (1<<15) ///< a generic parameter >> which can be set by the user via commands with filtering >> #define AV_OPT_FLAG_FILTERING_PARAM (1<<16) ///< a generic parameter >> which can be set by the user for filtering >> #define AV_OPT_FLAG_DEPRECATED (1<<17) ///< set if option is >> deprecated, users should refer to AVOption.help text for more information >> //FIXME think about enc-audio, ... style flags > > this looks good to me but iam not sure about the name > because what is a command parameter? (in the context of a libavutil > AVOption) > > What this really does is it marks AVOptions which > can be changed at runtime (through commands) but if it wasnt done through > commands but by other means i think this would also work > so maybe AV_OPT_FLAG_RUNTIME_PARAM or something like that would be a more > generic concept which could be usefull in other cases >
Agreed, will rename. > Thanks > > [...] > > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > it is not once nor twice but times without number that the same ideas make > their appearance in the world. -- Aristotle > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".