Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/usr.bin/indent indent: make compute_code_indent more readable



details:   https://anonhg.NetBSD.org/src/rev/68e8fc4006b5
branches:  trunk
changeset: 960286:68e8fc4006b5
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sun Mar 14 01:44:37 2021 +0000

description:
indent: make compute_code_indent more readable

The '?:' operator computing the factor was too hard to read.  When
quickly scanning the code, the 1 in the expression looked too much like
it would be added to the indentation, which would turn the indentation
length into a column number, and that again would smell like an
off-by-one error.

No functional change.

diffstat:

 usr.bin/indent/io.c |  10 ++++++----
 1 files changed, 6 insertions(+), 4 deletions(-)

diffs (31 lines):

diff -r 5622e2cb7f6f -r 68e8fc4006b5 usr.bin/indent/io.c
--- a/usr.bin/indent/io.c       Sun Mar 14 01:34:13 2021 +0000
+++ b/usr.bin/indent/io.c       Sun Mar 14 01:44:37 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: io.c,v 1.48 2021/03/14 00:22:16 rillig Exp $   */
+/*     $NetBSD: io.c,v 1.49 2021/03/14 01:44:37 rillig Exp $   */
 
 /*-
  * SPDX-License-Identifier: BSD-4-Clause
@@ -46,7 +46,7 @@
 #include <sys/cdefs.h>
 #ifndef lint
 #if defined(__NetBSD__)
-__RCSID("$NetBSD: io.c,v 1.48 2021/03/14 00:22:16 rillig Exp $");
+__RCSID("$NetBSD: io.c,v 1.49 2021/03/14 01:44:37 rillig Exp $");
 #elif defined(__FreeBSD__)
 __FBSDID("$FreeBSD: head/usr.bin/indent/io.c 334927 2018-06-10 16:44:18Z pstef $");
 #endif
@@ -291,8 +291,10 @@
 
     if (ps.paren_level != 0) {
        if (!opt.lineup_to_parens)
-           target_ind += opt.continuation_indent *
-               (2 * opt.continuation_indent == opt.indent_size ? 1 : ps.paren_level);
+           if (2 * opt.continuation_indent == opt.indent_size)
+               target_ind += opt.continuation_indent;
+           else
+               target_ind += opt.continuation_indent * ps.paren_level;
        else if (opt.lineup_to_parens_always)
            /*
             * XXX: where does this '- 1' come from?  It looks strange but is



Home | Main Index | Thread Index | Old Index