 @@ -1,359 +1,360 @@
-/*	$NetBSD: args.c,v 1.21 2021/03/13 13:51:08 rillig Exp $	*/
+/*	$NetBSD: args.c,v 1.22 2021/03/14 00:22:16 rillig Exp $	*/
 /*-
  * SPDX-License-Identifier: BSD-4-Clause
+ *
  * Copyright (c) 1985 Sun Microsystems, Inc.
  * Copyright (c) 1980, 1993
  *	The Regents of the University of California.  All rights reserved.
  * All rights reserved.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. All advertising materials mentioning features or use of this software
  *    must display the following acknowledgement:
  *	This product includes software developed by the University of
  *	California, Berkeley and its contributors.
  * 4. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
 #if 0
 #ifndef lint
 static char sccsid[] = "@(#)args.c	8.1 (Berkeley) 6/6/93";
 #endif /* not lint */
 #endif
 #include <sys/cdefs.h>
 #ifndef lint
 #if defined(__NetBSD__)
-__RCSID("$NetBSD: args.c,v 1.21 2021/03/13 13:51:08 rillig Exp $");
+__RCSID("$NetBSD: args.c,v 1.22 2021/03/14 00:22:16 rillig Exp $");
 #elif defined(__FreeBSD__)
 __FBSDID("$FreeBSD: head/usr.bin/indent/args.c 336318 2018-07-15 21:04:21Z pstef $");
 #endif
 #endif
 /*
  * Argument scanning and profile reading code.  Default parameters are set
  * here as well.
  */
 #include <ctype.h>
 #include <err.h>
 #include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include "indent.h"
 #define INDENT_VERSION	"2.0"
 /* profile types */
 #define	PRO_SPECIAL	1	/* special case */
 #define	PRO_BOOL	2	/* boolean */
 #define	PRO_INT		3	/* integer */
 /* profile specials for booleans */
 #define	ON		1	/* turn it on */
 #define	OFF		0	/* turn it off */
 /* profile specials for specials */
 #define	IGN		1	/* ignore it */
 #define	CLI		2	/* case label indent (float) */
 #define	STDIN		3	/* use stdin */
 #define	KEY		4	/* type (keyword) */
 static void scan_profile(FILE *);
 #define	KEY_FILE	5	/* only used for args */
 #define VERSION		6	/* only used for args */
 const char *option_source = "?";
 void add_typedefs_from_file(const char *str);
 /*
  * N.B.: because of the way the table here is scanned, options whose names are
  * substrings of other options must occur later; that is, with -lp vs -l, -lp
  * must be first.  Also, while (most) booleans occur more than once, the last
  * default value is the one actually assigned.
  */
 const struct pro {
     const char *p_name;		/* name, e.g. -bl, -cli */
     int         p_type;		/* type (int, bool, special) */
     int         p_default;	/* the default value (if int) */
     int         p_special;	/* depends on type */
     int        *p_obj;		/* the associated variable */
 }           pro[] = {
     {"T", PRO_SPECIAL, 0, KEY, 0},
     {"U", PRO_SPECIAL, 0, KEY_FILE, 0},
     {"-version", PRO_SPECIAL, 0, VERSION, 0},
     {"P", PRO_SPECIAL, 0, IGN, 0},
     {"bacc", PRO_BOOL, false, ON, &opt.blanklines_around_conditional_compilation},
     {"badp", PRO_BOOL, false, ON, &opt.blanklines_after_declarations_at_proctop},
     {"bad", PRO_BOOL, false, ON, &opt.blanklines_after_declarations},
     {"bap", PRO_BOOL, false, ON, &opt.blanklines_after_procs},
     {"bbb", PRO_BOOL, false, ON, &opt.blanklines_before_blockcomments},
     {"bc", PRO_BOOL, true, OFF, &opt.leave_comma},
     {"bl", PRO_BOOL, true, OFF, &opt.btype_2},
     {"br", PRO_BOOL, true, ON, &opt.btype_2},
     {"bs", PRO_BOOL, false, ON, &opt.Bill_Shannon},
     {"cdb", PRO_BOOL, true, ON, &opt.comment_delimiter_on_blankline},
     {"cd", PRO_INT, 0, 0, &opt.decl_comment_column},
     {"ce", PRO_BOOL, true, ON, &opt.cuddle_else},
     {"ci", PRO_INT, 0, 0, &opt.continuation_indent},
     {"cli", PRO_SPECIAL, 0, CLI, 0},
     {"cs", PRO_BOOL, false, ON, &opt.space_after_cast},
     {"c", PRO_INT, 33, 0, &opt.comment_column},
     {"di", PRO_INT, 16, 0, &opt.decl_indent},
     {"dj", PRO_BOOL, false, ON, &opt.ljust_decl},
     {"d", PRO_INT, 0, 0, &opt.unindent_displace},
     {"eei", PRO_BOOL, false, ON, &opt.extra_expression_indent},
     {"ei", PRO_BOOL, true, ON, &opt.else_if},
     {"fbs", PRO_BOOL, true, ON, &opt.function_brace_split},
     {"fc1", PRO_BOOL, true, ON, &opt.format_col1_comments},
     {"fcb", PRO_BOOL, true, ON, &opt.format_block_comments},
     {"ip", PRO_BOOL, true, ON, &opt.indent_parameters},
     {"i", PRO_INT, 8, 0, &opt.indent_size},
     {"lc", PRO_INT, 0, 0, &opt.block_comment_max_line_length},
     {"ldi", PRO_INT, -1, 0, &opt.local_decl_indent},
     {"lpl", PRO_BOOL, false, ON, &opt.lineup_to_parens_always},
     {"lp", PRO_BOOL, true, ON, &opt.lineup_to_parens},
     {"l", PRO_INT, 78, 0, &opt.max_line_length},
     {"nbacc", PRO_BOOL, false, OFF, &opt.blanklines_around_conditional_compilation},
     {"nbadp", PRO_BOOL, false, OFF, &opt.blanklines_after_declarations_at_proctop},
     {"nbad", PRO_BOOL, false, OFF, &opt.blanklines_after_declarations},
     {"nbap", PRO_BOOL, false, OFF, &opt.blanklines_after_procs},
     {"nbbb", PRO_BOOL, false, OFF, &opt.blanklines_before_blockcomments},
     {"nbc", PRO_BOOL, true, ON, &opt.leave_comma},
     {"nbs", PRO_BOOL, false, OFF, &opt.Bill_Shannon},
     {"ncdb", PRO_BOOL, true, OFF, &opt.comment_delimiter_on_blankline},
     {"nce", PRO_BOOL, true, OFF, &opt.cuddle_else},
     {"ncs", PRO_BOOL, false, OFF, &opt.space_after_cast},
     {"ndj", PRO_BOOL, false, OFF, &opt.ljust_decl},
     {"neei", PRO_BOOL, false, OFF, &opt.extra_expression_indent},
     {"nei", PRO_BOOL, true, OFF, &opt.else_if},
     {"nfbs", PRO_BOOL, true, OFF, &opt.function_brace_split},
     {"nfc1", PRO_BOOL, true, OFF, &opt.format_col1_comments},
     {"nfcb", PRO_BOOL, true, OFF, &opt.format_block_comments},
     {"nip", PRO_BOOL, true, OFF, &opt.indent_parameters},
     {"nlpl", PRO_BOOL, false, OFF, &opt.lineup_to_parens_always},
     {"nlp", PRO_BOOL, true, OFF, &opt.lineup_to_parens},
     {"npcs", PRO_BOOL, false, OFF, &opt.proc_calls_space},
     {"npro", PRO_SPECIAL, 0, IGN, 0},
     {"npsl", PRO_BOOL, true, OFF, &opt.procnames_start_line},
     {"nsc", PRO_BOOL, true, OFF, &opt.star_comment_cont},
     {"nsob", PRO_BOOL, false, OFF, &opt.swallow_optional_blanklines},
     {"nut", PRO_BOOL, true, OFF, &opt.use_tabs},
     {"nv", PRO_BOOL, false, OFF, &opt.verbose},
     {"pcs", PRO_BOOL, false, ON, &opt.proc_calls_space},
     {"psl", PRO_BOOL, true, ON, &opt.procnames_start_line},
     {"sc", PRO_BOOL, true, ON, &opt.star_comment_cont},
     {"sob", PRO_BOOL, false, ON, &opt.swallow_optional_blanklines},
     {"st", PRO_SPECIAL, 0, STDIN, 0},
     {"ta", PRO_BOOL, false, ON, &opt.auto_typedefs},
     {"ts", PRO_INT, 8, 0, &opt.tabsize},
     {"ut", PRO_BOOL, true, ON, &opt.use_tabs},
     {"v", PRO_BOOL, false, ON, &opt.verbose},
     /* whew! */
     {0, 0, 0, 0, 0}
 };
 /*
  * set_profile reads $HOME/.indent.pro and ./.indent.pro and handles arguments
  * given in these files.
  */
 void
 set_profile(const char *profile_name)
+{
     FILE *f;
     char fname[PATH_MAX];
     static char prof[] = ".indent.pro";
     if (profile_name == NULL)
 	snprintf(fname, sizeof(fname), "%s/%s", getenv("HOME"), prof);
     else
 	snprintf(fname, sizeof(fname), "%s", profile_name + 2);
     if ((f = fopen(option_source = fname, "r")) != NULL) {
 	scan_profile(f);
 	(void) fclose(f);
+    }
     if ((f = fopen(option_source = prof, "r")) != NULL) {
 	scan_profile(f);
 	(void) fclose(f);
+    }
     option_source = "Command line";
+}
 static void
 scan_profile(FILE *f)
+{
     int		comment_index, i;
     char	*p;
     char        buf[BUFSIZ];
-    while (1) {
+    for (;;) {
 	p = buf;
 	comment_index = 0;
 	while ((i = getc(f)) != EOF) {
 	    if (i == '*' && !comment_index && p > buf && p[-1] == '/') {
-		comment_index = p - buf;
+		comment_index = (int)(p - buf);
 		*p++ = i;
 	    } else if (i == '/' && comment_index && p > buf && p[-1] == '*') {
 		p = buf + comment_index - 1;
 		comment_index = 0;
 	    } else if (isspace((unsigned char)i)) {
 		if (p > buf && !comment_index)
 		    break;
 	    } else {
 		*p++ = i;
+	    }
+	}
 	if (p != buf) {
 	    *p++ = 0;
 	    if (opt.verbose)
 		printf("profile: %s\n", buf);
 	    set_option(buf);
 	} else if (i == EOF)
 	    return;
+    }
+}
 static const char *
 eqin(const char *s1, const char *s2)
+{
     while (*s1) {
 	if (*s1++ != *s2++)
 	    return NULL;
+    }
     return s2;
+}
 /*
  * Set the defaults.
  */
 void
 set_defaults(void)
+{
     /*
      * Because ps.case_indent is a float, we can't initialize it from the
      * table:
      */
     opt.case_indent = 0.0F;	/* -cli0.0 */
     for (const struct pro *p = pro; p->p_name; p++)
 	if (p->p_type != PRO_SPECIAL)
 	    *p->p_obj = p->p_default;
+}
 void
 set_option(char *arg)
+{
     const struct pro *p;
     const char	*param_start;
     arg++;			/* ignore leading "-" */
     for (p = pro; p->p_name; p++)
 	if (*p->p_name == *arg && (param_start = eqin(p->p_name, arg)) != NULL)
 	    goto found;
     errx(1, "%s: unknown parameter \"%s\"", option_source, arg - 1);
 found:
     switch (p->p_type) {
     case PRO_SPECIAL:
 	switch (p->p_special) {
 	case IGN:
 	    break;
 	case CLI:
 	    if (*param_start == 0)
 		goto need_param;
 	    opt.case_indent = atof(param_start);
 	    break;
 	case STDIN:
 	    if (input == NULL)
 		input = stdin;
 	    if (output == NULL)
 		output = stdout;
 	    break;
 	case KEY:
 	    if (*param_start == 0)
 		goto need_param;
 	    add_typename(param_start);
 	    break;
 	case KEY_FILE:
 	    if (*param_start == 0)
 		goto need_param;
 	    add_typedefs_from_file(param_start);
 	    break;
 	case VERSION:
 	    printf("FreeBSD indent %s\n", INDENT_VERSION);
 	    exit(0);
 	    /*NOTREACHED*/
 	default:
 	    errx(1, "set_option: internal error: p_special %d", p->p_special);
+	}
 	break;
     case PRO_BOOL:
 	if (p->p_special == OFF)
 	    *p->p_obj = false;
 	else
 	    *p->p_obj = true;
 	break;
     case PRO_INT:
 	if (!isdigit((unsigned char)*param_start)) {
     need_param:
 	    errx(1, "%s: ``%s'' requires a parameter", option_source, p->p_name);
+	}
 	*p->p_obj = atoi(param_start);
 	break;
     default:
 	errx(1, "set_option: internal error: p_type %d", p->p_type);
+    }
+}
 void
 add_typedefs_from_file(const char *str)
+{
     FILE *file;
     char line[BUFSIZ];
     if ((file = fopen(str, "r")) == NULL) {
 	fprintf(stderr, "indent: cannot open file %s\n", str);
 	exit(1);
+    }
     while ((fgets(line, BUFSIZ, file)) != NULL) {
 	/* Remove trailing whitespace */
 	line[strcspn(line, " \t\n\r")] = '\0';
 	add_typename(line);
+    }
     fclose(file);
+}

 @@ -1,1598 +1,1599 @@
-/*	$NetBSD: indent.c,v 1.58 2021/03/13 18:46:39 rillig Exp $	*/
+/*	$NetBSD: indent.c,v 1.59 2021/03/14 00:22:16 rillig Exp $	*/
 /*-
  * SPDX-License-Identifier: BSD-4-Clause
+ *
  * Copyright (c) 1985 Sun Microsystems, Inc.
  * Copyright (c) 1976 Board of Trustees of the University of Illinois.
  * Copyright (c) 1980, 1993
  *	The Regents of the University of California.  All rights reserved.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. All advertising materials mentioning features or use of this software
  *    must display the following acknowledgement:
  *	This product includes software developed by the University of
  *	California, Berkeley and its contributors.
  * 4. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
 #if 0
 #ifndef lint
 static char sccsid[] = "@(#)indent.c	5.17 (Berkeley) 6/7/93";
 #endif /* not lint */
 #endif
 #include <sys/cdefs.h>
 #ifndef lint
 #if defined(__NetBSD__)
-__RCSID("$NetBSD: indent.c,v 1.58 2021/03/13 18:46:39 rillig Exp $");
+__RCSID("$NetBSD: indent.c,v 1.59 2021/03/14 00:22:16 rillig Exp $");
 #elif defined(__FreeBSD__)
 __FBSDID("$FreeBSD: head/usr.bin/indent/indent.c 340138 2018-11-04 19:24:49Z oshogbo $");
 #endif
 #endif
 #include <sys/param.h>
 #if HAVE_CAPSICUM
 #include <sys/capsicum.h>
 #include <capsicum_helpers.h>
 #endif
 #include <err.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <unistd.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <ctype.h>
 #include "indent.h"
 struct options opt;
 struct parser_state ps;
 char       *labbuf;
 char       *s_lab;
 char       *e_lab;
 char       *l_lab;
 char       *codebuf;
 char       *s_code;
 char       *e_code;
 char       *l_code;
 char       *combuf;
 char       *s_com;
 char       *e_com;
 char       *l_com;
 char       *tokenbuf;
 char	   *s_token;
 char       *e_token;
 char	   *l_token;
 char       *in_buffer;
 char	   *in_buffer_limit;
 char       *buf_ptr;
 char       *buf_end;
 char        sc_buf[sc_size];
 char       *save_com;
 char       *sc_end;
 char       *bp_save;
 char       *be_save;
 int         found_err;
 int         n_real_blanklines;
 int         prefix_blankline_requested;
 int         postfix_blankline_requested;
 int         break_comma;
 float       case_ind;
 int         code_lines;
 int         had_eof;
 int         line_no;
 int         inhibit_formatting;
 int         suppress_blanklines;
 int         ifdef_level;
 struct parser_state state_stack[5];
 struct parser_state match_state[5];
 FILE       *input;
 FILE       *output;
 static void bakcopy(void);
 static void indent_declaration(int, int);
 const char *in_name = "Standard Input";	/* will always point to name of input
 					 * file */
 const char *out_name = "Standard Output";	/* will always point to name
 						 * of output file */
 const char *simple_backup_suffix = ".BAK";	/* Suffix to use for backup
 						 * files */
 char        bakfile[MAXPATHLEN] = "";
 static void
 check_size_code(size_t desired_size)
+{
     if (e_code + (desired_size) < l_code)
         return;
     size_t nsize = l_code - s_code + 400 + desired_size;
     size_t code_len = e_code - s_code;
     codebuf = realloc(codebuf, nsize);
     if (codebuf == NULL)
 	err(1, NULL);
     e_code = codebuf + code_len + 1;
     l_code = codebuf + nsize - 5;
     s_code = codebuf + 1;
+}
 static void
 check_size_label(size_t desired_size)
+{
     if (e_lab + (desired_size) < l_lab)
         return;
     size_t nsize = l_lab - s_lab + 400 + desired_size;
     size_t label_len = e_lab - s_lab;
     labbuf = realloc(labbuf, nsize);
     if (labbuf == NULL)
 	err(1, NULL);
     e_lab = labbuf + label_len + 1;
     l_lab = labbuf + nsize - 5;
     s_lab = labbuf + 1;
+}
 #if HAVE_CAPSICUM
 static void
 init_capsicum(void)
+{
     cap_rights_t rights;
     /* Restrict input/output descriptors and enter Capsicum sandbox. */
     cap_rights_init(&rights, CAP_FSTAT, CAP_WRITE);
     if (caph_rights_limit(fileno(output), &rights) < 0)
 	err(EXIT_FAILURE, "unable to limit rights for %s", out_name);
     cap_rights_init(&rights, CAP_FSTAT, CAP_READ);
     if (caph_rights_limit(fileno(input), &rights) < 0)
 	err(EXIT_FAILURE, "unable to limit rights for %s", in_name);
     if (caph_enter() < 0)
 	err(EXIT_FAILURE, "unable to enter capability mode");
+}
 #endif
 static void
 search_brace(token_type *inout_type_code, int *inout_force_nl,
 	     int *inout_comment_buffered, int *inout_last_else)
+{
     while (ps.search_brace) {
 	switch (*inout_type_code) {
 	case newline:
 	    if (sc_end == NULL) {
 		save_com = sc_buf;
 		save_com[0] = save_com[1] = ' ';
 		sc_end = &save_com[2];
+	    }
 	    *sc_end++ = '\n';
 	    /*
 	     * We may have inherited a force_nl == true from the previous
 	     * token (like a semicolon). But once we know that a newline
 	     * has been scanned in this loop, force_nl should be false.
+	     *
 	     * However, the force_nl == true must be preserved if newline
 	     * is never scanned in this loop, so this assignment cannot be
 	     * done earlier.
 	     */
 	    *inout_force_nl = false;
 	    break;
 	case form_feed:
 	    break;
 	case comment:
 	    if (sc_end == NULL) {
 		/*
 		 * Copy everything from the start of the line, because
 		 * process_comment() will use that to calculate original
 		 * indentation of a boxed comment.
 		 */
-		memcpy(sc_buf, in_buffer, buf_ptr - in_buffer - 4);
+		memcpy(sc_buf, in_buffer, (size_t)(buf_ptr - in_buffer) - 4);
 		save_com = sc_buf + (buf_ptr - in_buffer - 4);
 		save_com[0] = save_com[1] = ' ';
 		sc_end = &save_com[2];
+	    }
 	    *inout_comment_buffered = true;
 	    *sc_end++ = '/';	/* copy in start of comment */
 	    *sc_end++ = '*';
 	    for (;;) {		/* loop until the end of the comment */
 		*sc_end = *buf_ptr++;
 		if (buf_ptr >= buf_end)
 		    fill_buffer();
 		if (*sc_end++ == '*' && *buf_ptr == '/')
 		    break;	/* we are at end of comment */
 		if (sc_end >= &save_com[sc_size]) {	/* check for temp buffer
 							 * overflow */
 		    diag(1, "Internal buffer overflow - Move big comment from right after if, while, or whatever");
 		    fflush(output);
 		    exit(1);
+		}
+	    }
 	    *sc_end++ = '/';	/* add ending slash */
 	    if (++buf_ptr >= buf_end)	/* get past / in buffer */
 		fill_buffer();
 	    break;
 	case lbrace:
 	    /*
 	     * Put KNF-style lbraces before the buffered up tokens and
 	     * jump out of this loop in order to avoid copying the token
 	     * again under the default case of the switch below.
 	     */
 	    if (sc_end != NULL && opt.btype_2) {
 		save_com[0] = '{';
 		/*
 		 * Originally the lbrace may have been alone on its own
 		 * line, but it will be moved into "the else's line", so
 		 * if there was a newline resulting from the "{" before,
 		 * it must be scanned now and ignored.
 		 */
 		while (isspace((unsigned char)*buf_ptr)) {
 		    if (++buf_ptr >= buf_end)
 			fill_buffer();
 		    if (*buf_ptr == '\n')
 			break;
+		}
 		goto sw_buffer;
+	    }
 	    /* FALLTHROUGH */
 	default:		/* it is the start of a normal statement */
+	{
 	    int remove_newlines;
 	    remove_newlines =
 		    /* "} else" */
 		    (*inout_type_code == keyword_do_else && *token == 'e' &&
 		     e_code != s_code && e_code[-1] == '}')
 		    /* "else if" */
 		    || (*inout_type_code == keyword_for_if_while &&
 			*token == 'i' && *inout_last_else && opt.else_if);
 	    if (remove_newlines)
 		*inout_force_nl = false;
 	    if (sc_end == NULL) {	/* ignore buffering if
 					 * comment wasn't saved up */
 		ps.search_brace = false;
 		return;
+	    }
 	    while (sc_end > save_com && isblank((unsigned char)sc_end[-1])) {
 		sc_end--;
+	    }
 	    if (opt.swallow_optional_blanklines ||
 		(!*inout_comment_buffered && remove_newlines)) {
 		*inout_force_nl = !remove_newlines;
 		while (sc_end > save_com && sc_end[-1] == '\n') {
 		    sc_end--;
+		}
+	    }
 	    if (*inout_force_nl) {	/* if we should insert a nl here, put
 					 * it into the buffer */
 		*inout_force_nl = false;
 		--line_no;	/* this will be re-increased when the
 				 * newline is read from the buffer */
 		*sc_end++ = '\n';
 		*sc_end++ = ' ';
 		if (opt.verbose) /* print error msg if the line was
 				 * not already broken */
 		    diag(0, "Line broken");
+	    }
 	    for (const char *t_ptr = token; *t_ptr; ++t_ptr)
 		*sc_end++ = *t_ptr;
 	    sw_buffer:
 	    ps.search_brace = false;	/* stop looking for start of stmt */
 	    bp_save = buf_ptr;	/* save current input buffer */
 	    be_save = buf_end;
 	    buf_ptr = save_com;	/* fix so that subsequent calls to
 				 * lexi will take tokens out of save_com */
 	    *sc_end++ = ' ';	/* add trailing blank, just in case */
 	    buf_end = sc_end;
 	    sc_end = NULL;
 	    debug_println("switched buf_ptr to save_com");
 	    break;
+	}
 	}			/* end of switch */
 	/*
 	 * We must make this check, just in case there was an unexpected
 	 * EOF.
 	 */
 	if (*inout_type_code != end_of_file) {
 	    /*
 	     * The only intended purpose of calling lexi() below is to
 	     * categorize the next token in order to decide whether to
 	     * continue buffering forthcoming tokens. Once the buffering
 	     * is over, lexi() will be called again elsewhere on all of
 	     * the tokens - this time for normal processing.
+	     *
 	     * Calling it for this purpose is a bug, because lexi() also
 	     * changes the parser state and discards leading whitespace,
 	     * which is needed mostly for comment-related considerations.
+	     *
 	     * Work around the former problem by giving lexi() a copy of
 	     * the current parser state and discard it if the call turned
 	     * out to be just a look ahead.
+	     *
 	     * Work around the latter problem by copying all whitespace
 	     * characters into the buffer so that the later lexi() call
 	     * will read them.
 	     */
 	    if (sc_end != NULL) {
 		while (*buf_ptr == ' ' || *buf_ptr == '\t') {
 		    *sc_end++ = *buf_ptr++;
 		    if (sc_end >= &save_com[sc_size]) {
 			errx(1, "input too long");
+		    }
+		}
 		if (buf_ptr >= buf_end) {
 		    fill_buffer();
+		}
+	    }
 	    struct parser_state transient_state;
 	    transient_state = ps;
 	    *inout_type_code = lexi(&transient_state);	/* read another token */
 	    if (*inout_type_code != newline && *inout_type_code != form_feed &&
 		*inout_type_code != comment && !transient_state.search_brace) {
 		ps = transient_state;
+	    }
+	}
+    }
     *inout_last_else = 0;
+}
 static void
 main_init_globals(void)
+{
     found_err = 0;
     ps.p_stack[0] = stmt;	/* this is the parser's stack */
     ps.last_nl = true;		/* this is true if the last thing scanned was
 				 * a newline */
     ps.last_token = semicolon;
     combuf = malloc(bufsize);
     if (combuf == NULL)
 	err(1, NULL);
     labbuf = malloc(bufsize);
     if (labbuf == NULL)
 	err(1, NULL);
     codebuf = malloc(bufsize);
     if (codebuf == NULL)
 	err(1, NULL);
     tokenbuf = malloc(bufsize);
     if (tokenbuf == NULL)
 	err(1, NULL);
     alloc_typenames();
     init_constant_tt();
     l_com = combuf + bufsize - 5;
     l_lab = labbuf + bufsize - 5;
     l_code = codebuf + bufsize - 5;
     l_token = tokenbuf + bufsize - 5;
     combuf[0] = codebuf[0] = labbuf[0] = ' ';	/* set up code, label, and
 						 * comment buffers */
     combuf[1] = codebuf[1] = labbuf[1] = tokenbuf[1] = '\0';
     opt.else_if = 1;		/* Default else-if special processing to on */
     s_lab = e_lab = labbuf + 1;
     s_code = e_code = codebuf + 1;
     s_com = e_com = combuf + 1;
     s_token = e_token = tokenbuf + 1;
     in_buffer = malloc(10);
     if (in_buffer == NULL)
 	err(1, NULL);
     in_buffer_limit = in_buffer + 8;
     buf_ptr = buf_end = in_buffer;
     line_no = 1;
     had_eof = ps.in_decl = ps.decl_on_line = break_comma = false;
     ps.in_or_st = false;
     ps.bl_line = true;
     ps.want_blank = ps.in_stmt = ps.ind_stmt = false;
     ps.pcase = false;
     sc_end = NULL;
     bp_save = NULL;
     be_save = NULL;
     output = NULL;
     const char *suffix = getenv("SIMPLE_BACKUP_SUFFIX");
     if (suffix != NULL)
 	simple_backup_suffix = suffix;
+}
 static void
 main_parse_command_line(int argc, char **argv)
+{
     int i;
     const char *profile_name = NULL;
 #if 0
     max_line_length = 78;	/* -l78 */
     lineup_to_parens = 1;	/* -lp */
     lineup_to_parens_always = 0; /* -nlpl */
     ps.ljust_decl = 0;		/* -ndj */
     ps.com_ind = 33;		/* -c33 */
     star_comment_cont = 1;	/* -sc */
     ps.ind_size = 8;		/* -i8 */
     verbose = 0;
     ps.decl_indent = 16;	/* -di16 */
     ps.local_decl_indent = -1;	/* if this is not set to some nonnegative value
 				 * by an arg, we will set this equal to
 				 * ps.decl_ind */
     ps.indent_parameters = 1;	/* -ip */
     ps.decl_com_ind = 0;	/* if this is not set to some positive value
 				 * by an arg, we will set this equal to
 				 * ps.com_ind */
     btype_2 = 1;		/* -br */
     cuddle_else = 1;		/* -ce */
     ps.unindent_displace = 0;	/* -d0 */
     ps.case_indent = 0;		/* -cli0 */
     format_block_comments = 1;	/* -fcb */
     format_col1_comments = 1;	/* -fc1 */
     procnames_start_line = 1;	/* -psl */
     proc_calls_space = 0;	/* -npcs */
     comment_delimiter_on_blankline = 1;	/* -cdb */
     ps.leave_comma = 1;		/* -nbc */
 #endif
     for (i = 1; i < argc; ++i)
 	if (strcmp(argv[i], "-npro") == 0)
 	    break;
 	else if (argv[i][0] == '-' && argv[i][1] == 'P' && argv[i][2] != '\0')
 	    profile_name = argv[i];	/* non-empty -P (set profile) */
     set_defaults();
     if (i >= argc)
 	set_profile(profile_name);
     for (i = 1; i < argc; ++i) {
 	/*
 	 * look thru args (if any) for changes to defaults
 	 */
 	if (argv[i][0] != '-') {/* no flag on parameter */
 	    if (input == NULL) {	/* we must have the input file */
 		in_name = argv[i];	/* remember name of input file */
 		input = fopen(in_name, "r");
 		if (input == NULL)	/* check for open error */
 			err(1, "%s", in_name);
 		continue;
 	    } else if (output == NULL) {	/* we have the output file */
 		out_name = argv[i];	/* remember name of output file */
 		if (strcmp(in_name, out_name) == 0) {	/* attempt to overwrite
 							 * the file */
 		    errx(1, "input and output files must be different");
+		}
 		output = fopen(out_name, "w");
 		if (output == NULL)	/* check for create error */
 			err(1, "%s", out_name);
 		continue;
+	    }
 	    errx(1, "unknown parameter: %s", argv[i]);
 	} else
 	    set_option(argv[i]);
     }				/* end of for */
     if (input == NULL)
 	input = stdin;
     if (output == NULL) {
 	if (input == stdin)
 	    output = stdout;
 	else {
 	    out_name = in_name;
 	    bakcopy();
+	}
+    }
     if (opt.comment_column <= 1)
 	opt.comment_column = 2;	/* don't put normal comments before column 2 */
     if (opt.block_comment_max_line_length <= 0)
 	opt.block_comment_max_line_length = opt.max_line_length;
     if (opt.local_decl_indent < 0) /* if not specified by user, set this */
 	opt.local_decl_indent = opt.decl_indent;
     if (opt.decl_comment_column <= 0)	/* if not specified by user, set this */
 	opt.decl_comment_column = opt.ljust_decl
 	    ? (opt.comment_column <= 10 ? 2 : opt.comment_column - 8)
 	    : opt.comment_column;
     if (opt.continuation_indent == 0)
 	opt.continuation_indent = opt.indent_size;
+}
 static void
 main_prepare_parsing(void)
+{
     fill_buffer();		/* get first batch of stuff into input buffer */
     parse(semicolon);
     char *p = buf_ptr;
     int col = 1;
-    while (1) {
+    for (;;) {
 	if (*p == ' ')
 	    col++;
 	else if (*p == '\t')
 	    col = opt.tabsize * (1 + (col - 1) / opt.tabsize) + 1;
 	else
 	    break;
 	p++;
+    }
     if (col > opt.indent_size)
 	ps.ind_level = ps.i_l_follow = col / opt.indent_size;
+}
 static void
 process_end_of_file(void)
+{
     if (s_lab != e_lab || s_code != e_code || s_com != e_com)
 	dump_line();
     if (ps.tos > 1)		/* check for balanced braces */
 	diag(1, "Stuff missing from end of file");
     if (opt.verbose) {
 	printf("There were %d output lines and %d comments\n",
 	       ps.out_lines, ps.out_coms);
 	printf("(Lines with comments)/(Lines with code): %6.3f\n",
 	       (1.0 * ps.com_lines) / code_lines);
+    }
     fflush(output);
     exit(found_err);
+}
 static void
 process_comment_in_code(token_type type_code, int *inout_force_nl)
+{
     if (*inout_force_nl &&
 	type_code != semicolon &&
 	(type_code != lbrace || !opt.btype_2)) {
 	/* we should force a broken line here */
 	if (opt.verbose)
 	    diag(0, "Line broken");
 	dump_line();
 	ps.want_blank = false;	/* dont insert blank at line start */
 	*inout_force_nl = false;
+    }
     ps.in_stmt = true;		/* turn on flag which causes an extra level of
 				 * indentation. this is turned off by a ; or
 				 * '}' */
     if (s_com != e_com) {	/* the turkey has embedded a comment
 				 * in a line. fix it */
-	int len = e_com - s_com;
+	size_t len = e_com - s_com;
 	check_size_code(len + 3);
 	*e_code++ = ' ';
 	memcpy(e_code, s_com, len);
 	e_code += len;
 	*e_code++ = ' ';
 	*e_code = '\0';		/* null terminate code sect */
 	ps.want_blank = false;
 	e_com = s_com;
+    }
+}
 static void
 process_form_feed(void)
+{
     ps.use_ff = true;		/* a form feed is treated much like a newline */
     dump_line();
     ps.want_blank = false;
+}
 static void
 process_newline(void)
+{
     if (ps.last_token != comma || ps.p_l_follow > 0
 	|| !opt.leave_comma || ps.block_init || !break_comma || s_com != e_com) {
 	dump_line();
 	ps.want_blank = false;
+    }
     ++line_no;			/* keep track of input line number */
+}
 static void
 process_lparen_or_lbracket(int dec_ind, int tabs_to_var, int sp_sw)
+{
     /* count parens to make Healy happy */
     if (++ps.p_l_follow == nitems(ps.paren_indents)) {
 	diag(0, "Reached internal limit of %zu unclosed parens",
 	    nitems(ps.paren_indents));
 	ps.p_l_follow--;
+    }
     if (*token == '[')
 	/* not a function pointer declaration or a function call */;
     else if (ps.in_decl && !ps.block_init && !ps.dumped_decl_indent &&
 	ps.procname[0] == '\0' && ps.paren_level == 0) {
 	/* function pointer declarations */
 	indent_declaration(dec_ind, tabs_to_var);
 	ps.dumped_decl_indent = true;
     } else if (ps.want_blank &&
 	    ((ps.last_token != ident && ps.last_token != funcname) ||
 	    opt.proc_calls_space ||
 	    (ps.keyword == rw_sizeof ? opt.Bill_Shannon :
 	    ps.keyword != rw_0 && ps.keyword != rw_offsetof)))
 	*e_code++ = ' ';
     ps.want_blank = false;
     *e_code++ = token[0];
     ps.paren_indents[ps.p_l_follow - 1] =
 	indentation_after_range(0, s_code, e_code);
     debug_println("paren_indent[%d] is now %d",
 	ps.p_l_follow - 1, ps.paren_indents[ps.p_l_follow - 1]);
     if (sp_sw && ps.p_l_follow == 1 && opt.extra_expression_indent
 	    && ps.paren_indents[0] < 2 * opt.indent_size) {
 	ps.paren_indents[0] = 2 * opt.indent_size;
 	debug_println("paren_indent[0] is now %d", ps.paren_indents[0]);
+    }
     if (ps.in_or_st && *token == '(' && ps.tos <= 2) {
 	/*
 	 * this is a kluge to make sure that declarations will be
 	 * aligned right if proc decl has an explicit type on it, i.e.
 	 * "int a(x) {..."
 	 */
 	parse(semicolon);	/* I said this was a kluge... */
 	ps.in_or_st = false;	/* turn off flag for structure decl or
 				 * initialization */
+    }
     /* parenthesized type following sizeof or offsetof is not a cast */
     if (ps.keyword == rw_offsetof || ps.keyword == rw_sizeof)
 	ps.not_cast_mask |= 1 << ps.p_l_follow;
+}
 static void
 process_rparen_or_rbracket(int *inout_sp_sw, int *inout_force_nl,
 			 token_type hd_type)
+{
     if (ps.cast_mask & (1 << ps.p_l_follow) & ~ps.not_cast_mask) {
 	ps.last_u_d = true;
 	ps.cast_mask &= (1 << ps.p_l_follow) - 1;
 	ps.want_blank = opt.space_after_cast;
     } else
 	ps.want_blank = true;
     ps.not_cast_mask &= (1 << ps.p_l_follow) - 1;
     if (--ps.p_l_follow < 0) {
 	ps.p_l_follow = 0;
 	diag(0, "Extra %c", *token);
+    }
     if (e_code == s_code)	/* if the paren starts the line */
 	ps.paren_level = ps.p_l_follow;	/* then indent it */
     *e_code++ = token[0];
     if (*inout_sp_sw && (ps.p_l_follow == 0)) {	/* check for end of if
 				 * (...), or some such */
 	*inout_sp_sw = false;
 	*inout_force_nl = true;	/* must force newline after if */
 	ps.last_u_d = true;	/* inform lexi that a following
 				 * operator is unary */
 	ps.in_stmt = false;	/* dont use stmt continuation indentation */
 	parse(hd_type);		/* let parser worry about if, or whatever */
+    }
     ps.search_brace = opt.btype_2; /* this should ensure that constructs such
 				 * as main(){...} and int[]{...} have their
 				 * braces put in the right place */
+}
 static void
 process_unary_op(int dec_ind, int tabs_to_var)
+{
     if (!ps.dumped_decl_indent && ps.in_decl && !ps.block_init &&
 	ps.procname[0] == '\0' && ps.paren_level == 0) {
 	/* pointer declarations */
 	/*
 	 * if this is a unary op in a declaration, we should indent
 	 * this token
 	 */
 	int i;
 	for (i = 0; token[i]; ++i)
 	    /* find length of token */;
 	indent_declaration(dec_ind - i, tabs_to_var);
 	ps.dumped_decl_indent = true;
     } else if (ps.want_blank)
 	*e_code++ = ' ';
+    {
-	int len = e_token - s_token;
+	size_t len = e_token - s_token;
 	check_size_code(len);
 	memcpy(e_code, token, len);
 	e_code += len;
+    }
     ps.want_blank = false;
+}
 static void
 process_binary_op(void)
+{
-    int len = e_token - s_token;
+    size_t len = e_token - s_token;
     check_size_code(len + 1);
     if (ps.want_blank)
 	*e_code++ = ' ';
     memcpy(e_code, token, len);
     e_code += len;
     ps.want_blank = true;
+}
 static void
 process_postfix_op(void)
+{
     *e_code++ = token[0];
     *e_code++ = token[1];
     ps.want_blank = true;
+}
 static void
 process_question(int *inout_squest)
+{
     (*inout_squest)++;		/* this will be used when a later colon
 				 * appears so we can distinguish the
 				 * <c>?<n>:<n> construct */
     if (ps.want_blank)
 	*e_code++ = ' ';
     *e_code++ = '?';
     ps.want_blank = true;
+}
 static void
 process_colon(int *inout_squest, int *inout_force_nl, int *inout_scase)
+{
     if (*inout_squest > 0) {	/* it is part of the <c>?<n>: <n> construct */
 	--*inout_squest;
 	if (ps.want_blank)
 	    *e_code++ = ' ';
 	*e_code++ = ':';
 	ps.want_blank = true;
 	return;
+    }
     if (ps.in_or_st) {
 	*e_code++ = ':';
 	ps.want_blank = false;
 	return;
+    }
     ps.in_stmt = false;		/* seeing a label does not imply we are in a
 				 * stmt */
     /*
      * turn everything so far into a label
      */
+    {
-	int len = e_code - s_code;
+	size_t len = e_code - s_code;
 	check_size_label(len + 3);
 	memcpy(e_lab, s_code, len);
 	e_lab += len;
 	*e_lab++ = ':';
 	*e_lab = '\0';
 	e_code = s_code;
+    }
     *inout_force_nl = ps.pcase = *inout_scase;	/* ps.pcase will be used by
 						 * dump_line to decide how to
 						 * indent the label. force_nl
 						 * will force a case n: to be
 						 * on a line by itself */
     *inout_scase = false;
     ps.want_blank = false;
+}
 static void
 process_semicolon(int *inout_scase, int *inout_squest, int const dec_ind,
 		  int const tabs_to_var, int *inout_sp_sw,
 		  token_type const hd_type,
 		  int *inout_force_nl)
+{
     if (ps.dec_nest == 0)
 	ps.in_or_st = false;	/* we are not in an initialization or
 				 * structure declaration */
     *inout_scase = false; /* these will only need resetting in an error */
     *inout_squest = 0;
     if (ps.last_token == rparen)
 	ps.in_parameter_declaration = 0;
     ps.cast_mask = 0;
     ps.not_cast_mask = 0;
     ps.block_init = 0;
     ps.block_init_level = 0;
     ps.just_saw_decl--;
     if (ps.in_decl && s_code == e_code && !ps.block_init &&
 	!ps.dumped_decl_indent && ps.paren_level == 0) {
 	/* indent stray semicolons in declarations */
 	indent_declaration(dec_ind - 1, tabs_to_var);
 	ps.dumped_decl_indent = true;
+    }
     ps.in_decl = (ps.dec_nest > 0);	/* if we were in a first level
 						 * structure declaration, we
 						 * arent any more */
     if ((!*inout_sp_sw || hd_type != for_exprs) && ps.p_l_follow > 0) {
 	/*
 	 * This should be true iff there were unbalanced parens in the
 	 * stmt.  It is a bit complicated, because the semicolon might
 	 * be in a for stmt
 	 */
 	diag(1, "Unbalanced parens");
 	ps.p_l_follow = 0;
 	if (*inout_sp_sw) {	/* this is a check for an if, while, etc. with
 				 * unbalanced parens */
 	    *inout_sp_sw = false;
 	    parse(hd_type);	/* dont lose the if, or whatever */
+	}
+    }
     *e_code++ = ';';
     ps.want_blank = true;
     ps.in_stmt = (ps.p_l_follow > 0);	/* we are no longer in the
 				 * middle of a stmt */
     if (!*inout_sp_sw) {	/* if not if for (;;) */
 	parse(semicolon);	/* let parser know about end of stmt */
 	*inout_force_nl = true;/* force newline after an end of stmt */
+    }
+}
 static void
 process_lbrace(int *inout_force_nl, int *inout_sp_sw, token_type hd_type,
 	       int *di_stack, int di_stack_cap, int *inout_dec_ind)
+{
     ps.in_stmt = false;	/* dont indent the {} */
     if (!ps.block_init)
 	*inout_force_nl = true;	/* force other stuff on same line as '{' onto
 				 * new line */
     else if (ps.block_init_level <= 0)
 	ps.block_init_level = 1;
     else
 	ps.block_init_level++;
     if (s_code != e_code && !ps.block_init) {
 	if (!opt.btype_2) {
 	    dump_line();
 	    ps.want_blank = false;
 	} else if (ps.in_parameter_declaration && !ps.in_or_st) {
 	    ps.i_l_follow = 0;
 	    if (opt.function_brace_split) { /* dump the line prior
 				 * to the brace ... */
 		dump_line();
 		ps.want_blank = false;
 	    } else		/* add a space between the decl and brace */
 		ps.want_blank = true;
+	}
+    }
     if (ps.in_parameter_declaration)
 	prefix_blankline_requested = 0;
     if (ps.p_l_follow > 0) {	/* check for preceding unbalanced
 				 * parens */
 	diag(1, "Unbalanced parens");
 	ps.p_l_follow = 0;
 	if (*inout_sp_sw) {	/* check for unclosed if, for, etc. */
 	    *inout_sp_sw = false;
 	    parse(hd_type);
 	    ps.ind_level = ps.i_l_follow;
+	}
+    }
     if (s_code == e_code)
 	ps.ind_stmt = false;	/* dont put extra indentation on line
 				 * with '{' */
     if (ps.in_decl && ps.in_or_st) {	/* this is either a structure
 				 * declaration or an init */
 	di_stack[ps.dec_nest] = *inout_dec_ind;
 	if (++ps.dec_nest == di_stack_cap) {
 	    diag(0, "Reached internal limit of %d struct levels",
 		 di_stack_cap);
 	    ps.dec_nest--;
+	}
 	/* ?		dec_ind = 0; */
     } else {
 	ps.decl_on_line = false;	/* we can't be in the middle of
 						 * a declaration, so don't do
 						 * special indentation of
 						 * comments */
 	if (opt.blanklines_after_declarations_at_proctop
 	    && ps.in_parameter_declaration)
 	    postfix_blankline_requested = 1;
 	ps.in_parameter_declaration = 0;
 	ps.in_decl = false;
+    }
     *inout_dec_ind = 0;
     parse(lbrace);	/* let parser know about this */
     if (ps.want_blank)	/* put a blank before '{' if '{' is not at
 				 * start of line */
 	*e_code++ = ' ';
     ps.want_blank = false;
     *e_code++ = '{';
     ps.just_saw_decl = 0;
+}
 static void
 process_rbrace(int *inout_sp_sw, int *inout_dec_ind, const int *di_stack)
+{
     if (ps.p_stack[ps.tos] == decl && !ps.block_init)	/* semicolons can be
 				 * omitted in declarations */
 	parse(semicolon);
     if (ps.p_l_follow) {	/* check for unclosed if, for, else. */
 	diag(1, "Unbalanced parens");
 	ps.p_l_follow = 0;
 	*inout_sp_sw = false;
+    }
     ps.just_saw_decl = 0;
     ps.block_init_level--;
     if (s_code != e_code && !ps.block_init) {	/* '}' must be first on line */
 	if (opt.verbose)
 	    diag(0, "Line broken");
 	dump_line();
+    }
     *e_code++ = '}';
     ps.want_blank = true;
     ps.in_stmt = ps.ind_stmt = false;
     if (ps.dec_nest > 0) { /* we are in multi-level structure declaration */
 	*inout_dec_ind = di_stack[--ps.dec_nest];
 	if (ps.dec_nest == 0 && !ps.in_parameter_declaration)
 	    ps.just_saw_decl = 2;
 	ps.in_decl = true;
+    }
     prefix_blankline_requested = 0;
     parse(rbrace);		/* let parser know about this */
     ps.search_brace = opt.cuddle_else
 		      && ps.p_stack[ps.tos] == if_expr_stmt
 		      && ps.il[ps.tos] >= ps.ind_level;
     if (ps.tos <= 1 && opt.blanklines_after_procs && ps.dec_nest <= 0)
 	postfix_blankline_requested = 1;
+}
 static void
 process_keyword_do_else(int *inout_force_nl, int *inout_last_else)
+{
     ps.in_stmt = false;
     if (*token == 'e') {
 	if (e_code != s_code && (!opt.cuddle_else || e_code[-1] != '}')) {
 	    if (opt.verbose)
 		diag(0, "Line broken");
 	    dump_line();	/* make sure this starts a line */
 	    ps.want_blank = false;
+	}
 	*inout_force_nl = true;/* also, following stuff must go onto new line */
 	*inout_last_else = 1;
 	parse(keyword_else);
     } else {
 	if (e_code != s_code) {	/* make sure this starts a line */
 	    if (opt.verbose)
 		diag(0, "Line broken");
 	    dump_line();
 	    ps.want_blank = false;
+	}
 	*inout_force_nl = true;/* also, following stuff must go onto new line */
 	*inout_last_else = 0;
 	parse(keyword_do);
+    }
+}
 static void
 process_decl(int *out_dec_ind, int *out_tabs_to_var)
+{
     parse(decl);		/* let parser worry about indentation */
     if (ps.last_token == rparen && ps.tos <= 1) {
 	if (s_code != e_code) {
 	    dump_line();
 	    ps.want_blank = 0;
+	}
+    }
     if (ps.in_parameter_declaration && opt.indent_parameters && ps.dec_nest == 0) {
 	ps.ind_level = ps.i_l_follow = 1;
 	ps.ind_stmt = 0;
+    }
     ps.in_or_st = true;		/* this might be a structure or initialization
 				 * declaration */
     ps.in_decl = ps.decl_on_line = ps.last_token != type_def;
     if ( /* !ps.in_or_st && */ ps.dec_nest <= 0)
 	ps.just_saw_decl = 2;
     prefix_blankline_requested = 0;
     int i;
     for (i = 0; token[i++];);	/* get length of token */
     if (ps.ind_level == 0 || ps.dec_nest > 0) {
 	/* global variable or struct member in local variable */
 	*out_dec_ind = opt.decl_indent > 0 ? opt.decl_indent : i;
 	*out_tabs_to_var = (opt.use_tabs ? opt.decl_indent > 0 : 0);
     } else {
 	/* local variable */
 	*out_dec_ind = opt.local_decl_indent > 0 ? opt.local_decl_indent : i;
 	*out_tabs_to_var = (opt.use_tabs ? opt.local_decl_indent > 0 : 0);
+    }
+}
 static void
 process_ident(token_type type_code, int dec_ind, int tabs_to_var,
 	      int *inout_sp_sw, int *inout_force_nl, token_type hd_type)
+{
     if (ps.in_decl) {
 	if (type_code == funcname) {
 	    ps.in_decl = false;
 	    if (opt.procnames_start_line && s_code != e_code) {
 		*e_code = '\0';
 		dump_line();
 	    } else if (ps.want_blank) {
 		*e_code++ = ' ';
+	    }
 	    ps.want_blank = false;
 	} else if (!ps.block_init && !ps.dumped_decl_indent &&
 		   ps.paren_level == 0) { /* if we are in a declaration, we
 					    * must indent identifier */
 	    indent_declaration(dec_ind, tabs_to_var);
 	    ps.dumped_decl_indent = true;
 	    ps.want_blank = false;
+	}
     } else if (*inout_sp_sw && ps.p_l_follow == 0) {
 	*inout_sp_sw = false;
 	*inout_force_nl = true;
 	ps.last_u_d = true;
 	ps.in_stmt = false;
 	parse(hd_type);
+    }
+}
 static void
 copy_id(void)
+{
-    int len = e_token - s_token;
+    size_t len = e_token - s_token;
     check_size_code(len + 1);
     if (ps.want_blank)
 	*e_code++ = ' ';
     memcpy(e_code, s_token, len);
     e_code += len;
+}
 static void
 process_string_prefix(void)
+{
-    int len = e_token - s_token;
+    size_t len = e_token - s_token;
     check_size_code(len + 1);
     if (ps.want_blank)
 	*e_code++ = ' ';
     memcpy(e_code, token, len);
     e_code += len;
     ps.want_blank = false;
+}
 static void
 process_period(void)
+{
     *e_code++ = '.';		/* move the period into line */
     ps.want_blank = false;	/* dont put a blank after a period */
+}
 static void
 process_comma(int dec_ind, int tabs_to_var, int *inout_force_nl)
+{
     ps.want_blank = (s_code != e_code);	/* only put blank after comma
 				 * if comma does not start the line */
     if (ps.in_decl && ps.procname[0] == '\0' && !ps.block_init &&
 	!ps.dumped_decl_indent && ps.paren_level == 0) {
 	/* indent leading commas and not the actual identifiers */
 	indent_declaration(dec_ind - 1, tabs_to_var);
 	ps.dumped_decl_indent = true;
+    }
     *e_code++ = ',';
     if (ps.p_l_follow == 0) {
 	if (ps.block_init_level <= 0)
 	    ps.block_init = 0;
 	if (break_comma && (!opt.leave_comma ||
 			    indentation_after_range(
 				    compute_code_indent(), s_code, e_code)
 			    >= opt.max_line_length - opt.tabsize))
 	    *inout_force_nl = true;
+    }
+}
 static void
 process_preprocessing(void)
+{
     if (s_com != e_com || s_lab != e_lab || s_code != e_code)
 	dump_line();
     check_size_label(1);
     *e_lab++ = '#';	/* move whole line to 'label' buffer */
+    {
 	int         in_comment = 0;
 	int         com_start = 0;
 	char        quote = '\0';
 	int         com_end = 0;
 	while (*buf_ptr == ' ' || *buf_ptr == '\t') {
 	    buf_ptr++;
 	    if (buf_ptr >= buf_end)
 		fill_buffer();
+	}
 	while (*buf_ptr != '\n' || (in_comment && !had_eof)) {
 	    check_size_label(2);
 	    *e_lab = *buf_ptr++;
 	    if (buf_ptr >= buf_end)
 		fill_buffer();
 	    switch (*e_lab++) {
 	    case '\\':
 		if (!in_comment) {
 		    *e_lab++ = *buf_ptr++;
 		    if (buf_ptr >= buf_end)
 			fill_buffer();
+		}
 		break;
 	    case '/':
 		if (*buf_ptr == '*' && !in_comment && quote == '\0') {
 		    in_comment = 1;
 		    *e_lab++ = *buf_ptr++;
-		    com_start = e_lab - s_lab - 2;
+		    com_start = (int)(e_lab - s_lab) - 2;
+		}
 		break;
 	    case '"':
 		if (quote == '"')
 		    quote = '\0';
 		else if (quote == '\0')
 		    quote = '"';
 		break;
 	    case '\'':
 		if (quote == '\'')
 		    quote = '\0';
 		else if (quote == '\0')
 		    quote = '\'';
 		break;
 	    case '*':
 		if (*buf_ptr == '/' && in_comment) {
 		    in_comment = 0;
 		    *e_lab++ = *buf_ptr++;
-		    com_end = e_lab - s_lab;
+		    com_end = (int)(e_lab - s_lab);
+		}
 		break;
+	    }
+	}
 	while (e_lab > s_lab && (e_lab[-1] == ' ' || e_lab[-1] == '\t'))
 	    e_lab--;
 	if (e_lab - s_lab == com_end && bp_save == NULL) {
 	    /* comment on preprocessor line */
 	    if (sc_end == NULL) {	/* if this is the first comment,
 						 * we must set up the buffer */
 		save_com = sc_buf;
 		sc_end = &save_com[0];
 	    } else {
 		*sc_end++ = '\n';	/* add newline between
 						 * comments */
 		*sc_end++ = ' ';
 		--line_no;
+	    }
 	    if (sc_end - save_com + com_end - com_start > sc_size)
 		errx(1, "input too long");
 	    memmove(sc_end, s_lab + com_start, com_end - com_start);
 	    sc_end += com_end - com_start;
 	    e_lab = s_lab + com_start;
 	    while (e_lab > s_lab && (e_lab[-1] == ' ' || e_lab[-1] == '\t'))
 		e_lab--;
 	    bp_save = buf_ptr;	/* save current input buffer */
 	    be_save = buf_end;
 	    buf_ptr = save_com;	/* fix so that subsequent calls to lexi will
 				 * take tokens out of save_com */
 	    *sc_end++ = ' ';	/* add trailing blank, just in case */
 	    buf_end = sc_end;
 	    sc_end = NULL;
 	    debug_println("switched buf_ptr to save_com");
+	}
 	check_size_label(1);
 	*e_lab = '\0';	/* null terminate line */
 	ps.pcase = false;
+    }
     if (strncmp(s_lab, "#if", 3) == 0) { /* also ifdef, ifndef */
 	if ((size_t)ifdef_level < nitems(state_stack)) {
 	    match_state[ifdef_level].tos = -1;
 	    state_stack[ifdef_level++] = ps;
 	} else
 	    diag(1, "#if stack overflow");
     } else if (strncmp(s_lab, "#el", 3) == 0) { /* else, elif */
 	if (ifdef_level <= 0)
 	    diag(1, s_lab[3] == 'i' ? "Unmatched #elif" : "Unmatched #else");
 	else {
 	    match_state[ifdef_level - 1] = ps;
 	    ps = state_stack[ifdef_level - 1];
+	}
     } else if (strncmp(s_lab, "#endif", 6) == 0) {
 	if (ifdef_level <= 0)
 	    diag(1, "Unmatched #endif");
 	else
 	    ifdef_level--;
     } else {
 	static const struct directives {
 	    int size;
 	    const char *string;
 	} recognized[] = {
 		{7, "include"},
 		{6, "define"},
 		{5, "undef"},
 		{4, "line"},
 		{5, "error"},
 		{6, "pragma"}
 	};
 	int d = nitems(recognized);
 	while (--d >= 0)
 	    if (strncmp(s_lab + 1, recognized[d].string, recognized[d].size) == 0)
 		break;
 	if (d < 0) {
 	    diag(1, "Unrecognized cpp directive");
 	    return;
+	}
+    }
     if (opt.blanklines_around_conditional_compilation) {
 	postfix_blankline_requested++;
 	n_real_blanklines = 0;
     } else {
 	postfix_blankline_requested = 0;
 	prefix_blankline_requested = 0;
+    }
     /*
      * subsequent processing of the newline character will cause the line to
      * be printed
      */
+}
 static void
 main_loop(void)
+{
     token_type type_code;
     int force_nl;		/* when true, code must be broken */
     int last_else = false;	/* true iff last keyword was an else */
     int         dec_ind;	/* current indentation for declarations */
     int         di_stack[20];	/* a stack of structure indentation levels */
     int		tabs_to_var;	/* true if using tabs to indent to var name */
     int         sp_sw;		/* when true, we are in the expression of
 				 * if(...), while(...), etc. */
     token_type  hd_type = end_of_file; /* used to store type of stmt
 				 * for if (...), for (...), etc */
     int         squest;		/* when this is positive, we have seen a ?
 				 * without the matching : in a <c>?<s>:<s>
 				 * construct */
     int         scase;		/* set to true when we see a case, so we will
 				 * know what to do with the following colon */
     sp_sw = force_nl = false;
     dec_ind = 0;
     di_stack[ps.dec_nest = 0] = 0;
     scase = false;
     squest = 0;
     tabs_to_var = 0;
-    while (1) {			/* this is the main loop.  it will go until we
+    for (;;) {			/* this is the main loop.  it will go until we
 				 * reach eof */
 	int comment_buffered = false;
 	type_code = lexi(&ps);	/* lexi reads one token.  The actual
 				 * characters read are stored in "token". lexi
 				 * returns a code indicating the type of token */
 	/*
 	 * The following code moves newlines and comments following an if (),
 	 * while (), else, etc. up to the start of the following stmt to
 	 * a buffer. This allows proper handling of both kinds of brace
 	 * placement (-br, -bl) and cuddling "else" (-ce).
 	 */
 	search_brace(&type_code, &force_nl, &comment_buffered, &last_else);
 	if (type_code == end_of_file) {
 	    process_end_of_file();
 	    return;
+	}
 	if (
 		type_code != comment &&
 		type_code != newline &&
 		type_code != preprocessing &&
 		type_code != form_feed) {
 	    process_comment_in_code(type_code, &force_nl);
 	} else if (type_code != comment) /* preserve force_nl thru a comment */
 	    force_nl = false;	/* cancel forced newline after newline, form
 				 * feed, etc */
 	/*-----------------------------------------------------*\
 	|	   do switch on type of token scanned		|
 	\*-----------------------------------------------------*/
 	check_size_code(3);	/* maximum number of increments of e_code
 				 * before the next check_size_code or
 				 * dump_line() is 2. After that there's the
 				 * final increment for the null character. */
 	switch (type_code) {	/* now, decide what to do with the token */
 	case form_feed:		/* found a form feed in line */
 	    process_form_feed();
 	    break;
 	case newline:
 	    process_newline();
 	    break;
 	case lparen:		/* got a '(' or '[' */
 	    process_lparen_or_lbracket(dec_ind, tabs_to_var, sp_sw);
 	    break;
 	case rparen:		/* got a ')' or ']' */
 	    process_rparen_or_rbracket(&sp_sw, &force_nl, hd_type);
 	    break;
 	case unary_op:		/* this could be any unary operation */
 	    process_unary_op(dec_ind, tabs_to_var);
 	    break;
 	case binary_op:		/* any binary operation */
 	    process_binary_op();
 	    break;
 	case postfix_op:	/* got a trailing ++ or -- */
 	    process_postfix_op();
 	    break;
 	case question:		/* got a ? */
 	    process_question(&squest);
 	    break;
 	case case_label:	/* got word 'case' or 'default' */
 	    scase = true;	/* so we can process the later colon properly */
 	    goto copy_id;
 	case colon:		/* got a ':' */
 	    process_colon(&squest, &force_nl, &scase);
 	    break;
 	case semicolon:		/* got a ';' */
 	    process_semicolon(&scase, &squest, dec_ind, tabs_to_var, &sp_sw,
 		hd_type, &force_nl);
 	    break;
 	case lbrace:		/* got a '{' */
 	    process_lbrace(&force_nl, &sp_sw, hd_type, di_stack,
-		nitems(di_stack), &dec_ind);
+		(int)nitems(di_stack), &dec_ind);
 	    break;
 	case rbrace:		/* got a '}' */
 	    process_rbrace(&sp_sw, &dec_ind, di_stack);
 	    break;
 	case switch_expr:	/* got keyword "switch" */
 	    sp_sw = true;
 	    hd_type = switch_expr; /* keep this for when we have seen the
 				 * expression */
 	    goto copy_id;	/* go move the token into buffer */
 	case keyword_for_if_while:
 	    sp_sw = true;	/* the interesting stuff is done after the
 				 * expression is scanned */
 	    hd_type = (*token == 'i' ? if_expr :
 		       (*token == 'w' ? while_expr : for_exprs));
 	    /* remember the type of header for later use by parser */
 	    goto copy_id;	/* copy the token into line */
 	case keyword_do_else:
 	    process_keyword_do_else(&force_nl, &last_else);
 	    goto copy_id;	/* move the token into line */
 	case type_def:
 	case storage_class:
 	    prefix_blankline_requested = 0;
 	    goto copy_id;
 	case keyword_struct_union_enum:
 	    if (ps.p_l_follow > 0)
 		goto copy_id;
 	    /* FALLTHROUGH */
 	case decl:		/* we have a declaration type (int, etc.) */
 	    process_decl(&dec_ind, &tabs_to_var);
 	    goto copy_id;
 	case funcname:
 	case ident:		/* got an identifier or constant */
 	    process_ident(type_code, dec_ind, tabs_to_var, &sp_sw, &force_nl,
 		hd_type);
     copy_id:
 	    copy_id();
 	    if (type_code != funcname)
 		ps.want_blank = true;
 	    break;
 	case string_prefix:
 	    process_string_prefix();
 	    break;
 	case period:
 	    process_period();
 	    break;
 	case comma:
 	    process_comma(dec_ind, tabs_to_var, &force_nl);
 	    break;
 	case preprocessing:	/* '#' */
 	    process_preprocessing();
 	    break;
 	case comment:		/* the initial '/' '*' or '//' of a comment */
 	    process_comment();
 	    break;
 	default:
 	    break;
 	}			/* end of big switch stmt */
 	*e_code = '\0';		/* make sure code section is null terminated */
 	if (type_code != comment &&
 	    type_code != newline &&
 	    type_code != preprocessing)
 	    ps.last_token = type_code;
+    }
+}
 int
 main(int argc, char **argv)
+{
     main_init_globals();
     main_parse_command_line(argc, argv);
 #if HAVE_CAPSICUM
     init_capsicum();
 #endif
     main_prepare_parsing();
     main_loop();
+}
 /*
  * copy input file to backup file if in_name is /blah/blah/blah/file, then
  * backup file will be ".Bfile" then make the backup file the input and
  * original input file the output
  */
 static void
 bakcopy(void)
+{
-    int         n,
+    ssize_t n;
-                bakchn;
+    int bakchn;
-    char        buff[8 * 1024];
+    char buff[8 * 1024];
     const char *p;
     /* construct file name .Bfile */
     for (p = in_name; *p; p++);	/* skip to end of string */
     while (p > in_name && *p != '/')	/* find last '/' */
 	p--;
     if (*p == '/')
 	p++;
     sprintf(bakfile, "%s%s", p, simple_backup_suffix);
     /* copy in_name to backup file */
     bakchn = creat(bakfile, 0600);
     if (bakchn < 0)
 	err(1, "%s", bakfile);
     while ((n = read(fileno(input), buff, sizeof(buff))) > 0)
-	if (write(bakchn, buff, n) != n)
+	if (write(bakchn, buff, (size_t)n) != n)
 	    err(1, "%s", bakfile);
     if (n < 0)
 	err(1, "%s", in_name);
     close(bakchn);
     fclose(input);
     /* re-open backup file as the input file */
     input = fopen(bakfile, "r");
     if (input == NULL)
 	err(1, "%s", bakfile);
     /* now the original input file will be the output */
     output = fopen(in_name, "w");
     if (output == NULL) {
 	unlink(bakfile);
 	err(1, "%s", in_name);
+    }
+}
 static void
 indent_declaration(int cur_dec_ind, int tabs_to_var)
+{
-    int pos = e_code - s_code;
+    int pos = (int)(e_code - s_code);
     char *startpos = e_code;
     /*
      * get the tab math right for indentations that are not multiples of tabsize
      */
     if ((ps.ind_level * opt.indent_size) % opt.tabsize != 0) {
 	pos += (ps.ind_level * opt.indent_size) % opt.tabsize;
 	cur_dec_ind += (ps.ind_level * opt.indent_size) % opt.tabsize;
+    }
     if (tabs_to_var) {
 	int tpos;
 	check_size_code(cur_dec_ind / opt.tabsize);
 	while ((tpos = opt.tabsize * (1 + pos / opt.tabsize)) <= cur_dec_ind) {
 	    *e_code++ = '\t';
 	    pos = tpos;
+	}
+    }
     check_size_code(cur_dec_ind - pos + 1);
     while (pos < cur_dec_ind) {
 	*e_code++ = ' ';
 	pos++;
+    }
     if (e_code == startpos && ps.want_blank) {
 	*e_code++ = ' ';
 	ps.want_blank = false;
+    }
+}
 #ifdef debug
 void
 debug_printf(const char *fmt, ...)
+{
     FILE *f = output == stdout ? stderr : stdout;
     va_list ap;
     va_start(ap, fmt);
     vfprintf(f, fmt, ap);
     va_end(ap);
+}
 void
 debug_println(const char *fmt, ...)
+{
     FILE *f = output == stdout ? stderr : stdout;
     va_list ap;
     va_start(ap, fmt);
     vfprintf(f, fmt, ap);
     va_end(ap);
     fprintf(f, "\n");
+}
 void
 debug_vis_range(const char *prefix, const char *s, const char *e,
 		const char *suffix)
+{
     debug_printf("%s", prefix);
     for (const char *p = s; p < e; p++) {
 	if (isprint((unsigned char)*p) && *p != '\\' && *p != '"')
 	    debug_printf("%c", *p);
 	else if (*p == '\n')
 	    debug_printf("\\n");
 	else if (*p == '\t')
 	    debug_printf("\\t");
 	else
 	    debug_printf("\\x%02x", *p);
+    }
     debug_printf("%s", suffix);
+}
 #endif

 @@ -1,470 +1,470 @@
-/*	$NetBSD: io.c,v 1.47 2021/03/13 18:46:39 rillig Exp $	*/
+/*	$NetBSD: io.c,v 1.48 2021/03/14 00:22:16 rillig Exp $	*/
 /*-
  * SPDX-License-Identifier: BSD-4-Clause
+ *
  * Copyright (c) 1985 Sun Microsystems, Inc.
  * Copyright (c) 1980, 1993
  *	The Regents of the University of California.  All rights reserved.
  * All rights reserved.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. All advertising materials mentioning features or use of this software
  *    must display the following acknowledgement:
  *	This product includes software developed by the University of
  *	California, Berkeley and its contributors.
  * 4. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
 #if 0
 #ifndef lint
 static char sccsid[] = "@(#)io.c	8.1 (Berkeley) 6/6/93";
 #endif /* not lint */
 #endif
 #include <sys/cdefs.h>
 #ifndef lint
 #if defined(__NetBSD__)
-__RCSID("$NetBSD: io.c,v 1.47 2021/03/13 18:46:39 rillig Exp $");
+__RCSID("$NetBSD: io.c,v 1.48 2021/03/14 00:22:16 rillig Exp $");
 #elif defined(__FreeBSD__)
 __FBSDID("$FreeBSD: head/usr.bin/indent/io.c 334927 2018-06-10 16:44:18Z pstef $");
 #endif
 #endif
 #include <ctype.h>
 #include <err.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <stdarg.h>
 #include "indent.h"
 int         comment_open;
 static int  paren_indent;
 static void
 output_char(char ch)
+{
     fputc(ch, output);
     debug_vis_range("output_char '", &ch, &ch + 1, "'\n");
+}
 static void
 output_range(const char *s, const char *e)
+{
     fwrite(s, 1, (size_t)(e - s), output);
     debug_vis_range("output_range \"", s, e, "\"\n");
+}
 static inline void
 output_string(const char *s)
+{
     output_range(s, s + strlen(s));
+}
 static int
 output_indent(int old_ind, int new_ind)
+{
     int ind = old_ind;
     if (opt.use_tabs) {
 	int tabsize = opt.tabsize;
 	int n = new_ind / tabsize - ind / tabsize;
 	if (n > 0)
 	    ind -= ind % tabsize;
 	for (int i = 0; i < n; i++) {
 	    fputc('\t', output);
 	    ind += tabsize;
+	}
+    }
     for (; ind < new_ind; ind++)
         fputc(' ', output);
     debug_println("output_indent %d", ind);
     return ind;
+}
 /*
  * dump_line is the routine that actually effects the printing of the new
  * source. It prints the label section, followed by the code section with
  * the appropriate nesting level, followed by any comments.
  */
 void
 dump_line(void)
+{
     int cur_col;
     static int  not_first_line;
     if (ps.procname[0]) {
 	ps.ind_level = 0;
 	ps.procname[0] = 0;
+    }
     if (s_code == e_code && s_lab == e_lab && s_com == e_com) {
 	if (suppress_blanklines > 0)
 	    suppress_blanklines--;
 	else {
 	    ps.bl_line = true;
 	    n_real_blanklines++;
+	}
     } else if (!inhibit_formatting) {
 	suppress_blanklines = 0;
 	ps.bl_line = false;
 	if (prefix_blankline_requested && not_first_line) {
 	    if (opt.swallow_optional_blanklines) {
 		if (n_real_blanklines == 1)
 		    n_real_blanklines = 0;
 	    } else {
 		if (n_real_blanklines == 0)
 		    n_real_blanklines = 1;
+	    }
+	}
 	while (--n_real_blanklines >= 0)
 	    output_char('\n');
 	n_real_blanklines = 0;
 	if (ps.ind_level == 0)
 	    ps.ind_stmt = 0;	/* this is a class A kludge. dont do
 				 * additional statement indentation if we are
 				 * at bracket level 0 */
 	if (e_lab != s_lab || e_code != s_code)
 	    ++code_lines;	/* keep count of lines with code */
 	if (e_lab != s_lab) {	/* print lab, if any */
 	    if (comment_open) {
 		comment_open = 0;
 		output_string(".*/\n");
+	    }
 	    while (e_lab > s_lab && (e_lab[-1] == ' ' || e_lab[-1] == '\t'))
 		e_lab--;
 	    *e_lab = '\0';
 	    cur_col = 1 + output_indent(0, compute_label_indent());
 	    if (s_lab[0] == '#' && (strncmp(s_lab, "#else", 5) == 0
 				    || strncmp(s_lab, "#endif", 6) == 0)) {
 		char *s = s_lab;
 		if (e_lab[-1] == '\n') e_lab--;
 		do {
 		    output_char(*s++);
 		} while (s < e_lab && 'a' <= *s && *s <= 'z');
 		while ((*s == ' ' || *s == '\t') && s < e_lab)
 		    s++;
 		if (s < e_lab) {
 		    if (s[0] == '/' && s[1] == '*') {
 			output_char('\t');
 			output_range(s, e_lab);
 		    } else {
 		        output_string("\t/* ");
 			output_range(s, e_lab);
 			output_string(" */");
+		    }
+		}
 	    } else
 	        output_range(s_lab, e_lab);
 	    cur_col = 1 + indentation_after(cur_col - 1, s_lab);
 	} else
 	    cur_col = 1;	/* there is no label section */
 	ps.pcase = false;
 	if (s_code != e_code) {	/* print code section, if any */
 	    if (comment_open) {
 		comment_open = 0;
 		output_string(".*/\n");
+	    }
 	    int target_col = 1 + compute_code_indent();
+	    {
 		int i;
 		for (i = 0; i < ps.p_l_follow; i++) {
 		    if (ps.paren_indents[i] >= 0) {
 			int ind = ps.paren_indents[i];
 			/*
 			 * XXX: this mix of 'indent' and 'column' smells like
 			 * an off-by-one error.
 			 */
 			ps.paren_indents[i] = -(ind + target_col);
 			debug_println(
 			    "setting pi[%d] from %d to %d for column %d",
 			    i, ind, ps.paren_indents[i], target_col);
+		    }
+		}
+	    }
 	    cur_col = 1 + output_indent(cur_col - 1, target_col - 1);
 	    output_range(s_code, e_code);
 	    cur_col = 1 + indentation_after(cur_col - 1, s_code);
+	}
 	if (s_com != e_com) {		/* print comment, if any */
 	    int target_col = ps.com_col;
 	    char *com_st = s_com;
 	    target_col += ps.comment_delta;
 	    while (*com_st == '\t')	/* consider original indentation in
 				 * case this is a box comment */
 		com_st++, target_col += opt.tabsize;
 	    while (target_col <= 0)
 		if (*com_st == ' ')
 		    target_col++, com_st++;
 		else if (*com_st == '\t') {
 		    target_col = opt.tabsize * (1 + (target_col - 1) / opt.tabsize) + 1;
 		    com_st++;
 		} else
 		    target_col = 1;
 	    if (cur_col > target_col) {	/* if comment can't fit on this line,
 				 * put it on next line */
 		output_char('\n');
 		cur_col = 1;
 		++ps.out_lines;
+	    }
 	    while (e_com > com_st && isspace((unsigned char)e_com[-1]))
 		e_com--;
 	    (void)output_indent(cur_col - 1, target_col - 1);
 	    output_range(com_st, e_com);
 	    ps.comment_delta = ps.n_comment_delta;
 	    ++ps.com_lines;	/* count lines with comments */
+	}
 	if (ps.use_ff)
 	    output_char('\014');
 	else
 	    output_char('\n');
 	++ps.out_lines;
 	if (ps.just_saw_decl == 1 && opt.blanklines_after_declarations) {
 	    prefix_blankline_requested = 1;
 	    ps.just_saw_decl = 0;
 	} else
 	    prefix_blankline_requested = postfix_blankline_requested;
 	postfix_blankline_requested = 0;
+    }
     /* keep blank lines after '//' comments */
     if (e_com - s_com > 1 && s_com[1] == '/')
 	output_range(s_token, e_token);
     ps.decl_on_line = ps.in_decl; /* if we are in the middle of a declaration,
 				 * remember that fact for proper comment
 				 * indentation */
     ps.ind_stmt = ps.in_stmt & ~ps.in_decl; /* next line should be indented if
 				 * we have not completed this stmt and if we
 				 * are not in the middle of a declaration */
     ps.use_ff = false;
     ps.dumped_decl_indent = 0;
     *(e_lab = s_lab) = '\0';	/* reset buffers */
     *(e_code = s_code) = '\0';
     *(e_com = s_com = combuf + 1) = '\0';
     ps.ind_level = ps.i_l_follow;
     ps.paren_level = ps.p_l_follow;
     if (ps.paren_level > 0) {
         /* TODO: explain what negative indentation means */
 	paren_indent = -ps.paren_indents[ps.paren_level - 1];
 	debug_println("paren_indent is now %d", paren_indent);
+    }
     not_first_line = 1;
+}
 int
 compute_code_indent(void)
+{
     int target_ind = opt.indent_size * ps.ind_level;
     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);
 	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
 	     * the test opt-lpl.0.
 	     */
 	    target_ind = paren_indent - 1;
 	else {
 	    int w;
 	    int t = paren_indent;
 	    if ((w = 1 + indentation_after(t - 1, s_code) - opt.max_line_length) > 0
 		&& 1 + indentation_after(target_ind, s_code) <= opt.max_line_length) {
 		t -= w + 1;
 		if (t > target_ind + 1)
 		    target_ind = t - 1;
 	    } else
 		target_ind = t - 1;
+	}
     } else if (ps.ind_stmt)
 	target_ind += opt.continuation_indent;
     return target_ind;
+}
 int
 compute_label_indent(void)
+{
     if (ps.pcase)
 	return (int) (case_ind * opt.indent_size);
     if (s_lab[0] == '#')
         return 0;
     return opt.indent_size * (ps.ind_level - label_offset);
+}
 /*
  * Copyright (C) 1976 by the Board of Trustees of the University of Illinois
+ *
  * All rights reserved
+ *
  * FUNCTION: Reads one block of input into the input buffer
  */
 void
 fill_buffer(void)
 {				/* this routine reads stuff from the input */
     char *p;
     int i;
     FILE *f = input;
     if (bp_save != NULL) {	/* there is a partly filled input buffer left */
 	buf_ptr = bp_save;	/* do not read anything, just switch buffers */
 	buf_end = be_save;
 	bp_save = be_save = NULL;
 	debug_println("switched buf_ptr back to bp_save");
 	if (buf_ptr < buf_end)
 	    return;		/* only return if there is really something in
 				 * this buffer */
+    }
     for (p = in_buffer;;) {
 	if (p >= in_buffer_limit) {
-	    int size = (in_buffer_limit - in_buffer) * 2 + 10;
+	    size_t size = (in_buffer_limit - in_buffer) * 2 + 10;
-	    int offset = p - in_buffer;
+	    size_t offset = p - in_buffer;
 	    in_buffer = realloc(in_buffer, size);
 	    if (in_buffer == NULL)
 		errx(1, "input line too long");
 	    p = in_buffer + offset;
 	    in_buffer_limit = in_buffer + size - 2;
+	}
 	if ((i = getc(f)) == EOF) {
 	    *p++ = ' ';
 	    *p++ = '\n';
 	    had_eof = true;
 	    break;
+	}
 	if (i != '\0')
 	    *p++ = i;
 	if (i == '\n')
 	    break;
+    }
     buf_ptr = in_buffer;
     buf_end = p;
     if (p - in_buffer > 2 && p[-2] == '/' && p[-3] == '*') {
 	if (in_buffer[3] == 'I' && strncmp(in_buffer, "/**INDENT**", 11) == 0)
 	    fill_buffer();	/* flush indent error message */
 	else {
 	    int com = 0;
 	    p = in_buffer;
 	    while (*p == ' ' || *p == '\t')
 		p++;
 	    if (*p == '/' && p[1] == '*') {
 		p += 2;
 		while (*p == ' ' || *p == '\t')
 		    p++;
 		if (p[0] == 'I' && p[1] == 'N' && p[2] == 'D' && p[3] == 'E'
 			&& p[4] == 'N' && p[5] == 'T') {
 		    p += 6;
 		    while (*p == ' ' || *p == '\t')
 			p++;
 		    if (*p == '*')
 			com = 1;
 		    else if (*p == 'O') {
 			if (*++p == 'N')
 			    p++, com = 1;
 			else if (*p == 'F' && *++p == 'F')
 			    p++, com = 2;
+		    }
 		    while (*p == ' ' || *p == '\t')
 			p++;
 		    if (p[0] == '*' && p[1] == '/' && p[2] == '\n' && com) {
 			if (s_com != e_com || s_lab != e_lab || s_code != e_code)
 			    dump_line();
 			if (!(inhibit_formatting = com - 1)) {
 			    n_real_blanklines = 0;
 			    postfix_blankline_requested = 0;
 			    prefix_blankline_requested = 0;
 			    suppress_blanklines = 1;
+			}
+		    }
+		}
+	    }
+	}
+    }
     if (inhibit_formatting) {
 	p = in_buffer;
 	do {
 	    output_char(*p);
 	} while (*p++ != '\n');
+    }
+}
 int
 indentation_after_range(int ind, const char *start, const char *end)
+{
     for (const char *p = start; *p != '\0' && p != end; ++p) {
 	if (*p == '\n' || *p == '\f')
 	    ind = 0;
 	else if (*p == '\t')
 	    ind = opt.tabsize * (ind / opt.tabsize + 1);
 	else if (*p == '\b')
 	    --ind;
 	else
 	    ++ind;
+    }
     return ind;
+}
 int
 indentation_after(int ind, const char *s)
+{
     return indentation_after_range(ind, s, NULL);
+}
 void
 diag(int level, const char *msg, ...)
+{
     va_list ap;
     const char *s, *e;
     if (level)
 	found_err = 1;
     if (output == stdout) {
 	s = "/**INDENT** ";
 	e = " */";
     } else {
 	s = e = "";
+    }
     va_start(ap, msg);
     fprintf(stderr, "%s%s@%d: ", s, level == 0 ? "Warning" : "Error", line_no);
     vfprintf(stderr, msg, ap);
     fprintf(stderr, "%s\n", e);
     va_end(ap);
+}

 @@ -1,720 +1,720 @@
-/*	$NetBSD: lexi.c,v 1.40 2021/03/13 11:27:01 rillig Exp $	*/
+/*	$NetBSD: lexi.c,v 1.41 2021/03/14 00:22:16 rillig Exp $	*/
 /*-
  * SPDX-License-Identifier: BSD-4-Clause
+ *
  * Copyright (c) 1985 Sun Microsystems, Inc.
  * Copyright (c) 1980, 1993
  *	The Regents of the University of California.  All rights reserved.
  * All rights reserved.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. All advertising materials mentioning features or use of this software
  *    must display the following acknowledgement:
  *	This product includes software developed by the University of
  *	California, Berkeley and its contributors.
  * 4. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
 #if 0
 #ifndef lint
 static char sccsid[] = "@(#)lexi.c	8.1 (Berkeley) 6/6/93";
 #endif /* not lint */
 #endif
 #include <sys/cdefs.h>
 #ifndef lint
 #if defined(__NetBSD__)
-__RCSID("$NetBSD: lexi.c,v 1.40 2021/03/13 11:27:01 rillig Exp $");
+__RCSID("$NetBSD: lexi.c,v 1.41 2021/03/14 00:22:16 rillig Exp $");
 #elif defined(__FreeBSD__)
 __FBSDID("$FreeBSD: head/usr.bin/indent/lexi.c 337862 2018-08-15 18:19:45Z pstef $");
 #endif
 #endif
 /*
  * Here we have the token scanner for indent.  It scans off one token and puts
  * it in the global variable "token".  It returns a code, indicating the type
  * of token scanned.
  */
 #include <assert.h>
 #include <err.h>
 #include <stdio.h>
 #include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 #include <sys/param.h>
 #include "indent.h"
 struct templ {
     const char *rwd;
     enum rwcode rwcode;
 };
 /*
  * This table has to be sorted alphabetically, because it'll be used in binary
  * search.
  */
 const struct templ specials[] =
+{
     {"_Bool", rw_type},
     {"_Complex", rw_type},
     {"_Imaginary", rw_type},
     {"auto", rw_storage_class},
     {"bool", rw_type},
     {"break", rw_jump},
     {"case", rw_case_or_default},
     {"char", rw_type},
     {"complex", rw_type},
     {"const", rw_type},
     {"continue", rw_jump},
     {"default", rw_case_or_default},
     {"do", rw_do_or_else},
     {"double", rw_type},
     {"else", rw_do_or_else},
     {"enum", rw_struct_or_union_or_enum},
     {"extern", rw_storage_class},
     {"float", rw_type},
     {"for", rw_for_or_if_or_while},
     {"global", rw_type},
     {"goto", rw_jump},
     {"if", rw_for_or_if_or_while},
     {"imaginary", rw_type},
     {"inline", rw_inline_or_restrict},
     {"int", rw_type},
     {"long", rw_type},
     {"offsetof", rw_offsetof},
     {"register", rw_storage_class},
     {"restrict", rw_inline_or_restrict},
     {"return", rw_jump},
     {"short", rw_type},
     {"signed", rw_type},
     {"sizeof", rw_sizeof},
     {"static", rw_storage_class},
     {"struct", rw_struct_or_union_or_enum},
     {"switch", rw_switch},
     {"typedef", rw_typedef},
     {"union", rw_struct_or_union_or_enum},
     {"unsigned", rw_type},
     {"void", rw_type},
     {"volatile", rw_type},
     {"while", rw_for_or_if_or_while}
 };
 const char **typenames;
 int         typename_count;
 int         typename_top = -1;
 /*
  * The transition table below was rewritten by hand from lx's output, given
  * the following definitions. lx is Katherine Flavel's lexer generator.
+ *
  * O  = /[0-7]/;        D  = /[0-9]/;          NZ = /[1-9]/;
  * H  = /[a-f0-9]/i;    B  = /[0-1]/;          HP = /0x/i;
  * BP = /0b/i;          E  = /e[+\-]?/i D+;    P  = /p[+\-]?/i D+;
  * FS = /[fl]/i;        IS = /u/i /(l|L|ll|LL)/? | /(l|L|ll|LL)/ /u/i?;
+ *
  * D+           E  FS? -> $float;
  * D*    "." D+ E? FS? -> $float;
  * D+    "."    E? FS? -> $float;    HP H+           IS? -> $int;
  * HP H+        P  FS? -> $float;    NZ D*           IS? -> $int;
  * HP H* "." H+ P  FS? -> $float;    "0" O*          IS? -> $int;
  * HP H+ "."    P  FS  -> $float;    BP B+           IS? -> $int;
  */
 static char const *table[] = {
     /*                examples:
              s                      0xx
              t                    00xaa
              a     11       101100xxa..
              r   11ee0001101lbuuxx.a.pp
              t.01.e+008bLuxll0Ll.aa.p+0
     states:  ABCDEFGHIJKLMNOPQRSTUVWXYZ */
     ['0'] = "CEIDEHHHIJQ  U  Q  VUVVZZZ",
     ['1'] = "DEIDEHHHIJQ  U  Q  VUVVZZZ",
     ['7'] = "DEIDEHHHIJ   U     VUVVZZZ",
     ['9'] = "DEJDEHHHJJ   U     VUVVZZZ",
     ['a'] = "             U     VUVV   ",
     ['b'] = "  K          U     VUVV   ",
     ['e'] = "  FFF   FF   U     VUVV   ",
     ['f'] = "    f  f     U     VUVV  f",
     ['u'] = "  MM    M  i  iiM   M     ",
     ['x'] = "  N                       ",
     ['p'] = "                    FFX   ",
     ['L'] = "  LLf  fL  PR   Li  L    f",
     ['l'] = "  OOf  fO   S P O i O    f",
     ['+'] = "     G                 Y  ",
     ['.'] = "B EE    EE   T      W     ",
     /*       ABCDEFGHIJKLMNOPQRSTUVWXYZ */
     [0]   = "uuiifuufiuuiiuiiiiiuiuuuuu",
 };
 /* Initialize constant transition table */
 void
 init_constant_tt(void)
+{
     table['-'] = table['+'];
     table['8'] = table['9'];
     table['2'] = table['3'] = table['4'] = table['5'] = table['6'] = table['7'];
     table['A'] = table['C'] = table['D'] = table['c'] = table['d'] = table['a'];
     table['B'] = table['b'];
     table['E'] = table['e'];
     table['U'] = table['u'];
     table['X'] = table['x'];
     table['P'] = table['p'];
     table['F'] = table['f'];
+}
 static char
 inbuf_peek(void)
+{
     return *buf_ptr;
+}
 static void
 inbuf_skip(void)
+{
     buf_ptr++;
     if (buf_ptr >= buf_end)
 	fill_buffer();
+}
 static char
 inbuf_next(void)
+{
     char ch = inbuf_peek();
     inbuf_skip();
     return ch;
+}
 static void
 check_size_token(size_t desired_size)
+{
     if (e_token + (desired_size) < l_token)
         return;
     size_t nsize = l_token - s_token + 400 + desired_size;
     size_t token_len = e_token - s_token;
     tokenbuf = realloc(tokenbuf, nsize);
     if (tokenbuf == NULL)
 	err(1, NULL);
     e_token = tokenbuf + token_len + 1;
     l_token = tokenbuf + nsize - 5;
     s_token = tokenbuf + 1;
+}
 static int
 compare_templ_array(const void *key, const void *elem)
+{
     return strcmp(key, ((const struct templ *)elem)->rwd);
+}
 static int
 compare_string_array(const void *key, const void *elem)
+{
     return strcmp(key, *((const char *const *)elem));
+}
 #ifdef debug
 const char *
 token_type_name(token_type tk)
+{
     static const char *const name[] = {
 	"end_of_file", "newline", "lparen", "rparen", "unary_op",
 	"binary_op", "postfix_op", "question", "case_label", "colon",
 	"semicolon", "lbrace", "rbrace", "ident", "comma",
 	"comment", "switch_expr", "preprocessing", "form_feed", "decl",
 	"keyword_for_if_while", "keyword_do_else",
 	"if_expr", "while_expr", "for_exprs",
 	"stmt", "stmt_list", "keyword_else", "keyword_do", "do_stmt",
 	"if_expr_stmt", "if_expr_stmt_else", "period", "string_prefix",
 	"storage_class", "funcname", "type_def", "keyword_struct_union_enum"
     };
     assert(0 <= tk && tk < sizeof name / sizeof name[0]);
     return name[tk];
+}
 static void
 print_buf(const char *name, const char *s, const char *e)
+{
     if (s < e) {
 	debug_printf(" %s ", name);
 	debug_vis_range("\"", s, e, "\"");
+    }
+}
 static token_type
 lexi_end(token_type code)
+{
     debug_printf("in line %d, lexi returns '%s'",
 	line_no, token_type_name(code));
     print_buf("token", s_token, e_token);
     print_buf("label", s_lab, e_lab);
     print_buf("code", s_code, e_code);
     print_buf("comment", s_com, e_com);
     debug_printf("\n");
     return code;
+}
 #else
 #  define lexi_end(tk) (tk)
 #endif
 token_type
 lexi(struct parser_state *state)
+{
     int         unary_delim;	/* this is set to 1 if the current token
 				 * forces a following operator to be unary */
     token_type  code;		/* internal code to be returned */
     char        qchar;		/* the delimiter character for a string */
     e_token = s_token;		/* point to start of place to save token */
     unary_delim = false;
     state->col_1 = state->last_nl;	/* tell world that this token started
 					 * in column 1 iff the last thing
 					 * scanned was a newline */
     state->last_nl = false;
     while (*buf_ptr == ' ' || *buf_ptr == '\t') {	/* get rid of blanks */
 	state->col_1 = false;	/* leading blanks imply token is not in column
 				 * 1 */
 	inbuf_skip();
+    }
     /* Scan an alphanumeric token */
     if (isalnum((unsigned char)*buf_ptr) ||
 	*buf_ptr == '_' || *buf_ptr == '$' ||
 	(buf_ptr[0] == '.' && isdigit((unsigned char)buf_ptr[1]))) {
 	/*
 	 * we have a character or number
 	 */
 	struct templ *p;
 	if (isdigit((unsigned char)*buf_ptr) ||
 	    (buf_ptr[0] == '.' && isdigit((unsigned char)buf_ptr[1]))) {
 	    char s;
 	    unsigned char i;
 	    for (s = 'A'; s != 'f' && s != 'i' && s != 'u'; ) {
 		i = (unsigned char)*buf_ptr;
 		if (i >= nitems(table) || table[i] == NULL ||
 		    table[i][s - 'A'] == ' ') {
 		    s = table[0][s - 'A'];
 		    break;
+		}
 		s = table[i][s - 'A'];
 		check_size_token(1);
 		*e_token++ = inbuf_next();
+	    }
 	    /* s now indicates the type: f(loating), i(integer), u(nknown) */
 	} else {
 	    while (isalnum((unsigned char)*buf_ptr) ||
 		   *buf_ptr == '\\' ||
 		   *buf_ptr == '_' || *buf_ptr == '$') {
 		/* fill_buffer() terminates buffer with newline */
 		if (*buf_ptr == '\\') {
 		    if (buf_ptr[1] == '\n') {
 			buf_ptr += 2;
 			if (buf_ptr >= buf_end)
 			    fill_buffer();
 		    } else
 			break;
+		}
 		check_size_token(1);
 		*e_token++ = inbuf_next();
+	    }
+	}
 	*e_token = '\0';
 	if (s_token[0] == 'L' && s_token[1] == '\0' &&
 	      (*buf_ptr == '"' || *buf_ptr == '\''))
 	    return lexi_end(string_prefix);
 	while (*buf_ptr == ' ' || *buf_ptr == '\t')	/* get rid of blanks */
 	    inbuf_next();
 	state->keyword = rw_0;
 	if (state->last_token == keyword_struct_union_enum &&
 	    !state->p_l_follow) {
 	    /* if last token was 'struct' and we're not in parentheses, then
 	     * this token should be treated as a declaration */
 	    state->last_u_d = true;
 	    return lexi_end(decl);
+	}
 	/*
 	 * Operator after identifier is binary unless last token was 'struct'
 	 */
 	state->last_u_d = (state->last_token == keyword_struct_union_enum);
 	p = bsearch(s_token, specials, sizeof specials / sizeof specials[0],
 	    sizeof specials[0], compare_templ_array);
 	if (p == NULL) {	/* not a special keyword... */
 	    char *u;
 	    /* ... so maybe a type_t or a typedef */
 	    if ((opt.auto_typedefs && ((u = strrchr(s_token, '_')) != NULL) &&
 	        strcmp(u, "_t") == 0) || (typename_top >= 0 &&
 		  bsearch(s_token, typenames, typename_top + 1,
 		    sizeof typenames[0], compare_string_array))) {
 		state->keyword = rw_type;
 		state->last_u_d = true;
 	        goto found_typename;
+	    }
 	} else {			/* we have a keyword */
 	    state->keyword = p->rwcode;
 	    state->last_u_d = true;
 	    switch (p->rwcode) {
 	    case rw_switch:
 		return lexi_end(switch_expr);
 	    case rw_case_or_default:
 		return lexi_end(case_label);
 	    case rw_struct_or_union_or_enum:
 	    case rw_type:
 	    found_typename:
 		if (state->p_l_follow) {
 		    /* inside parens: cast, param list, offsetof or sizeof */
 		    state->cast_mask |= (1 << state->p_l_follow) & ~state->not_cast_mask;
+		}
 		if (state->last_token == period || state->last_token == unary_op) {
 		    state->keyword = rw_0;
 		    break;
+		}
 		if (p != NULL && p->rwcode == rw_struct_or_union_or_enum)
 		    return lexi_end(keyword_struct_union_enum);
 		if (state->p_l_follow)
 		    break;
 		return lexi_end(decl);
 	    case rw_for_or_if_or_while:
 		return lexi_end(keyword_for_if_while);
 	    case rw_do_or_else:
 		return lexi_end(keyword_do_else);
 	    case rw_storage_class:
 		return lexi_end(storage_class);
 	    case rw_typedef:
 		return lexi_end(type_def);
 	    default:		/* all others are treated like any other
 				 * identifier */
 		return lexi_end(ident);
 	    }			/* end of switch */
 	}			/* end of if (found_it) */
 	if (*buf_ptr == '(' && state->tos <= 1 && state->ind_level == 0 &&
 	    state->in_parameter_declaration == 0 && state->block_init == 0) {
 	    char *tp = buf_ptr;
 	    while (tp < buf_end)
 		if (*tp++ == ')' && (*tp == ';' || *tp == ','))
 		    goto not_proc;
 	    strncpy(state->procname, token, sizeof state->procname - 1);
 	    if (state->in_decl)
 		state->in_parameter_declaration = 1;
 	    return lexi_end(funcname);
     not_proc:;
+	}
 	/*
 	 * The following hack attempts to guess whether or not the current
 	 * token is in fact a declaration keyword -- one that has been
 	 * typedefd
 	 */
 	else if (!state->p_l_follow && !state->block_init &&
 	    !state->in_stmt &&
 	    ((*buf_ptr == '*' && buf_ptr[1] != '=') ||
 		isalpha((unsigned char)*buf_ptr)) &&
 	    (state->last_token == semicolon || state->last_token == lbrace ||
 		state->last_token == rbrace)) {
 	    state->keyword = rw_type;
 	    state->last_u_d = true;
 	    return lexi_end(decl);
+	}
 	if (state->last_token == decl)	/* if this is a declared variable,
 					 * then following sign is unary */
 	    state->last_u_d = true;	/* will make "int a -1" work */
 	return lexi_end(ident);		/* the ident is not in the list */
     }				/* end of procesing for alpanum character */
     /* Scan a non-alphanumeric token */
     check_size_token(3);	/* things like "<<=" */
     *e_token++ = inbuf_next();	/* if it is only a one-character token, it is
 				 * moved here */
     *e_token = '\0';
     switch (*token) {
     case '\n':
 	unary_delim = state->last_u_d;
 	state->last_nl = true;	/* remember that we just had a newline */
 	code = (had_eof ? end_of_file : newline);
 	/*
 	 * if data has been exhausted, the newline is a dummy, and we should
 	 * return code to stop
 	 */
 	break;
     case '\'':			/* start of quoted character */
     case '"':			/* start of string */
 	qchar = *token;
 	do {			/* copy the string */
-	    while (1) {		/* move one character or [/<char>]<char> */
+	    for (;;) {		/* move one character or [/<char>]<char> */
 		if (*buf_ptr == '\n') {
 		    diag(1, "Unterminated literal");
 		    goto stop_lit;
+		}
 		check_size_token(2);
 		*e_token = inbuf_next();
 		if (*e_token == '\\') {		/* if escape, copy extra char */
 		    if (*buf_ptr == '\n')	/* check for escaped newline */
 			++line_no;
 		    *++e_token = inbuf_next();
 		    ++e_token;	/* we must increment this again because we
 				 * copied two chars */
 		} else
 		    break;	/* we copied one character */
 	    }			/* end of while (1) */
 	} while (*e_token++ != qchar);
 stop_lit:
 	code = ident;
 	break;
     case '(':
     case '[':
 	unary_delim = true;
 	code = lparen;
 	break;
     case ')':
     case ']':
 	code = rparen;
 	break;
     case '#':
 	unary_delim = state->last_u_d;
 	code = preprocessing;
 	break;
     case '?':
 	unary_delim = true;
 	code = question;
 	break;
     case ':':
 	code = colon;
 	unary_delim = true;
 	break;
     case ';':
 	unary_delim = true;
 	code = semicolon;
 	break;
     case '{':
 	unary_delim = true;
 	/*
 	 * if (state->in_or_st) state->block_init = 1;
 	 */
 	/* ?	code = state->block_init ? lparen : lbrace; */
 	code = lbrace;
 	break;
     case '}':
 	unary_delim = true;
 	/* ?	code = state->block_init ? rparen : rbrace; */
 	code = rbrace;
 	break;
     case 014:			/* a form feed */
 	unary_delim = state->last_u_d;
 	state->last_nl = true;	/* remember this so we can set 'state->col_1'
 				 * right */
 	code = form_feed;
 	break;
     case ',':
 	unary_delim = true;
 	code = comma;
 	break;
     case '.':
 	unary_delim = false;
 	code = period;
 	break;
     case '-':
     case '+':			/* check for -, +, --, ++ */
 	code = (state->last_u_d ? unary_op : binary_op);
 	unary_delim = true;
 	if (*buf_ptr == token[0]) {
 	    /* check for doubled character */
 	    *e_token++ = *buf_ptr++;
 	    /* buffer overflow will be checked at end of loop */
 	    if (state->last_token == ident || state->last_token == rparen) {
 		code = (state->last_u_d ? unary_op : postfix_op);
 		/* check for following ++ or -- */
 		unary_delim = false;
+	    }
 	} else if (*buf_ptr == '=')
 	    /* check for operator += */
 	    *e_token++ = *buf_ptr++;
 	else if (*buf_ptr == '>') {
 	    /* check for operator -> */
 	    *e_token++ = *buf_ptr++;
 	    unary_delim = false;
 	    code = unary_op;
 	    state->want_blank = false;
+	}
 	break;			/* buffer overflow will be checked at end of
 				 * switch */
     case '=':
 	if (state->in_or_st)
 	    state->block_init = 1;
 	if (*buf_ptr == '=') {	/* == */
 	    *e_token++ = '=';	/* Flip =+ to += */
 	    buf_ptr++;
 	    *e_token = 0;
+	}
 	code = binary_op;
 	unary_delim = true;
 	break;
 	/* can drop thru!!! */
     case '>':
     case '<':
     case '!':			/* ops like <, <<, <=, !=, etc */
 	if (*buf_ptr == '>' || *buf_ptr == '<' || *buf_ptr == '=')
 	    *e_token++ = inbuf_next();
 	if (*buf_ptr == '=')
 	    *e_token++ = *buf_ptr++;
 	code = (state->last_u_d ? unary_op : binary_op);
 	unary_delim = true;
 	break;
     case '*':
 	unary_delim = true;
 	if (!state->last_u_d) {
 	    if (*buf_ptr == '=')
 		*e_token++ = *buf_ptr++;
 	    code = binary_op;
 	    break;
+	}
 	while (*buf_ptr == '*' || isspace((unsigned char)*buf_ptr)) {
 	    if (*buf_ptr == '*') {
 		check_size_token(1);
 		*e_token++ = *buf_ptr;
+	    }
 	    inbuf_skip();
+	}
 	if (ps.in_decl) {
 	    char *tp = buf_ptr;
 	    while (isalpha((unsigned char)*tp) ||
 		   isspace((unsigned char)*tp)) {
 		if (++tp >= buf_end)
 		    fill_buffer();
+	    }
 	    if (*tp == '(')
 		ps.procname[0] = ' ';
+	}
 	code = unary_op;
 	break;
     default:
 	if (token[0] == '/' && (*buf_ptr == '*' || *buf_ptr == '/')) {
 	    /* it is start of comment */
 	    *e_token++ = inbuf_next();
 	    code = comment;
 	    unary_delim = state->last_u_d;
 	    break;
+	}
 	while (e_token[-1] == *buf_ptr || *buf_ptr == '=') {
 	    /*
 	     * handle ||, &&, etc, and also things as in int *****i
 	     */
 	    check_size_token(1);
 	    *e_token++ = inbuf_next();
+	}
 	code = (state->last_u_d ? unary_op : binary_op);
 	unary_delim = true;
     }				/* end of switch */
     if (buf_ptr >= buf_end)	/* check for input buffer empty */
 	fill_buffer();
     state->last_u_d = unary_delim;
     check_size_token(1);
     *e_token = '\0';		/* null terminate the token */
     return lexi_end(code);
+}
 void
 alloc_typenames(void)
+{
     typenames = malloc(sizeof(typenames[0]) * (typename_count = 16));
     if (typenames == NULL)
 	err(1, NULL);
+}
 void
 add_typename(const char *key)
+{
     int comparison;
     const char *copy;
     if (typename_top + 1 >= typename_count) {
 	typenames = realloc((void *)typenames,
 	    sizeof(typenames[0]) * (typename_count *= 2));
 	if (typenames == NULL)
 	    err(1, NULL);
+    }
     if (typename_top == -1)
 	typenames[++typename_top] = copy = strdup(key);
     else if ((comparison = strcmp(key, typenames[typename_top])) >= 0) {
 	/* take advantage of sorted input */
 	if (comparison == 0)	/* remove duplicates */
 	    return;
 	typenames[++typename_top] = copy = strdup(key);
     } else {
 	int p;
 	for (p = 0; (comparison = strcmp(key, typenames[p])) > 0; p++)
 	    /* find place for the new key */;
 	if (comparison == 0)	/* remove duplicates */
 	    return;
 	memmove(&typenames[p + 1], &typenames[p],
 	    sizeof(typenames[0]) * (++typename_top - p));
 	typenames[p] = copy = strdup(key);
+    }
     if (copy == NULL)
 	err(1, NULL);
+}

 @@ -1,378 +1,378 @@
-/*	$NetBSD: pr_comment.c,v 1.30 2021/03/13 18:11:31 rillig Exp $	*/
+/*	$NetBSD: pr_comment.c,v 1.31 2021/03/14 00:22:16 rillig Exp $	*/
 /*-
  * SPDX-License-Identifier: BSD-4-Clause
+ *
  * Copyright (c) 1985 Sun Microsystems, Inc.
  * Copyright (c) 1980, 1993
  *	The Regents of the University of California.  All rights reserved.
  * All rights reserved.
+ *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
  * 3. All advertising materials mentioning features or use of this software
  *    must display the following acknowledgement:
  *	This product includes software developed by the University of
  *	California, Berkeley and its contributors.
  * 4. Neither the name of the University nor the names of its contributors
  *    may be used to endorse or promote products derived from this software
  *    without specific prior written permission.
+ *
  * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
  * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
  * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
  * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  */
 #if 0
 #ifndef lint
 static char sccsid[] = "@(#)pr_comment.c	8.1 (Berkeley) 6/6/93";
 #endif /* not lint */
 #endif
 #include <sys/cdefs.h>
 #ifndef lint
 #if defined(__NetBSD__)
-__RCSID("$NetBSD: pr_comment.c,v 1.30 2021/03/13 18:11:31 rillig Exp $");
+__RCSID("$NetBSD: pr_comment.c,v 1.31 2021/03/14 00:22:16 rillig Exp $");
 #elif defined(__FreeBSD__)
 __FBSDID("$FreeBSD: head/usr.bin/indent/pr_comment.c 334927 2018-06-10 16:44:18Z pstef $");
 #endif
 #endif
 #include <err.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include "indent.h"
 static void
 check_size_comment(size_t desired_size, char **last_bl_ptr)
+{
     if (e_com + (desired_size) < l_com)
         return;
     size_t nsize = l_com - s_com + 400 + desired_size;
     size_t com_len = e_com - s_com;
     ssize_t blank_pos = *last_bl_ptr != NULL ? *last_bl_ptr - combuf : -1;
     combuf = realloc(combuf, nsize);
     if (combuf == NULL)
 	err(1, NULL);
     e_com = combuf + com_len + 1;
     if (blank_pos > 0)
 	*last_bl_ptr = combuf + blank_pos;
     l_com = combuf + nsize - 5;
     s_com = combuf + 1;
+}
 /*
  * Scan, reformat and output a single comment, which is either a block comment
  * starting with '/' '*' or an end-of-line comment starting with '//'.
+ *
  * Try to keep comments from going over the maximum line length.  If a line is
  * too long, move everything starting from the last blank to the next comment
  * line.  Blanks and tabs from the beginning of the input line are removed.
+ *
  * ALGORITHM:
  *	1) Decide where the comment should be aligned, and if lines should
  *	   be broken.
  *	2) If lines should not be broken and filled, just copy up to end of
  *	   comment.
  *	3) If lines should be filled, then scan through the input buffer,
  *	   copying characters to com_buf.  Remember where the last blank,
  *	   tab, or newline was.  When line is filled, print up to last blank
  *	   and continue copying.
  */
 void
 process_comment(void)
+{
     int         adj_max_line_length; /* Adjusted max_line_length for comments
 				 * that spill over the right margin */
     char       *last_bl;	/* points to the last blank in the output
 				 * buffer */
     char       *t_ptr;		/* used for moving string */
     int         break_delim = opt.comment_delimiter_on_blankline;
     int         l_just_saw_decl = ps.just_saw_decl;
     adj_max_line_length = opt.max_line_length;
     ps.just_saw_decl = 0;
     last_bl = NULL;		/* no blanks found so far */
     ps.box_com = false;		/* at first, assume that we are not in
 				 * a boxed comment or some other
 				 * comment that should not be touched */
     ++ps.out_coms;		/* keep track of number of comments */
     /* Figure where to align and how to treat the comment */
     if (ps.col_1 && !opt.format_col1_comments) { /* if the comment starts in
 				 * column 1, it should not be touched */
 	ps.box_com = true;
 	break_delim = false;
 	ps.com_col = 1;
     } else {
 	if (*buf_ptr == '-' || *buf_ptr == '*' || e_token[-1] == '/' ||
 	    (*buf_ptr == '\n' && !opt.format_block_comments)) {
 	    ps.box_com = true;	/* A comment with a '-' or '*' immediately
 				 * after the /+* is assumed to be a boxed
 				 * comment. A comment with a newline
 				 * immediately after the /+* is assumed to
 				 * be a block comment and is treated as a
 				 * box comment unless format_block_comments
 				 * is nonzero (the default). */
 	    break_delim = false;
+	}
 	if ( /* ps.bl_line && */ s_lab == e_lab && s_code == e_code) {
 	    /* klg: check only if this line is blank */
 	    /*
 	     * If this (*and previous lines are*) blank, dont put comment way
 	     * out at left
 	     */
 	    ps.com_col = (ps.ind_level - opt.unindent_displace) * opt.indent_size + 1;
 	    adj_max_line_length = opt.block_comment_max_line_length;
 	    if (ps.com_col <= 1)
 		ps.com_col = 1 + !opt.format_col1_comments;
 	} else {
 	    int target_col;
 	    break_delim = false;
 	    if (s_code != e_code)
 		target_col = 1 + indentation_after(compute_code_indent(), s_code);
 	    else {
 		target_col = 1;
 		if (s_lab != e_lab)
 		    target_col = 1 + indentation_after(compute_label_indent(), s_lab);
+	    }
 	    ps.com_col = ps.decl_on_line || ps.ind_level == 0
 		? opt.decl_comment_column : opt.comment_column;
 	    if (ps.com_col <= target_col)
 		ps.com_col = opt.tabsize * (1 + (target_col - 1) / opt.tabsize) + 1;
 	    if (ps.com_col + 24 > adj_max_line_length)
 	        /* XXX: mismatch between column and length */
 		adj_max_line_length = ps.com_col + 24;
+	}
+    }
     if (ps.box_com) {
 	/*
 	 * Find out how much indentation there was originally, because that
 	 * much will have to be ignored by pad_output() in dump_line(). This
 	 * is a box comment, so nothing changes -- not even indentation.
+	 *
 	 * The comment we're about to read usually comes from in_buffer,
 	 * unless it has been copied into save_com.
 	 */
 	char *start;
 	/*
 	 * XXX: ordered comparison between pointers from different objects
 	 * invokes undefined behavior (C99 6.5.8).
 	 */
 	start = buf_ptr >= save_com && buf_ptr < save_com + sc_size ?
 	    sc_buf : in_buffer;
 	ps.n_comment_delta = -indentation_after_range(0, start, buf_ptr - 2);
     } else {
 	ps.n_comment_delta = 0;
 	while (*buf_ptr == ' ' || *buf_ptr == '\t')
 	    buf_ptr++;
+    }
     ps.comment_delta = 0;
     *e_com++ = '/';
     *e_com++ = e_token[-1];
     if (*buf_ptr != ' ' && !ps.box_com)
 	*e_com++ = ' ';
     /*
      * Don't put a break delimiter if this is a one-liner that won't wrap.
      */
     if (break_delim)
 	for (t_ptr = buf_ptr; *t_ptr != '\0' && *t_ptr != '\n'; t_ptr++) {
 	    if (t_ptr >= buf_end)
 		fill_buffer();
 	    if (t_ptr[0] == '*' && t_ptr[1] == '/') {
 		int right_margin = indentation_after_range(ps.com_col - 1,
 		    buf_ptr, t_ptr + 2);
 		if (right_margin < adj_max_line_length)
 		    break_delim = false;
 		break;
+	    }
+	}
     if (break_delim) {
 	char       *t = e_com;
 	e_com = s_com + 2;
 	*e_com = 0;
 	if (opt.blanklines_before_blockcomments && ps.last_token != lbrace)
 	    prefix_blankline_requested = 1;
 	dump_line();
 	e_com = s_com = t;
 	if (!ps.box_com && opt.star_comment_cont)
 	    *e_com++ = ' ', *e_com++ = '*', *e_com++ = ' ';
+    }
     /* Start to copy the comment */
-    while (1) {			/* this loop will go until the comment is
+    for (;;) {			/* this loop will go until the comment is
 				 * copied */
 	switch (*buf_ptr) {	/* this checks for various special cases */
 	case 014:		/* check for a form feed */
 	    check_size_comment(3, &last_bl);
 	    if (!ps.box_com) {	/* in a text comment, break the line here */
 		ps.use_ff = true;
 		/* fix so dump_line uses a form feed */
 		dump_line();
 		last_bl = NULL;
 		if (!ps.box_com && opt.star_comment_cont)
 		    *e_com++ = ' ', *e_com++ = '*', *e_com++ = ' ';
 		while (*++buf_ptr == ' ' || *buf_ptr == '\t')
+		    ;
 	    } else {
 		if (++buf_ptr >= buf_end)
 		    fill_buffer();
 		*e_com++ = 014;
+	    }
 	    break;
 	case '\n':
 	    if (e_token[-1] == '/') {
 		++line_no;
 		goto end_of_comment;
+	    }
 	    if (had_eof) {	/* check for unexpected eof */
 		printf("Unterminated comment\n");
 		dump_line();
 		return;
+	    }
 	    last_bl = NULL;
 	    check_size_comment(4, &last_bl);
 	    if (ps.box_com || ps.last_nl) {	/* if this is a boxed comment,
 						 * we dont ignore the newline */
 		if (s_com == e_com)
 		    *e_com++ = ' ';
 		if (!ps.box_com && e_com - s_com > 3) {
 		    dump_line();
 		    if (opt.star_comment_cont)
 			*e_com++ = ' ', *e_com++ = '*', *e_com++ = ' ';
+		}
 		dump_line();
 		if (!ps.box_com && opt.star_comment_cont)
 		    *e_com++ = ' ', *e_com++ = '*', *e_com++ = ' ';
 	    } else {
 		ps.last_nl = 1;
 		if (e_com[-1] == ' ' || e_com[-1] == '\t')
 		    last_bl = e_com - 1;
 		/*
 		 * if there was a space at the end of the last line, remember
 		 * where it was
 		 */
 		else {		/* otherwise, insert one */
 		    last_bl = e_com;
 		    *e_com++ = ' ';
+		}
+	    }
 	    ++line_no;		/* keep track of input line number */
 	    if (!ps.box_com) {
 		int         nstar = 1;
 		do {		/* flush any blanks and/or tabs at start of
 				 * next line */
 		    if (++buf_ptr >= buf_end)
 			fill_buffer();
 		    if (*buf_ptr == '*' && --nstar >= 0) {
 			if (++buf_ptr >= buf_end)
 			    fill_buffer();
 			if (*buf_ptr == '/')
 			    goto end_of_comment;
+		    }
 		} while (*buf_ptr == ' ' || *buf_ptr == '\t');
 	    } else if (++buf_ptr >= buf_end)
 		fill_buffer();
 	    break;		/* end of case for newline */
 	case '*':		/* must check for possibility of being at end
 				 * of comment */
 	    if (++buf_ptr >= buf_end)	/* get to next char after * */
 		fill_buffer();
 	    check_size_comment(4, &last_bl);
 	    if (*buf_ptr == '/') {	/* it is the end!!! */
 	end_of_comment:
 		if (++buf_ptr >= buf_end)
 		    fill_buffer();
 		if (break_delim) {
 		    if (e_com > s_com + 3)
 			dump_line();
 		    else
 			s_com = e_com;
 		    *e_com++ = ' ';
+		}
 		if (e_com[-1] != ' ' && e_com[-1] != '\t' && !ps.box_com)
 		    *e_com++ = ' ';	/* ensure blank before end */
 		if (e_token[-1] == '/')
 		    *e_com++ = '\n', *e_com = '\0';
 		else
 		    *e_com++ = '*', *e_com++ = '/', *e_com = '\0';
 		ps.just_saw_decl = l_just_saw_decl;
 		return;
 	    } else		/* handle isolated '*' */
 		*e_com++ = '*';
 	    break;
 	default:		/* we have a random char */
+	    ;
 	    int now_len = indentation_after_range(ps.com_col - 1, s_com, e_com);
 	    do {
 		check_size_comment(1, &last_bl);
 		*e_com = *buf_ptr++;
 		if (buf_ptr >= buf_end)
 		    fill_buffer();
 		if (*e_com == ' ' || *e_com == '\t')
 		    last_bl = e_com;	/* remember we saw a blank */
 		++e_com;
 		now_len++;
 	    } while (!memchr("*\n\r\b\t", *buf_ptr, 6) &&
 		(now_len < adj_max_line_length || !last_bl));
 	    ps.last_nl = false;
 	    /* XXX: signed character comparison '>' does not work for UTF-8 */
 	    if (now_len >= adj_max_line_length &&
 		    !ps.box_com && e_com[-1] > ' ') {
 		/*
 		 * the comment is too long, it must be broken up
 		 */
 		if (last_bl == NULL) {
 		    dump_line();
 		    if (!ps.box_com && opt.star_comment_cont)
 			*e_com++ = ' ', *e_com++ = '*', *e_com++ = ' ';
 		    break;
+		}
 		*e_com = '\0';
 		e_com = last_bl;
 		dump_line();
 		if (!ps.box_com && opt.star_comment_cont)
 		    *e_com++ = ' ', *e_com++ = '*', *e_com++ = ' ';
 		for (t_ptr = last_bl + 1; *t_ptr == ' ' || *t_ptr == '\t';
 		    t_ptr++)
+			;
 		last_bl = NULL;
 		/*
 		 * t_ptr will be somewhere between e_com (dump_line() reset)
 		 * and l_com. So it's safe to copy byte by byte from t_ptr
 		 * to e_com without any check_size_comment().
 		 */
 		while (*t_ptr != '\0') {
 		    if (*t_ptr == ' ' || *t_ptr == '\t')
 			last_bl = e_com;
 		    *e_com++ = *t_ptr++;
+		}
+	    }
 	    break;
+	}
+    }
+}

 @@ -1,10 +1,13 @@
-#	$NetBSD: Makefile,v 1.10 2021/03/12 19:11:29 rillig Exp $
+#	$NetBSD: Makefile,v 1.11 2021/03/14 00:22:16 rillig Exp $
 #	from: @(#)Makefile	8.1 (Berkeley) 6/6/93
 PROG=	indent
 SRCS=	indent.c io.c lexi.c parse.c pr_comment.c args.c
 CPPFLAGS+=	${DEBUG:D-Ddebug}
-LINTFLAGS+=	-e
+LINTFLAGS+=	-e -w
 # bug in lint; see tests/usr.bin/lint/lint1/msg_168.c
 LINTFLAGS.lexi.c+=	-X 168
 .include <bsd.prog.mk>