[FFmpeg-devel] [PATCH 1/8] avutil/opt: add AV_OPT_FLAG_COMMAND_PARAM flag

Michael Niedermayer michael at niedermayer.cc
Wed Oct 9 17:19:12 EEST 2019


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&friendly otherwise

Thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is what and why we do it that matters, not just one of them.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20191009/e7d306ae/attachment.sig>


More information about the ffmpeg-devel mailing list