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 ps.keyword easier to understand
details: https://anonhg.NetBSD.org/src/rev/30c14a7fd52d
branches: trunk
changeset: 990417:30c14a7fd52d
user: rillig <rillig%NetBSD.org@localhost>
date: Tue Oct 26 20:43:35 2021 +0000
description:
indent: make ps.keyword easier to understand
Previously, ps.keyword did not have any documentation and was not
straight-forward. In some cases it was reset to kw_0, in others it was
set to an interesting value. The idea behind it was to remember the kind
of word of the previous token, to decide whether to have a space between
sizeof or offsetof and a following '('.
No functional change.
diffstat:
usr.bin/indent/indent.c | 13 ++++++-------
usr.bin/indent/indent.h | 5 +++--
usr.bin/indent/lexi.c | 23 ++++++++++++-----------
3 files changed, 21 insertions(+), 20 deletions(-)
diffs (153 lines):
diff -r e076debf285c -r 30c14a7fd52d usr.bin/indent/indent.c
--- a/usr.bin/indent/indent.c Tue Oct 26 20:37:26 2021 +0000
+++ b/usr.bin/indent/indent.c Tue Oct 26 20:43:35 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: indent.c,v 1.160 2021/10/26 19:36:30 rillig Exp $ */
+/* $NetBSD: indent.c,v 1.161 2021/10/26 20:43:35 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.160 2021/10/26 19:36:30 rillig Exp $");
+__RCSID("$NetBSD: indent.c,v 1.161 2021/10/26 20:43:35 rillig Exp $");
#elif defined(__FreeBSD__)
__FBSDID("$FreeBSD: head/usr.bin/indent/indent.c 340138 2018-11-04 19:24:49Z oshogbo $");
#endif
@@ -333,8 +333,7 @@
inbuf_read_line();
}
- struct parser_state transient_state;
- transient_state = ps;
+ struct parser_state transient_state = ps;
*lsym = lexi(&transient_state); /* read another token */
if (*lsym != lsym_newline && *lsym != lsym_form_feed &&
*lsym != lsym_comment && !transient_state.search_stmt) {
@@ -707,9 +706,9 @@
return true;
if (opt.proc_calls_space)
return true;
- if (ps.keyword == kw_sizeof)
+ if (ps.prev_keyword == kw_sizeof)
return opt.blank_after_sizeof;
- return ps.keyword != kw_0 && ps.keyword != kw_offsetof;
+ return ps.prev_keyword != kw_0 && ps.prev_keyword != kw_offsetof;
}
static void
@@ -753,7 +752,7 @@
}
/* parenthesized type following sizeof or offsetof is not a cast */
- if (ps.keyword == kw_offsetof || ps.keyword == kw_sizeof)
+ if (ps.prev_keyword == kw_offsetof || ps.prev_keyword == kw_sizeof)
ps.not_cast_mask |= 1 << ps.p_l_follow;
}
diff -r e076debf285c -r 30c14a7fd52d usr.bin/indent/indent.h
--- a/usr.bin/indent/indent.h Tue Oct 26 20:37:26 2021 +0000
+++ b/usr.bin/indent/indent.h Tue Oct 26 20:43:35 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: indent.h,v 1.51 2021/10/26 19:36:30 rillig Exp $ */
+/* $NetBSD: indent.h,v 1.52 2021/10/26 20:43:35 rillig Exp $ */
/*-
* SPDX-License-Identifier: BSD-2-Clause-FreeBSD
@@ -332,7 +332,8 @@
bool want_blank; /* whether the following token should be
* prefixed by a blank. (Said prefixing is
* ignored in some cases.) */
- enum keyword_kind keyword;
+ enum keyword_kind prev_keyword;
+ enum keyword_kind curr_keyword;
bool dumped_decl_indent;
bool in_parameter_declaration;
char procname[100]; /* The name of the current procedure */
diff -r e076debf285c -r 30c14a7fd52d usr.bin/indent/lexi.c
--- a/usr.bin/indent/lexi.c Tue Oct 26 20:37:26 2021 +0000
+++ b/usr.bin/indent/lexi.c Tue Oct 26 20:43:35 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: lexi.c,v 1.104 2021/10/26 20:17:42 rillig Exp $ */
+/* $NetBSD: lexi.c,v 1.105 2021/10/26 20:43:35 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.104 2021/10/26 20:17:42 rillig Exp $");
+__RCSID("$NetBSD: lexi.c,v 1.105 2021/10/26 20:43:35 rillig Exp $");
#elif defined(__FreeBSD__)
__FBSDID("$FreeBSD: head/usr.bin/indent/lexi.c 337862 2018-08-15 18:19:45Z pstef $");
#endif
@@ -305,8 +305,10 @@
debug_print_buf("code", &code);
debug_print_buf("comment", &com);
debug_printf("lexi returns '%s'", lsym_name(lsym));
- if (state->keyword != kw_0)
- debug_printf(" keyword '%s'", kw_name(state->keyword));
+ if (state->curr_keyword != kw_0)
+ debug_printf(" keyword '%s'", kw_name(state->curr_keyword));
+ if (state->prev_keyword != kw_0)
+ debug_printf(" previous keyword '%s'", kw_name(state->prev_keyword));
debug_println("");
debug_print_buf("token", &token);
}
@@ -455,7 +457,6 @@
while (is_hspace(inbuf_peek()))
inbuf_skip();
- state->keyword = kw_0;
if (state->last_token == lsym_tag && state->p_l_follow == 0) {
state->next_unary = true;
@@ -469,13 +470,13 @@
array_length(keywords), sizeof(keywords[0]), cmp_keyword_by_name);
if (kw == NULL) {
if (is_typename()) {
- state->keyword = kw_type;
+ state->curr_keyword = kw_type;
state->next_unary = true;
goto found_typename;
}
} else { /* we have a keyword */
- state->keyword = kw->kind;
+ state->curr_keyword = kw->kind;
state->next_unary = true;
switch (kw->kind) {
@@ -493,10 +494,8 @@
state->cast_mask |= (1 << state->p_l_follow) & ~state->not_cast_mask;
}
if (state->last_token == lsym_period ||
- state->last_token == lsym_unary_op) {
- state->keyword = kw_0;
+ state->last_token == lsym_unary_op)
break;
- }
if (kw != NULL && kw->kind == kw_struct_or_union_or_enum)
return lsym_tag;
if (state->p_l_follow != 0)
@@ -544,7 +543,7 @@
not_proc:;
} else if (probably_typename(state)) {
- state->keyword = kw_type;
+ state->curr_keyword = kw_type;
state->next_unary = true;
return lsym_type;
}
@@ -563,6 +562,8 @@
token.e = token.s;
state->col_1 = state->last_nl;
state->last_nl = false;
+ state->prev_keyword = state->curr_keyword;
+ state->curr_keyword = kw_0;
while (is_hspace(*inp.s)) {
state->col_1 = false;
Home |
Main Index |
Thread Index |
Old Index