[FFmpeg-devel] Patchwork hook for bad commit messages
Lynne
dev at lynne.ee
Sun Aug 15 16:01:13 EEST 2021
15 Aug 2021, 13:39 by george at nsup.org:
> Lynne (12021-08-15):
>
>> Could you acknowledge that at least some commits do not deserve
>> mandatory 3-line vapid comments?
>>
>
> I have no idea what you are talking about.
>
> The hook I am suggesting would catch:
>
> - cases without "context:" on the first line, like:
> [FFmpeg-devel] [PATCH] Handle AV_PIX_FMT_PAL8
> [FFmpeg-devel] [PATCH v3 1/1] return value check for init_get_bits in vc1dec.c
>
> - cases with something on the second line;
>
> - possibly cases with overly long lines (but it is harder to avoid
> flagging code, code should not be wrapped; I am undecided on this).
>
Oh, that's all fine. I was talking about the change to the developer manual,
about adding patch descriptions. Sometimes, you just don't need any.
More information about the ffmpeg-devel
mailing list