On Thu, 8 May 2025 at 11:32, Philippe Mathieu-Daudé <phi...@linaro.org> wrote: > FWIW I noticed the patch subject is truncated to my default git-view > because it is over 72 chars. Since there is no enforcement on patch > subject / description lines length in checkpatch.pl I suppose nobody > really cares about that so I'll merge as is.
I treat subject length stuff as an "it would be nice to be below this" thing -- I try to keep them from being overlong lines. But sometimes you really can't express what the patch is doing in that length limit, and in those cases I favour having a slightly long subject line over having one that doesn't say what the patch is doing. In the description part, we should be keeping away from overlength lines by having them wrap at 75 chars or less, unless there's a strong reason to not wrap (usually it's a URL or we're quoting an error message). thanks -- PMM
