Re: [FFmpeg-devel] [PATCH 01/14] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-10 Thread Nicolas George
Michael Niedermayer (12019-10-09): > 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 thro

Re: [FFmpeg-devel] [PATCH 01/14] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Paul B Mahol
On 10/9/19, Michael Niedermayer wrote: > On Wed, Oct 09, 2019 at 12:16:52PM +0200, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> libavutil/opt.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/libavutil/opt.h b/libavutil/opt.h >> index 39f4a8dda0..693a170bcc 100644 >>

Re: [FFmpeg-devel] [PATCH 01/14] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Michael Niedermayer
On Wed, Oct 09, 2019 at 12:16:52PM +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > 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 > @

[FFmpeg-devel] [PATCH 01/14] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

2019-10-09 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- 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