* [RFC] Increase libstdc++ line length to 100(?) columns
@ 2020-11-26 23:50 Jonathan Wakely
2020-11-27 2:45 ` Liu Hao
` (3 more replies)
0 siblings, 4 replies; 15+ messages in thread
From: Jonathan Wakely @ 2020-11-26 23:50 UTC (permalink / raw)
To: libstdc++; +Cc: gcc
I've touched on the subject a few times, e.g.
https://gcc.gnu.org/pipermail/gcc/2019-December/230993.html
and https://gcc.gnu.org/pipermail/gcc/2019-December/231013.html
Libstdc++ code is indented by 2 columns for the enclosing namespace,
usually another two for being in a template, and is full of __
prefixes for reserved names. On top of that, modern C++ declarations
are *noisy* (template head, requires-clause, noexcept-specifier, often
'constexpr' or 'inline' and 'explicit', and maybe some attributes.
All that gets hard to fit in 80 columns without compromising
readability with line breaks in unnatural places.
Does anybody object to raising the line length for libstdc++ code
(not the rest of GCC) to 100 columns?
Please read my replies in the thread linked above before telling me
that the code should be split up into smaller functions to avoid deep
nesting. The function I pointed to cannot easily be split up without
making the code slower to compile and potentially slower to run:
https://gcc.gnu.org/git/?p=gcc.git;a=blob;f=libstdc%2B%2B-v3/include/bits/stl_algobase.h;h=a2fd306e6d0cca579b510148ba1a7089e2b2f3a2;hb=HEAD#l1499
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-26 23:50 [RFC] Increase libstdc++ line length to 100(?) columns Jonathan Wakely
@ 2020-11-27 2:45 ` Liu Hao
2020-11-27 8:14 ` Richard Biener
2020-11-27 11:08 ` Allan Sandfeld Jensen
` (2 subsequent siblings)
3 siblings, 1 reply; 15+ messages in thread
From: Liu Hao @ 2020-11-27 2:45 UTC (permalink / raw)
To: Jonathan Wakely, libstdc++; +Cc: gcc
[-- Attachment #1.1: Type: text/plain, Size: 1841 bytes --]
在 2020/11/27 上午7:50, Jonathan Wakely via Gcc 写道:
> I've touched on the subject a few times, e.g.
> https://gcc.gnu.org/pipermail/gcc/2019-December/230993.html
> and https://gcc.gnu.org/pipermail/gcc/2019-December/231013.html
>
> Libstdc++ code is indented by 2 columns for the enclosing namespace,
> usually another two for being in a template, and is full of __
> prefixes for reserved names. On top of that, modern C++ declarations
> are *noisy* (template head, requires-clause, noexcept-specifier, often
> 'constexpr' or 'inline' and 'explicit', and maybe some attributes.
>
> All that gets hard to fit in 80 columns without compromising
> readability with line breaks in unnatural places.
>
I think I want to vote +1 for this. On my 1920x1080 laptop screen with an 11pt monospace font, 100
colons allows me to open two terminals side by side, while still providing 3 colons for line
numbers. On a larger desktop screen with a 10pt font it'd be 132 colomns, but more often I find
lines longer than 110 characters hard to read, so I agree with 100 (but I suggest making it a
'recommended limit' instead of a 'hard limit' anyway).
There was a small fragment of code in <https://gcc.gnu.org/pipermail/gcc/2019-December/231003.html>:
> if (present)
> ptr
> = gfc_build_conditional_assign_expr (block, present,
> ptr, nullarg);
Why not change this to:
> if (present)
> ptr = gfc_build_conditional_assign_expr (
> block, present, ptr, nullarg);
>
I think it looks balanced and way more comfortable, and doesn't waste much leading space.
--
Best regards,
LH_Mouse
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-27 2:45 ` Liu Hao
@ 2020-11-27 8:14 ` Richard Biener
2020-11-27 8:21 ` Ville Voutilainen
2020-11-27 9:49 ` Liu Hao
0 siblings, 2 replies; 15+ messages in thread
From: Richard Biener @ 2020-11-27 8:14 UTC (permalink / raw)
To: Liu Hao; +Cc: Jonathan Wakely, libstdc++, GCC Development
On Fri, Nov 27, 2020 at 3:48 AM Liu Hao via Gcc <gcc@gcc.gnu.org> wrote:
>
> 在 2020/11/27 上午7:50, Jonathan Wakely via Gcc 写道:
> > I've touched on the subject a few times, e.g.
> > https://gcc.gnu.org/pipermail/gcc/2019-December/230993.html
> > and https://gcc.gnu.org/pipermail/gcc/2019-December/231013.html
> >
> > Libstdc++ code is indented by 2 columns for the enclosing namespace,
> > usually another two for being in a template, and is full of __
> > prefixes for reserved names. On top of that, modern C++ declarations
> > are *noisy* (template head, requires-clause, noexcept-specifier, often
> > 'constexpr' or 'inline' and 'explicit', and maybe some attributes.
> >
> > All that gets hard to fit in 80 columns without compromising
> > readability with line breaks in unnatural places.
> >
>
> I think I want to vote +1 for this. On my 1920x1080 laptop screen with an 11pt monospace font, 100
> colons allows me to open two terminals side by side, while still providing 3 colons for line
> numbers. On a larger desktop screen with a 10pt font it'd be 132 colomns, but more often I find
> lines longer than 110 characters hard to read, so I agree with 100 (but I suggest making it a
> 'recommended limit' instead of a 'hard limit' anyway).
>
>
> There was a small fragment of code in <https://gcc.gnu.org/pipermail/gcc/2019-December/231003.html>:
>
> > if (present)
> > ptr
> > = gfc_build_conditional_assign_expr (block, present,
> > ptr, nullarg);
>
> Why not change this to:
>
> > if (present)
> > ptr = gfc_build_conditional_assign_expr (
> > block, present, ptr, nullarg);
> >
>
> I think it looks balanced and way more comfortable, and doesn't waste much leading space.
Other places use
if (present)
ptr = gfc_build_conditional_assign_expr
(block, present, ptr, nullarg);
I prefer the ( on the next line. The argument list can be two spaces
indented from
the function name or "right justified" (I think the latter looks
visually better).
Richard.
>
>
>
> --
> Best regards,
> LH_Mouse
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-27 8:14 ` Richard Biener
@ 2020-11-27 8:21 ` Ville Voutilainen
2020-11-27 9:49 ` Liu Hao
1 sibling, 0 replies; 15+ messages in thread
From: Ville Voutilainen @ 2020-11-27 8:21 UTC (permalink / raw)
To: Richard Biener; +Cc: Liu Hao, GCC Development, Jonathan Wakely, libstdc++
On Fri, 27 Nov 2020 at 10:16, Richard Biener via Libstdc++
<libstdc++@gcc.gnu.org> wrote:
> > Why not change this to:
> >
> > > if (present)
> > > ptr = gfc_build_conditional_assign_expr (
> > > block, present, ptr, nullarg);
> > >
> >
> > I think it looks balanced and way more comfortable, and doesn't waste much leading space.
>
> Other places use
>
> if (present)
> ptr = gfc_build_conditional_assign_expr
> (block, present, ptr, nullarg);
>
> I prefer the ( on the next line. The argument list can be two spaces
> indented from
> the function name or "right justified" (I think the latter looks
> visually better).
I find it easier to grok the code when the opening paren is on the
first line, I know instantly that I'm
looking at a function call. That sort of style probably fits better
the C++ code in libstdc++ than the code
in gcc, because in libstdc++ code we don't have a space before the
argument list.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-27 8:14 ` Richard Biener
2020-11-27 8:21 ` Ville Voutilainen
@ 2020-11-27 9:49 ` Liu Hao
2020-11-27 10:13 ` Ville Voutilainen
1 sibling, 1 reply; 15+ messages in thread
From: Liu Hao @ 2020-11-27 9:49 UTC (permalink / raw)
To: Richard Biener; +Cc: Jonathan Wakely, libstdc++, GCC Development
[-- Attachment #1.1: Type: text/plain, Size: 1521 bytes --]
在 2020/11/27 下午4:14, Richard Biener 写道:
>
> I prefer the ( on the next line. The argument list can be two spaces
> indented from
> the function name or "right justified" (I think the latter looks
> visually better).
>
The right justification thing looks reasonable. For example, I think this
```c++
basic_cow_string(const basic_cow_string& other)
noexcept
: m_sth(allocator_traits<allocator_type>::select_on_container_copy_construction(
other.m_sth.as_allocator()))
{ this->assign(other); }
```
looks better than
```c++
basic_cow_string(const basic_cow_string& other)
noexcept
: m_sth(allocator_traits<allocator_type>::select_on_container_copy_construction(
other.m_sth.as_allocator()))
{ this->assign(other); }
```
In the former fragment, indention of the 4th line is probably arbitrary. The only purpose is that,
if there wasn't a line break, the 3rd line would exceed a given length limit.
As you can see, qualified names in C++ can grow up to ~100 characters quite frequently. This may
deteriorate when `typename` and `template` are sometimes required. I don't think there is
practically a set of rules which governs all cases. So, if something looks better, go for it, and
that's why I think a (suggested) 100-char limit is better than the conventional 80-char limit, which
forces another line break in front of `select_on_container_copy_construction`.
--
Best regards,
LH_Mouse
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-27 9:49 ` Liu Hao
@ 2020-11-27 10:13 ` Ville Voutilainen
0 siblings, 0 replies; 15+ messages in thread
From: Ville Voutilainen @ 2020-11-27 10:13 UTC (permalink / raw)
To: Liu Hao; +Cc: Richard Biener, GCC Development, Jonathan Wakely, libstdc++
On Fri, 27 Nov 2020 at 11:54, Liu Hao via Libstdc++
<libstdc++@gcc.gnu.org> wrote:
> As you can see, qualified names in C++ can grow up to ~100 characters quite frequently. This may
> deteriorate when `typename` and `template` are sometimes required. I don't think there is
> practically a set of rules which governs all cases. So, if something looks better, go for it, and
> that's why I think a (suggested) 100-char limit is better than the conventional 80-char limit, which
> forces another line break in front of `select_on_container_copy_construction`.
I do have a general question/thought/rumination here, though.
Shouldn't the paren-style and the line length
of libstdc++ be mainly decided by those who develop and maintain it?
It's already not written in the same style
as gcc is, so tweaking that different style to better suit the need of
a template-heavy C++ library perhaps
should be more or less a slam dunk? :)
Despite my sizable contributions to libstd++, I don't have a
particularly strong opinion here. Except that,
my somewhat strong opinion is "let's give Jonathan what he wants,
because it helps his work". ;)
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-26 23:50 [RFC] Increase libstdc++ line length to 100(?) columns Jonathan Wakely
2020-11-27 2:45 ` Liu Hao
@ 2020-11-27 11:08 ` Allan Sandfeld Jensen
2020-11-29 17:38 ` Florian Weimer
2020-12-01 3:05 ` Thomas Rodgers
2020-12-01 9:25 ` Matthias Kretz
3 siblings, 1 reply; 15+ messages in thread
From: Allan Sandfeld Jensen @ 2020-11-27 11:08 UTC (permalink / raw)
To: libstdc++, gcc; +Cc: gcc, Jonathan Wakely
On Freitag, 27. November 2020 00:50:57 CET Jonathan Wakely via Gcc wrote:
> I've touched on the subject a few times, e.g.
> https://gcc.gnu.org/pipermail/gcc/2019-December/230993.html
> and https://gcc.gnu.org/pipermail/gcc/2019-December/231013.html
>
> Libstdc++ code is indented by 2 columns for the enclosing namespace,
> usually another two for being in a template, and is full of __
> prefixes for reserved names. On top of that, modern C++ declarations
> are *noisy* (template head, requires-clause, noexcept-specifier, often
> 'constexpr' or 'inline' and 'explicit', and maybe some attributes.
>
> All that gets hard to fit in 80 columns without compromising
> readability with line breaks in unnatural places.
>
> Does anybody object to raising the line length for libstdc++ code
> (not the rest of GCC) to 100 columns?
>
If you _do_ change it. I would suggest changing it to 120, which is next
common step for a lot of C++ projects.
Often also with an allowance for overruns if that makes the code cleaner.
'Allan
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-27 11:08 ` Allan Sandfeld Jensen
@ 2020-11-29 17:38 ` Florian Weimer
2020-11-29 18:51 ` Allan Sandfeld Jensen
2020-12-03 12:11 ` Richard Earnshaw
0 siblings, 2 replies; 15+ messages in thread
From: Florian Weimer @ 2020-11-29 17:38 UTC (permalink / raw)
To: Allan Sandfeld Jensen; +Cc: libstdc++, gcc, Jonathan Wakely
* Allan Sandfeld Jensen:
> If you _do_ change it. I would suggest changing it to 120, which is next
> common step for a lot of C++ projects.
120 can be problematic for a full HD screen in portrait mode. Nine
pixels per character is not a lot (it's what VGA used), and you can't
have any window decoration. With a good font and screen, it's doable.
But if the screen isn't quite sharp, then I think you wouldn't be able
to use portrait mode anymore.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-29 17:38 ` Florian Weimer
@ 2020-11-29 18:51 ` Allan Sandfeld Jensen
2020-11-30 15:47 ` Michael Matz
2020-12-03 12:11 ` Richard Earnshaw
1 sibling, 1 reply; 15+ messages in thread
From: Allan Sandfeld Jensen @ 2020-11-29 18:51 UTC (permalink / raw)
To: Florian Weimer; +Cc: libstdc++, gcc, Jonathan Wakely
On Sonntag, 29. November 2020 18:38:15 CET Florian Weimer wrote:
> * Allan Sandfeld Jensen:
> > If you _do_ change it. I would suggest changing it to 120, which is next
> > common step for a lot of C++ projects.
>
> 120 can be problematic for a full HD screen in portrait mode. Nine
> pixels per character is not a lot (it's what VGA used), and you can't
> have any window decoration. With a good font and screen, it's doable.
> But if the screen isn't quite sharp, then I think you wouldn't be able
> to use portrait mode anymore.
Using a standard condensed monospace font of 9px, it has a width of 7px, 120
char would take up 940px fitting two windows in horizontal mode and one in
vertical. 9px isn't fuzzy, and 8px variants are even narrower.
Sure using square monospace fonts might not fit, but that is an unusual
configuration and easily worked around by living with a non-square monospace
font, or accepting occational line overflow. Remember nobody is suggesting
every line should be that long, just allowing it to allow better structural
indentation.
'Allan
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-29 18:51 ` Allan Sandfeld Jensen
@ 2020-11-30 15:47 ` Michael Matz
2020-11-30 16:28 ` Allan Sandfeld Jensen
0 siblings, 1 reply; 15+ messages in thread
From: Michael Matz @ 2020-11-30 15:47 UTC (permalink / raw)
To: Allan Sandfeld Jensen; +Cc: Florian Weimer, gcc, Jonathan Wakely, libstdc++
Hello,
On Sun, 29 Nov 2020, Allan Sandfeld Jensen wrote:
> On Sonntag, 29. November 2020 18:38:15 CET Florian Weimer wrote:
> > * Allan Sandfeld Jensen:
> > > If you _do_ change it. I would suggest changing it to 120, which is next
> > > common step for a lot of C++ projects.
> >
> > 120 can be problematic for a full HD screen in portrait mode. Nine
> > pixels per character is not a lot (it's what VGA used), and you can't
> > have any window decoration. With a good font and screen, it's doable.
> > But if the screen isn't quite sharp, then I think you wouldn't be able
> > to use portrait mode anymore.
>
> Using a standard condensed monospace font of 9px, it has a width of 7px, 120
A char width of 7px implies a cell width of at least 8px (so 960px for 120
chars), more often of 9px. With your cell width of 7px your characters
will be max 6px, symmetric characters will be 5px, which is really small.
> char would take up 940px fitting two windows in horizontal mode and one in
> vertical. 9px isn't fuzzy, and 8px variants are even narrower.
Well, and if you're fine with a 5px cell-width font then you can even fit
216 chars on a line in HD portrait mode. But Florian posed the width of
9px, and I agree with him that it's not a lot (if my monitor weren't as
big as it is I would need to use an even wider font for comfortable
reading, as it is 9px width are exactly right for me, I'm not using
portrait, though). So, it's the question if the line lengths should or
should not cater for this situation.
> Sure using square monospace fonts might not fit, but that is an unusual
> configuration and easily worked around by living with a non-square monospace
> font, or accepting occational line overflow. Remember nobody is suggesting
> every line should be that long, just allowing it to allow better structural
> indentation.
The occasional line overflow will automatically become the usual case with
time, space allowed to be filled will eventually be filled.
Ciao,
Michael.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-30 15:47 ` Michael Matz
@ 2020-11-30 16:28 ` Allan Sandfeld Jensen
2020-11-30 16:44 ` Michael Matz
0 siblings, 1 reply; 15+ messages in thread
From: Allan Sandfeld Jensen @ 2020-11-30 16:28 UTC (permalink / raw)
To: Michael Matz; +Cc: Florian Weimer, gcc, Jonathan Wakely, libstdc++
On Montag, 30. November 2020 16:47:08 CET Michael Matz wrote:
> Hello,
>
> On Sun, 29 Nov 2020, Allan Sandfeld Jensen wrote:
> > On Sonntag, 29. November 2020 18:38:15 CET Florian Weimer wrote:
> > > * Allan Sandfeld Jensen:
> > > > If you _do_ change it. I would suggest changing it to 120, which is
> > > > next
> > > > common step for a lot of C++ projects.
> > >
> > > 120 can be problematic for a full HD screen in portrait mode. Nine
> > > pixels per character is not a lot (it's what VGA used), and you can't
> > > have any window decoration. With a good font and screen, it's doable.
> > > But if the screen isn't quite sharp, then I think you wouldn't be able
> > > to use portrait mode anymore.
> >
> > Using a standard condensed monospace font of 9px, it has a width of 7px,
> > 120
> A char width of 7px implies a cell width of at least 8px (so 960px for 120
> chars), more often of 9px. With your cell width of 7px your characters
> will be max 6px, symmetric characters will be 5px, which is really small.
>
I was talking about the full cell width. I tested it before commenting,
measuring the width in pixels of a line of text.
'Allan
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-30 16:28 ` Allan Sandfeld Jensen
@ 2020-11-30 16:44 ` Michael Matz
0 siblings, 0 replies; 15+ messages in thread
From: Michael Matz @ 2020-11-30 16:44 UTC (permalink / raw)
To: Allan Sandfeld Jensen; +Cc: Florian Weimer, gcc, Jonathan Wakely, libstdc++
Hello,
On Mon, 30 Nov 2020, Allan Sandfeld Jensen wrote:
> > > On Sonntag, 29. November 2020 18:38:15 CET Florian Weimer wrote:
> > > > * Allan Sandfeld Jensen:
> > > > > If you _do_ change it. I would suggest changing it to 120, which is
> > > > > next
> > > > > common step for a lot of C++ projects.
> > > >
> > > > 120 can be problematic for a full HD screen in portrait mode. Nine
> > > > pixels per character is not a lot (it's what VGA used), and you can't
> > > > have any window decoration. With a good font and screen, it's doable.
> > > > But if the screen isn't quite sharp, then I think you wouldn't be able
> > > > to use portrait mode anymore.
> > >
> > > Using a standard condensed monospace font of 9px, it has a width of 7px,
> > > 120
> > A char width of 7px implies a cell width of at least 8px (so 960px for 120
> > chars), more often of 9px. With your cell width of 7px your characters
> > will be max 6px, symmetric characters will be 5px, which is really small.
> >
> I was talking about the full cell width. I tested it before commenting,
> measuring the width in pixels of a line of text.
Yes, and I was saying that a cell width of 7px is very narrow because the
characters itself will only be using 5px or 6px max (to leave room for
inter-character spacing in normal words). You might be fine with such
narrow characters, but not everyone will be.
Ciao,
Michael.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-26 23:50 [RFC] Increase libstdc++ line length to 100(?) columns Jonathan Wakely
2020-11-27 2:45 ` Liu Hao
2020-11-27 11:08 ` Allan Sandfeld Jensen
@ 2020-12-01 3:05 ` Thomas Rodgers
2020-12-01 9:25 ` Matthias Kretz
3 siblings, 0 replies; 15+ messages in thread
From: Thomas Rodgers @ 2020-12-01 3:05 UTC (permalink / raw)
To: libstdc++
+1
I don't have a firm opinion the discussion of 100 or 120 elsewhere, but
I like the suggestion to allow some "slop" if it clearly improves
readability.
Jonathan Wakely via Libstdc++ writes:
> I've touched on the subject a few times, e.g.
> https://gcc.gnu.org/pipermail/gcc/2019-December/230993.html
> and https://gcc.gnu.org/pipermail/gcc/2019-December/231013.html
>
> Libstdc++ code is indented by 2 columns for the enclosing namespace,
> usually another two for being in a template, and is full of __
> prefixes for reserved names. On top of that, modern C++ declarations
> are *noisy* (template head, requires-clause, noexcept-specifier, often
> 'constexpr' or 'inline' and 'explicit', and maybe some attributes.
>
> All that gets hard to fit in 80 columns without compromising
> readability with line breaks in unnatural places.
>
> Does anybody object to raising the line length for libstdc++ code
> (not the rest of GCC) to 100 columns?
>
> Please read my replies in the thread linked above before telling me
> that the code should be split up into smaller functions to avoid deep
> nesting. The function I pointed to cannot easily be split up without
> making the code slower to compile and potentially slower to run:
> https://gcc.gnu.org/git/?p=gcc.git;a=blob;f=libstdc%2B%2B-v3/include/bits/stl_algobase.h;h=a2fd306e6d0cca579b510148ba1a7089e2b2f3a2;hb=HEAD#l1499
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-26 23:50 [RFC] Increase libstdc++ line length to 100(?) columns Jonathan Wakely
` (2 preceding siblings ...)
2020-12-01 3:05 ` Thomas Rodgers
@ 2020-12-01 9:25 ` Matthias Kretz
3 siblings, 0 replies; 15+ messages in thread
From: Matthias Kretz @ 2020-12-01 9:25 UTC (permalink / raw)
To: libstdc++
On Freitag, 27. November 2020 00:50:57 CET Jonathan Wakely via Libstdc++
wrote:
> Does anybody object to raising the line length for libstdc++ code
> (not the rest of GCC) to 100 columns?
Having reformatted *a lot* of code to libstdc++ style recently, the 80-column
limit had me scratching my head and investing time better spent elsewhere
(IMHO) way too much.
I'd object to staying at 80, FWIW. ;-) Going to 100 cols is an improvement.
And I agree with others in this thread, it shouldn't be a hard limit. If the
semicolon lands on column 101 does an additional linebreak really help
readability?
There's another topic that came up in this thread: how to best indent after a
line break. After indenting way too much code manually, I wrote https://
github.com/mattkretz/vim-gnuindent. You can see how it indents in its
test.cpp. Maybe this is useful for some of you. :-)
Cheers,
Matthias
--
──────────────────────────────────────────────────────────────────────────
Dr. Matthias Kretz https://mattkretz.github.io
GSI Helmholtz Centre for Heavy Ion Research https://gsi.de
std::experimental::simd https://github.com/VcDevel/std-simd
──────────────────────────────────────────────────────────────────────────
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [RFC] Increase libstdc++ line length to 100(?) columns
2020-11-29 17:38 ` Florian Weimer
2020-11-29 18:51 ` Allan Sandfeld Jensen
@ 2020-12-03 12:11 ` Richard Earnshaw
1 sibling, 0 replies; 15+ messages in thread
From: Richard Earnshaw @ 2020-12-03 12:11 UTC (permalink / raw)
To: Florian Weimer, Allan Sandfeld Jensen; +Cc: gcc, Jonathan Wakely, libstdc++
On 29/11/2020 17:38, Florian Weimer wrote:
> * Allan Sandfeld Jensen:
>
>> If you _do_ change it. I would suggest changing it to 120, which is next
>> common step for a lot of C++ projects.
>
> 120 can be problematic for a full HD screen in portrait mode. Nine
> pixels per character is not a lot (it's what VGA used), and you can't
> have any window decoration. With a good font and screen, it's doable.
> But if the screen isn't quite sharp, then I think you wouldn't be able
> to use portrait mode anymore.
>
Please remember that not everyone has 20:20 vision. Requiring a
terminal width that's so large that the text is wrapped (or, worse, you
get horizontal scroll bars) is not acceptable, IMO.
R.
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2020-12-03 12:11 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-26 23:50 [RFC] Increase libstdc++ line length to 100(?) columns Jonathan Wakely
2020-11-27 2:45 ` Liu Hao
2020-11-27 8:14 ` Richard Biener
2020-11-27 8:21 ` Ville Voutilainen
2020-11-27 9:49 ` Liu Hao
2020-11-27 10:13 ` Ville Voutilainen
2020-11-27 11:08 ` Allan Sandfeld Jensen
2020-11-29 17:38 ` Florian Weimer
2020-11-29 18:51 ` Allan Sandfeld Jensen
2020-11-30 15:47 ` Michael Matz
2020-11-30 16:28 ` Allan Sandfeld Jensen
2020-11-30 16:44 ` Michael Matz
2020-12-03 12:11 ` Richard Earnshaw
2020-12-01 3:05 ` Thomas Rodgers
2020-12-01 9:25 ` Matthias Kretz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).