Re: [PATCH 1/4] editorconfig: set maximum line length to 120 characters
From: Justin Tobler <hidden>
Date: 2025-06-27 15:18:10
On 25/06/27 01:51AM, Karthik Nayak wrote:
Justin Tobler [off-list ref] writes:quoted
On 25/06/25 06:43PM, Karthik Nayak wrote:quoted
As per 'Documentation/CodingGuidelines', we try to keep to at most 80 characters per line. However, there are often certain cases where we extend this for the sake of readability. Add a maximum limit of 120 characters to the '.editorconfig'. This means that if an individual line exceeds 120 characters, the editor will wrap that line. This provides a lot wiggle room over the recommended 80 character limit.Hello Justin,quoted
I frequently use the format operator in vim to reformat entire blocks of text and it is commonly configured to use `max_line_length` from an `.editorconfig` file to know when to wrap lines. Changing the value to 120 would cause my editor to prefer 120 character lines when reformatting, which I would personally not like.It would only wrap lines longer than 120 columns. Currently editorconfig doesn't wrap any line length. So we're essentially saying, any line above 120 is not something we want to accept and hence wrap. This doesn't mean that shorter lines will be combined together. Wouldn't this be better than the current situation?
When `max_line_length` is set in a ".editorconfig" file, in my vim editor it overrides the `textwidth` configuration which was already set to 80 by default. So changing to 120 would change line wrapping behavior for me at least. I could disable using the ".editorconfig", but I would prefer to avoid doing that :) -Justin