Re: [FFmpeg-devel] [PATCH V2 01/11] lavu/opt: add AV_OPT_FLAG_BSF_PARAM
On 2018/3/14 6:02, Michael Niedermayer wrote: > On Mon, Mar 12, 2018 at 09:23:02AM +0800, Jun Zhao wrote: >> V2: update opt fate test ref file >> opt.c |1 + >> opt.h |1 + >> 2 files changed, 2 insertions(+) >> 5e930c1851c198bd35369b87965c6eeadfd39a95 >> 0001-lavu-opt-add-AV_OPT_FLAG_BSF_PARAM.patch >> From 5cd2a18ebe1494e11b08e33ca5587f7d17f41964 Mon Sep 17 00:00:00 2001 >> From: Jun Zhao>> Date: Thu, 8 Mar 2018 13:47:23 +0800 >> Subject: [PATCH V2 01/11] lavu/opt: add AV_OPT_FLAG_BSF_PARAM >> >> add AV_OPT_FLAG_BSF_PARAM for bit stream filter options. >> >> Signed-off-by: Jun Zhao >> --- >> libavutil/opt.c | 1 + >> libavutil/opt.h | 1 + >> 2 files changed, 2 insertions(+) >> >> diff --git a/libavutil/opt.c b/libavutil/opt.c >> index df88663e3f..3b0aab4ee8 100644 >> --- a/libavutil/opt.c >> +++ b/libavutil/opt.c >> @@ -1181,6 +1181,7 @@ static void opt_list(void *obj, void *av_log_obj, >> const char *unit, >> av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & >> AV_OPT_FLAG_SUBTITLE_PARAM) ? 'S' : '.'); >> av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & >> AV_OPT_FLAG_EXPORT) ? 'X' : '.'); >> av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & >> AV_OPT_FLAG_READONLY) ? 'R' : '.'); >> +av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & >> AV_OPT_FLAG_BSF_PARAM) ? 'B' : '.'); >> >> if (opt->help) >> av_log(av_log_obj, AV_LOG_INFO, " %s", opt->help); >> diff --git a/libavutil/opt.h b/libavutil/opt.h >> index 391720f2e2..07da68ea23 100644 >> --- a/libavutil/opt.h >> +++ b/libavutil/opt.h >> @@ -287,6 +287,7 @@ typedef struct AVOption { >> * This flag only makes sense when AV_OPT_FLAG_EXPORT is also set. >> */ >> #define AV_OPT_FLAG_READONLY128 >> +#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_FILTERING_PARAM (1<<16) ///< a generic parameter which >> can be set by the user for filtering >> //FIXME think about enc-audio, ... style flags > these 2 hunks should be split into 2 seperate patches > this is needed to keep the patchset a series of patches where none breaks > fate Will split into 2 separate patches and combine option update patch (https://patchwork.ffmpeg.org/patch/7936/) with one of them. Tks. > > [...] > > > > ___ > 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
Re: [FFmpeg-devel] [PATCH V2 01/11] lavu/opt: add AV_OPT_FLAG_BSF_PARAM
On Mon, Mar 12, 2018 at 09:23:02AM +0800, Jun Zhao wrote: > V2: update opt fate test ref file > opt.c |1 + > opt.h |1 + > 2 files changed, 2 insertions(+) > 5e930c1851c198bd35369b87965c6eeadfd39a95 > 0001-lavu-opt-add-AV_OPT_FLAG_BSF_PARAM.patch > From 5cd2a18ebe1494e11b08e33ca5587f7d17f41964 Mon Sep 17 00:00:00 2001 > From: Jun Zhao> Date: Thu, 8 Mar 2018 13:47:23 +0800 > Subject: [PATCH V2 01/11] lavu/opt: add AV_OPT_FLAG_BSF_PARAM > > add AV_OPT_FLAG_BSF_PARAM for bit stream filter options. > > Signed-off-by: Jun Zhao > --- > libavutil/opt.c | 1 + > libavutil/opt.h | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/libavutil/opt.c b/libavutil/opt.c > index df88663e3f..3b0aab4ee8 100644 > --- a/libavutil/opt.c > +++ b/libavutil/opt.c > @@ -1181,6 +1181,7 @@ static void opt_list(void *obj, void *av_log_obj, const > char *unit, > av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & > AV_OPT_FLAG_SUBTITLE_PARAM) ? 'S' : '.'); > av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & > AV_OPT_FLAG_EXPORT) ? 'X' : '.'); > av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & > AV_OPT_FLAG_READONLY) ? 'R' : '.'); > +av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & > AV_OPT_FLAG_BSF_PARAM) ? 'B' : '.'); > > if (opt->help) > av_log(av_log_obj, AV_LOG_INFO, " %s", opt->help); > diff --git a/libavutil/opt.h b/libavutil/opt.h > index 391720f2e2..07da68ea23 100644 > --- a/libavutil/opt.h > +++ b/libavutil/opt.h > @@ -287,6 +287,7 @@ typedef struct AVOption { > * This flag only makes sense when AV_OPT_FLAG_EXPORT is also set. > */ > #define AV_OPT_FLAG_READONLY128 > +#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_FILTERING_PARAM (1<<16) ///< a generic parameter which > can be set by the user for filtering > //FIXME think about enc-audio, ... style flags these 2 hunks should be split into 2 seperate patches this is needed to keep the patchset a series of patches where none breaks fate [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The bravest are surely those who have the clearest vision of what is before them, glory and danger alike, and yet notwithstanding go out to meet it. -- Thucydides signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH V2 01/11] lavu/opt: add AV_OPT_FLAG_BSF_PARAM
On 2018/3/12 22:23, James Almer wrote: > On 3/11/2018 10:23 PM, Jun Zhao wrote: >> V2: update opt fate test ref file > Unrelated to this patch, but can you please make sure to the patchsets > you send are contained in a single thread? Every email in a set after > the first should be a reply to it. Now I working in a unfriendly network environment for open source development/ patches submit, so I can't use git send-mail, I use a thunderbird GUI mail client send the patch with the attached file, this is the reason I can't send the patchsets in a single thread. > ___ > 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
Re: [FFmpeg-devel] [PATCH V2 01/11] lavu/opt: add AV_OPT_FLAG_BSF_PARAM
On 3/11/2018 10:23 PM, Jun Zhao wrote: > V2: update opt fate test ref file Unrelated to this patch, but can you please make sure to the patchsets you send are contained in a single thread? Every email in a set after the first should be a reply to it. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH V2 01/11] lavu/opt: add AV_OPT_FLAG_BSF_PARAM
V2: update opt fate test ref file From 5cd2a18ebe1494e11b08e33ca5587f7d17f41964 Mon Sep 17 00:00:00 2001 From: Jun ZhaoDate: Thu, 8 Mar 2018 13:47:23 +0800 Subject: [PATCH V2 01/11] lavu/opt: add AV_OPT_FLAG_BSF_PARAM add AV_OPT_FLAG_BSF_PARAM for bit stream filter options. Signed-off-by: Jun Zhao --- libavutil/opt.c | 1 + libavutil/opt.h | 1 + 2 files changed, 2 insertions(+) diff --git a/libavutil/opt.c b/libavutil/opt.c index df88663e3f..3b0aab4ee8 100644 --- a/libavutil/opt.c +++ b/libavutil/opt.c @@ -1181,6 +1181,7 @@ static void opt_list(void *obj, void *av_log_obj, const char *unit, av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_SUBTITLE_PARAM) ? 'S' : '.'); av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_EXPORT) ? 'X' : '.'); av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_READONLY) ? 'R' : '.'); +av_log(av_log_obj, AV_LOG_INFO, "%c", (opt->flags & AV_OPT_FLAG_BSF_PARAM) ? 'B' : '.'); if (opt->help) av_log(av_log_obj, AV_LOG_INFO, " %s", opt->help); diff --git a/libavutil/opt.h b/libavutil/opt.h index 391720f2e2..07da68ea23 100644 --- a/libavutil/opt.h +++ b/libavutil/opt.h @@ -287,6 +287,7 @@ typedef struct AVOption { * This flag only makes sense when AV_OPT_FLAG_EXPORT is also set. */ #define AV_OPT_FLAG_READONLY128 +#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_FILTERING_PARAM (1<<16) ///< a generic parameter which can be set by the user for filtering //FIXME think about enc-audio, ... style flags -- 2.14.1 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel