On 7/18/17, Derek Buitenhuis <derek.buitenh...@gmail.com> wrote:
> On 7/17/2017 3:19 PM, Nicolas George wrote:
>> Signed-off-by: Nicolas George <geo...@nsup.org>
>> ---
>>  libavfilter/framesync2.c | 166
>> +++++++++++++++++++----------------------------
>>  libavfilter/framesync2.h |  77 ++++------------------
>>  2 files changed, 82 insertions(+), 161 deletions(-)
>
> So what does this patch set *do*? The commit message has literally no
> information on what this actually does. One can say "well go read the
> entirety of the mailing list backlog", but that sure isn't a very good
> way to help e.g. people who git blame in the future. There is no
> substitute for a commit message.
>
> Also 'activate design' is gibberish in English, which doesn't help
> letting one know what it does.

I actually like this 'design', as its more like VS one.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to