[FFmpeg-devel] [PATCH] libavfilter: temporarily remove DNN framework and vf_sr filter

James Almer jamrial at gmail.com
Fri Jul 27 17:11:54 EEST 2018


On 7/27/2018 8:04 AM, Rostislav Pehlivanov wrote:
> Reference - IRC and j-b's earlier email.
> Coding style issues:
> DNNModel* ff_dnn_load_model_native(const char* model_filename)
> 
> We never ever do stupid things like put the asterix first. The author of
> the patch should have known better and the patch should have been checked.
> Even a glance could have told you its wrong.

Tone it down. It's a style issue. New contributors don't always know
things like that and we always tell them to fix it. It's the reviewer
who should have pointed it out, and if they missed it then it's
harmless. It's not like he used a public suffix like av_, where it can
be a problem.

> I described them in the sentence above.
> But I'm not willing to wait for a potential fix, and especially not for a
> whole bunch of them rewriting everything. The whole code needs to be thrown
> out and thoroughly reviewed properly, by at least yourself and one other
> person, preferably before gsoc ends.
> You should start coordinating with your student on how to fix everything
> mentioned and then resend the patchsets once fixed. I'll apply the revert
> patch tomorrow.

No, you wont. Not until this has been discussed. I don't know what got
to you but you're acting like someone pushed code that would get a cop
on your doorstep. Calm down for once in your life, you're seemingly
angry in every other email you write, and you're not helping making this
project a friendly place at all for new and old contributors alike.


More information about the ffmpeg-devel mailing list