[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