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: use braces around multi-line statements
details: https://anonhg.NetBSD.org/src/rev/efcf293b2e6e
branches: trunk
changeset: 988622:efcf293b2e6e
user: rillig <rillig%NetBSD.org@localhost>
date: Thu Oct 07 21:57:21 2021 +0000
description:
indent: use braces around multi-line statements
No functional change.
diffstat:
usr.bin/indent/indent.c | 11 ++++-------
usr.bin/indent/io.c | 10 +++++-----
usr.bin/indent/lexi.c | 8 ++++----
usr.bin/indent/pr_comment.c | 7 ++++---
4 files changed, 17 insertions(+), 19 deletions(-)
diffs (150 lines):
diff -r a188cbaf3aad -r efcf293b2e6e usr.bin/indent/indent.c
--- a/usr.bin/indent/indent.c Thu Oct 07 21:52:54 2021 +0000
+++ b/usr.bin/indent/indent.c Thu Oct 07 21:57:21 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: indent.c,v 1.119 2021/10/07 21:52:54 rillig Exp $ */
+/* $NetBSD: indent.c,v 1.120 2021/10/07 21:57:21 rillig Exp $ */
/*-
* SPDX-License-Identifier: BSD-4-Clause
@@ -43,7 +43,7 @@
#include <sys/cdefs.h>
#if defined(__NetBSD__)
-__RCSID("$NetBSD: indent.c,v 1.119 2021/10/07 21:52:54 rillig Exp $");
+__RCSID("$NetBSD: indent.c,v 1.120 2021/10/07 21:57:21 rillig Exp $");
#elif defined(__FreeBSD__)
__FBSDID("$FreeBSD: head/usr.bin/indent/indent.c 340138 2018-11-04 19:24:49Z oshogbo $");
#endif
@@ -248,9 +248,8 @@
return false;
}
- while (sc_end > save_com && isblank((unsigned char)sc_end[-1])) {
+ while (sc_end > save_com && isblank((unsigned char)sc_end[-1]))
sc_end--;
- }
if (opt.swallow_optional_blanklines ||
(!comment_buffered && remove_newlines)) {
@@ -1430,9 +1429,7 @@
}
*code.e = '\0';
- if (ttype != comment &&
- ttype != newline &&
- ttype != preprocessing)
+ if (ttype != comment && ttype != newline && ttype != preprocessing)
ps.last_token = ttype;
}
}
diff -r a188cbaf3aad -r efcf293b2e6e usr.bin/indent/io.c
--- a/usr.bin/indent/io.c Thu Oct 07 21:52:54 2021 +0000
+++ b/usr.bin/indent/io.c Thu Oct 07 21:57:21 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: io.c,v 1.77 2021/10/07 21:52:54 rillig Exp $ */
+/* $NetBSD: io.c,v 1.78 2021/10/07 21:57:21 rillig Exp $ */
/*-
* SPDX-License-Identifier: BSD-4-Clause
@@ -43,7 +43,7 @@
#include <sys/cdefs.h>
#if defined(__NetBSD__)
-__RCSID("$NetBSD: io.c,v 1.77 2021/10/07 21:52:54 rillig Exp $");
+__RCSID("$NetBSD: io.c,v 1.78 2021/10/07 21:57:21 rillig Exp $");
#elif defined(__FreeBSD__)
__FBSDID("$FreeBSD: head/usr.bin/indent/io.c 334927 2018-06-10 16:44:18Z pstef $");
#endif
@@ -296,13 +296,13 @@
int target_ind = opt.indent_size * ps.ind_level;
if (ps.paren_level != 0) {
- if (!opt.lineup_to_parens)
+ if (!opt.lineup_to_parens) {
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)
+ } else if (opt.lineup_to_parens_always) {
/*
* XXX: where does this '- 1' come from? It looks strange but is
* nevertheless needed for proper indentation, as demonstrated in
@@ -310,7 +310,7 @@
*/
target_ind = paren_indent - 1;
- else {
+ } else {
int w;
int t = paren_indent;
diff -r a188cbaf3aad -r efcf293b2e6e usr.bin/indent/lexi.c
--- a/usr.bin/indent/lexi.c Thu Oct 07 21:52:54 2021 +0000
+++ b/usr.bin/indent/lexi.c Thu Oct 07 21:57:21 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: lexi.c,v 1.75 2021/10/07 21:52:54 rillig Exp $ */
+/* $NetBSD: lexi.c,v 1.76 2021/10/07 21:57:21 rillig Exp $ */
/*-
* SPDX-License-Identifier: BSD-4-Clause
@@ -43,7 +43,7 @@
#include <sys/cdefs.h>
#if defined(__NetBSD__)
-__RCSID("$NetBSD: lexi.c,v 1.75 2021/10/07 21:52:54 rillig Exp $");
+__RCSID("$NetBSD: lexi.c,v 1.76 2021/10/07 21:57:21 rillig Exp $");
#elif defined(__FreeBSD__)
__FBSDID("$FreeBSD: head/usr.bin/indent/lexi.c 337862 2018-08-15 18:19:45Z pstef $");
#endif
@@ -596,11 +596,11 @@
unary_delim = false;
}
- } else if (*buf_ptr == '=')
+ } else if (*buf_ptr == '=') {
/* check for operator += */
*token.e++ = *buf_ptr++;
- else if (*buf_ptr == '>') {
+ } else if (*buf_ptr == '>') {
/* check for operator -> */
*token.e++ = *buf_ptr++;
unary_delim = false;
diff -r a188cbaf3aad -r efcf293b2e6e usr.bin/indent/pr_comment.c
--- a/usr.bin/indent/pr_comment.c Thu Oct 07 21:52:54 2021 +0000
+++ b/usr.bin/indent/pr_comment.c Thu Oct 07 21:57:21 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: pr_comment.c,v 1.56 2021/10/07 21:52:54 rillig Exp $ */
+/* $NetBSD: pr_comment.c,v 1.57 2021/10/07 21:57:21 rillig Exp $ */
/*-
* SPDX-License-Identifier: BSD-4-Clause
@@ -43,7 +43,7 @@
#include <sys/cdefs.h>
#if defined(__NetBSD__)
-__RCSID("$NetBSD: pr_comment.c,v 1.56 2021/10/07 21:52:54 rillig Exp $");
+__RCSID("$NetBSD: pr_comment.c,v 1.57 2021/10/07 21:57:21 rillig Exp $");
#elif defined(__FreeBSD__)
__FBSDID("$FreeBSD: head/usr.bin/indent/pr_comment.c 334927 2018-06-10 16:44:18Z pstef $");
#endif
@@ -175,7 +175,7 @@
*com.e++ = ' ';
/* Don't put a break delimiter if this is a one-liner that won't wrap. */
- if (break_delim)
+ if (break_delim) {
for (t_ptr = buf_ptr; *t_ptr != '\0' && *t_ptr != '\n'; t_ptr++) {
if (t_ptr >= buf_end)
fill_buffer();
@@ -192,6 +192,7 @@
break;
}
}
+ }
if (break_delim) {
char *t = com.e;
Home |
Main Index |
Thread Index |
Old Index