Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
On Wed, Oct 09, 2019 at 11:24:55AM +0200, Paul B Mahol wrote: > Because this receives no comments so fat at all, will apply asap this set. I think you should be a bit more patient, threatening to apply after only about 24h. Just causes people who have an interrest in the code but cannot immedeatly review it to be upset ... I think the patchset is all fine but really i also have not had time to test it yet nor did i fully review it just had a quick look. What i would suggest is to give interrested developers time to review and comment. Not threaten to apply so quickly, you can see here it just ended in escalation. And i think this thread would have been much more peacefull otherwise Thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB It is what and why we do it that matters, not just one of them. signature.asc Description: PGP signature ___ 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".
Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
On 10/9/19, Nicolas George wrote: > Paul B Mahol (12019-10-09): >> Such interpretation is not in documentation. > >> Such line is not in MAINTAINERS. > >> This is violent attack. > >> I do not believe in your good faith regarding me ever. > > I will not argue further with you. You have been warned. > Finally you showed your true self! > Regards, > > -- > Nicolas George > ___ 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".
Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
Paul B Mahol (12019-10-09): > Such interpretation is not in documentation. > Such line is not in MAINTAINERS. > This is violent attack. > I do not believe in your good faith regarding me ever. I will not argue further with you. You have been warned. Regards, -- Nicolas George signature.asc Description: PGP signature ___ 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".
Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
On 10/9/19, Nicolas George wrote: > Paul B Mahol (12019-10-09): >> This is small patch > > Not when it touches public API. Such interpretation is not in documentation. > > Anyway, I maintain lavfi, and I tell you not to commit before I > reviewed. Such line is not in MAINTAINERS. > > Break the rules and I will make sure that there are consequences. This is violent attack. > > By the way, I had some time this morning, and I had considered reviewing > precisely these patches during this time. Your aggression against me > (yes, telling somebody you will ignore their legitimate advice is an > aggression) put me in a frame of mind that prevents doing a fair review. > Good job working against your own interests. I do not believe in your good faith regarding me ever. > > Regards, > > -- > Nicolas George > ___ 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".
Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
Paul B Mahol (12019-10-09): > This is small patch Not when it touches public API. Anyway, I maintain lavfi, and I tell you not to commit before I reviewed. Break the rules and I will make sure that there are consequences. By the way, I had some time this morning, and I had considered reviewing precisely these patches during this time. Your aggression against me (yes, telling somebody you will ignore their legitimate advice is an aggression) put me in a frame of mind that prevents doing a fair review. Good job working against your own interests. Regards, -- Nicolas George signature.asc Description: PGP signature ___ 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".
Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
On 10/9/19, Nicolas George wrote: > Paul B Mahol (12019-10-09): >> This only adds single flag. > > A flag that belongs in the public API, and therefore that we will have > to support for a long time. Hence the need to make sure it is properly > designed. > >> Out of question. Your reply with all due respect is ignored. > > This is not respect, this is aggression. One week for important patches, > that is the common policy, and I will see it applied. This is small patch, and can be applied within three days. Nobody is attacking you, since when stating ignorance is considered aggression? > > Once again, this is not your own personal project, this is a > collaborative endeavour, and rules applies to everybody. Except that rule does not apply to you and usual other suspects. > > Regards, > > -- > Nicolas George > ___ 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".
Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
Paul B Mahol (12019-10-09): > This only adds single flag. A flag that belongs in the public API, and therefore that we will have to support for a long time. Hence the need to make sure it is properly designed. > Out of question. Your reply with all due respect is ignored. This is not respect, this is aggression. One week for important patches, that is the common policy, and I will see it applied. Once again, this is not your own personal project, this is a collaborative endeavour, and rules applies to everybody. Regards, -- Nicolas George signature.asc Description: PGP signature ___ 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".
Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
On 10/9/19, Nicolas George wrote: > Paul B Mahol (12019-10-09): >> Because this receives no comments so fat at all, will apply asap this >> set. > > Less than 24 hours for something that changes the public API? Out of the > question. This only adds single flag. > > If in one week you had no comments, you can ask again that people look > into it. Out of question. Your reply with all due respect is ignored. > > Regards, > > -- > Nicolas George > ___ 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".
Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
Paul B Mahol (12019-10-09): > Because this receives no comments so fat at all, will apply asap this set. Less than 24 hours for something that changes the public API? Out of the question. If in one week you had no comments, you can ask again that people look into it. Regards, -- Nicolas George signature.asc Description: PGP signature ___ 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".
Re: [FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag
Because this receives no comments so fat at all, will apply asap this set.
On 10/8/19, 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
> @@ -288,6 +288,7 @@ typedef struct AVOption {
> */
> #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_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
> --
> 2.17.1
>
>
___
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".
