diff options
| author | Dudemanguy <random342@airmail.cc> | 2023-10-21 10:28:30 -0500 |
|---|---|---|
| committer | Dudemanguy <random342@airmail.cc> | 2023-10-22 14:31:36 +0000 |
| commit | 7768c3d035a5d483400bedbab24a37b05c0739c3 (patch) | |
| tree | da33faebaa677cd88cade2ab40fa3e4e8ab67adc | |
| parent | 1382a854c9f80d6519d060a5796accaf1328c93e (diff) | |
| download | mpv-7768c3d035a5d483400bedbab24a37b05c0739c3.tar.bz2 mpv-7768c3d035a5d483400bedbab24a37b05c0739c3.tar.xz | |
DOCS/contribute: increase hard column limit to 100
I'm guilty of violating this, but surely I can't be the only one. 85 is
pretty small and there's plenty of lines in the codebase that go well
over that. Surely nobody programs on tiny screens anymore and the kernel
raised the limit to 100 a few years ago so let's just copy that.
| -rw-r--r-- | DOCS/contribute.md | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/DOCS/contribute.md b/DOCS/contribute.md index 8464ddab8b..91422a7289 100644 --- a/DOCS/contribute.md +++ b/DOCS/contribute.md @@ -153,9 +153,9 @@ mpv uses C11 with K&R formatting, with some exceptions. some_function(a, b, c); } ``` -- Break lines on 80 columns. There is a hard limit of 85 columns. You may ignore +- Break lines on 80 columns. There is a hard limit of 100 columns. You may ignore this limit if there's a strong case that not breaking the line will increase - readability. Going over 85 columns might provoke endless discussions about + readability. Going over 100 columns might provoke endless discussions about whether such a limit is needed or not, so avoid it. - If the body of an if/for/while statement has more than 1 physical lines, then always add braces, even if they're technically redundant. |