Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend character limit to 100 characters #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GallaisPoutine
Copy link

80 character limit is now deprecated[1][2]. This patch extends the character
limit to 100 characters.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=bdc48fa11e46f867ea4d75fa59ee87a7f48be144
[2] https://lkml.org/lkml/2020/5/29/1038

Signed-off-by: Raphaël Gallais-Pou [email protected]

80 character limit is now deprecated[1][2]. This patch extends the character
limit to 100 characters.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=bdc48fa11e46f867ea4d75fa59ee87a7f48be144
[2] https://lkml.org/lkml/2020/5/29/1038

Signed-off-by: Raphaël Gallais-Pou <[email protected]>
@gregkh
Copy link

gregkh commented Dec 20, 2022

note, I've merged this here: gregkh@1593698

diederikdehaas added a commit to diederikdehaas/kernel-coding-style that referenced this pull request Oct 16, 2024
Today I was reminded again that the current line length limit isn't 80
chars, but 100 chars, so I wanted to submit a PR to fix that in the
original repo, only to find someone else already did (PR 25).
And that's when I found this repo.
While the rule was updated, the README wasn't, so do that now.

Note that the current upstream 'Linux kernel coding style' does NOT
mention the 100 char limit, but only mentions the preferred max length
of 80.

Signed-off-by: Diederik de Haas <[email protected]>
Link: vivien#25
Link: https://git.kernel.org/linus/bdc48fa11e46f867ea4d75fa59ee87a7f48be144
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants