[haiku-development] Re: The 80 columns limit?

  • From: Fredrik Holmqvist <fredrik.holmqvist@xxxxxxxxx>
  • To: haiku-development <haiku-development@xxxxxxxxxxxxx>
  • Date: Fri, 3 Sep 2021 06:57:47 +0000

Den fre 3 sep. 2021 kl 03:20 skrev Jessica Hamilton <
jessica.l.hamilton@xxxxxxxxx>:

On Fri, 3 Sept 2021 at 14:49, Pascal Abresch <dmarc-noreply@xxxxxxxxxxxxx>
wrote:



On 3. Sep 2021, at 00:02, Jessica Hamilton <jessica.l.hamilton@xxxxxxxxx>
wrote:


On Fri, 3 Sept 2021 at 05:16, Mr. waddlesplash <waddlesplash@xxxxxxxxx>
wrote:

Hello,

(Yes, there are more discussions on the development list this week
than normal ... sorry about that.)

At the risk of starting a bikeshed: do we think it is time to raise
the 80 columns limit somewhat?


I would prefer we stick to 80 columns, as it's better for split windows
on one monitor, which I use extensively. It also seems better for
side-by-side diffs.

Cheers,
Jessica

I agree here, I do have a 1200p monitor, but deskbar is there aswell, and
I use bigger fonts to more easily view stuff, I doubt 120 would fit nicely.

mfg,
Pascal Abresch


At most, 100 wide from me, but that's very, very begrudgingly. I'd be okay
with maybe increasing to 90, but it's still blech. If code is so heavily
indented that you need more than 100 columns, then perhaps you should be
using helper functions, as the code is already likely terrible to read.
Heavily nested branching and loops is just plain bad. And the Haiku style
guide values readability.


I don't think that is a good argument. We should not allow
nested craziness, no matter how many columns they fit in.
For longer if-cases and with proper types like
memory_region->physical_start 80 columns is not enough, and the wrapping
makes
it worse to read IMO.


But for readable side-by-side, 95 is the most I can manage split screen
when accounting for margins, line numbers, etc.

120 seems plain ridiculous, and is a firm no from me. 100 is too wide, and
90 is starting to get uncomfortable. BTW, 90 for me is equivalent to the
max-width of many web pages designed for proper readability.

We really shouldn't need more than 10 extra characters on a single line.


So I think we do need a bit more for proper names and types and for
if-conditions.
However I think the majority of lines will be <80 columns, so I don't see
this as a big problem.
The only difference is that your editor will wrap long lines, instead of a
developer spending time trying to wrap them for you.

How about we recommend 80 columns, and either with no real limit, or some
bigger value that it seems many of us want.

/Fredrik Holmqvist, TQH

Other related posts: