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: remove unused code
details: https://anonhg.NetBSD.org/src/rev/9a13eb67a77e
branches: trunk
changeset: 374767:9a13eb67a77e
user: rillig <rillig%NetBSD.org@localhost>
date: Thu May 11 18:26:56 2023 +0000
description:
indent: remove unused code
diffstat:
usr.bin/indent/indent.h | 8 +---
usr.bin/indent/io.c | 87 +-----------------------------------------------
2 files changed, 3 insertions(+), 92 deletions(-)
diffs (153 lines):
diff -r 4ad8de056a3b -r 9a13eb67a77e usr.bin/indent/indent.h
--- a/usr.bin/indent/indent.h Thu May 11 18:13:55 2023 +0000
+++ b/usr.bin/indent/indent.h Thu May 11 18:26:56 2023 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: indent.h,v 1.117 2023/05/11 18:13:55 rillig Exp $ */
+/* $NetBSD: indent.h,v 1.118 2023/05/11 18:26:56 rillig Exp $ */
/*-
* SPDX-License-Identifier: BSD-2-Clause-FreeBSD
@@ -389,16 +389,10 @@ char inp_lookahead(size_t);
void inp_skip(void);
char inp_next(void);
-void inp_comment_init_newline(void);
-void inp_comment_init_comment(void);
void inp_comment_init_preproc(void);
void inp_comment_add_char(char);
void inp_comment_add_range(const char *, const char *);
-bool inp_comment_complete_block(void);
bool inp_comment_seen(void);
-void inp_comment_rtrim_blank(void);
-void inp_comment_rtrim_newline(void);
-void inp_comment_insert_lbrace(void);
void inp_from_comment(void);
diff -r 4ad8de056a3b -r 9a13eb67a77e usr.bin/indent/io.c
--- a/usr.bin/indent/io.c Thu May 11 18:13:55 2023 +0000
+++ b/usr.bin/indent/io.c Thu May 11 18:26:56 2023 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: io.c,v 1.150 2023/05/11 18:13:55 rillig Exp $ */
+/* $NetBSD: io.c,v 1.151 2023/05/11 18:26:56 rillig Exp $ */
/*-
* SPDX-License-Identifier: BSD-4-Clause
@@ -43,7 +43,7 @@ static char sccsid[] = "@(#)io.c 8.1 (Be
#include <sys/cdefs.h>
#if defined(__NetBSD__)
-__RCSID("$NetBSD: io.c,v 1.150 2023/05/11 18:13:55 rillig Exp $");
+__RCSID("$NetBSD: io.c,v 1.151 2023/05/11 18:26:56 rillig Exp $");
#elif defined(__FreeBSD__)
__FBSDID("$FreeBSD: head/usr.bin/indent/io.c 334927 2018-06-10 16:44:18Z pstef $");
#endif
@@ -194,60 +194,6 @@ inp_comment_check_size(size_t n)
}
void
-inp_comment_init_newline(void)
-{
- if (inbuf.save_com_e != NULL)
- return;
-
- inbuf.save_com_s = inbuf.save_com_buf;
- inbuf.save_com_s[0] = ' '; /* see inp_comment_insert_lbrace */
- inbuf.save_com_s[1] = ' '; /* see inp_comment_insert_lbrace */
- inbuf.save_com_e = &inbuf.save_com_s[2];
- debug_inp(__func__);
-}
-
-void
-inp_comment_init_comment(void)
-{
- if (inbuf.save_com_e != NULL)
- return;
-
- /*
- * Copy everything from the start of the line, because process_comment()
- * will use that to calculate the original indentation of a boxed comment.
- */
- /*
- * TODO: Don't store anything in the memory range [input.inp.buf,
- * input.inp.s), as that data can easily get lost.
- */
- /*
- * FIXME: The '4' below is completely wrong. For example, in the snippet
- * 'if(expr)/''*comment', the 'r)' of the code is not copied. If there is
- * an additional line break before the ')', memcpy tries to copy
- * (size_t)-1 bytes.
- *
- * The original author of this magic number doesn't remember its purpose
- * anymore, so there is no point in keeping it. The existing tests must
- * still pass though.
- */
- assert((size_t)(inbuf.inp.s - inbuf.inp.buf) >= 4);
- size_t line_len = (size_t)(inbuf.inp.s - inbuf.inp.buf) - 4;
- assert(line_len < array_length(inbuf.save_com_buf));
-
- memcpy(inbuf.save_com_buf, inbuf.inp.buf, line_len);
- inbuf.save_com_s = inbuf.save_com_buf + line_len;
-
- inbuf.save_com_s[0] = ' '; /* see inp_comment_insert_lbrace */
- inbuf.save_com_s[1] = ' '; /* see inp_comment_insert_lbrace */
- inbuf.save_com_e = &inbuf.save_com_s[2];
-
- debug_vis_range("search_stmt_comment: before save_com is \"",
- inbuf.save_com_buf, inbuf.save_com_s, "\"\n");
- debug_vis_range("search_stmt_comment: save_com is \"",
- inbuf.save_com_s, inbuf.save_com_e, "\"\n");
-}
-
-void
inp_comment_init_preproc(void)
{
if (inbuf.save_com_e == NULL) { /* if this is the first comment, we
@@ -282,33 +228,11 @@ inp_comment_add_range(const char *s, con
}
bool
-inp_comment_complete_block(void)
-{
- return inbuf.save_com_e[-2] == '*' && inbuf.save_com_e[-1] == '/';
-}
-
-bool
inp_comment_seen(void)
{
return inbuf.save_com_e != NULL;
}
-void
-inp_comment_rtrim_blank(void)
-{
- while (inbuf.save_com_e > inbuf.save_com_s &&
- ch_isblank(inbuf.save_com_e[-1]))
- inbuf.save_com_e--;
-}
-
-void
-inp_comment_rtrim_newline(void)
-{
- while (inbuf.save_com_e > inbuf.save_com_s &&
- inbuf.save_com_e[-1] == '\n')
- inbuf.save_com_e--;
-}
-
/*
* Switch the input to come from save_com, replaying the copied tokens while
* looking for the next '{'.
@@ -344,13 +268,6 @@ inp_from_file(void)
return inbuf.inp.s < inbuf.inp.e;
}
-void
-inp_comment_insert_lbrace(void)
-{
- assert(inbuf.save_com_s[0] == ' '); /* see inp_comment_init_newline */
- inbuf.save_com_s[0] = '{';
-}
-
static void
inp_add(char ch)
{
Home |
Main Index |
Thread Index |
Old Index