Module Name:    src
Committed By:   rillig
Date:           Sat Jun 10 07:48:55 UTC 2023

Modified Files:
        src/usr.bin/indent: io.c

Log Message:
indent: clean up function names and order in output


To generate a diff of this commit:
cvs rdiff -u -r1.210 -r1.211 src/usr.bin/indent/io.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/usr.bin/indent/io.c
diff -u src/usr.bin/indent/io.c:1.210 src/usr.bin/indent/io.c:1.211
--- src/usr.bin/indent/io.c:1.210	Sat Jun 10 07:42:41 2023
+++ src/usr.bin/indent/io.c	Sat Jun 10 07:48:55 2023
@@ -1,4 +1,4 @@
-/*	$NetBSD: io.c,v 1.210 2023/06/10 07:42:41 rillig Exp $	*/
+/*	$NetBSD: io.c,v 1.211 2023/06/10 07:48:55 rillig Exp $	*/
 
 /*-
  * SPDX-License-Identifier: BSD-4-Clause
@@ -38,7 +38,7 @@
  */
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: io.c,v 1.210 2023/06/10 07:42:41 rillig Exp $");
+__RCSID("$NetBSD: io.c,v 1.211 2023/06/10 07:48:55 rillig Exp $");
 
 #include <stdio.h>
 
@@ -108,7 +108,7 @@ inp_next(void)
 
 
 static void
-output_newline(void)
+write_newline(void)
 {
 	buffered_blank_lines++;
 	wrote_newlines++;
@@ -125,7 +125,7 @@ write_buffered_blank_lines(void)
 }
 
 static void
-output_range(const char *s, size_t len)
+write_range(const char *s, size_t len)
 {
 	write_buffered_blank_lines();
 	fwrite(s, 1, len, output);
@@ -136,7 +136,7 @@ output_range(const char *s, size_t len)
 }
 
 static void
-output_indent(int new_ind)
+write_indent(int new_ind)
 {
 	write_buffered_blank_lines();
 
@@ -161,22 +161,6 @@ output_indent(int new_ind)
 	out_ind = ind;
 }
 
-void
-output_finish(void)
-{
-	if (lab.len > 0 || code.len > 0 || com.len > 0)
-		output_line();
-	if (indent_enabled == indent_on) {
-		if (buffered_blank_lines > 1)
-			buffered_blank_lines = 1;
-		write_buffered_blank_lines();
-	} else {
-		indent_enabled = indent_last_off_line;
-		output_line();
-	}
-	fflush(output);
-}
-
 static bool
 want_blank_line(void)
 {
@@ -236,8 +220,8 @@ compute_label_indent(void)
 static void
 output_line_label(void)
 {
-	output_indent(compute_label_indent());
-	output_range(lab.s, lab.len);
+	write_indent(compute_label_indent());
+	write_range(lab.s, lab.len);
 }
 
 static int
@@ -300,9 +284,9 @@ output_line_code(void)
 	}
 
 	if (lab.len > 0 && target_ind <= out_ind)
-		output_range(" ", 1);
-	output_indent(target_ind);
-	output_range(code.s, code.len);
+		write_range(" ", 1);
+	write_indent(target_ind);
+	write_range(code.s, code.len);
 }
 
 static void
@@ -327,13 +311,13 @@ output_line_comment(void)
 	}
 
 	if (out_ind > target_ind)
-		output_newline();
+		write_newline();
 
 	while (com.s + com.len > p && ch_isspace(com.s[com.len - 1]))
 		com.len--;
 
-	output_indent(target_ind);
-	output_range(p, com.len - (size_t)(p - com.s));
+	write_indent(target_ind);
+	write_range(p, com.len - (size_t)(p - com.s));
 
 	ps.comment_delta = ps.n_comment_delta;
 }
@@ -355,7 +339,7 @@ output_line(void)
 
 		if (want_blank_line() && wrote_newlines < 2
 		    && out.line_kind != lk_blank)
-			output_newline();
+			write_newline();
 
 		/* This kludge aligns function definitions correctly. */
 		if (ps.ind_level == 0)
@@ -379,13 +363,13 @@ output_line(void)
 		if (com.len > 0)
 			output_line_comment();
 
-		output_newline();
+		write_newline();
 		out.prev_line_kind = out.line_kind;
 	}
 
 	if (indent_enabled == indent_last_off_line) {
 		indent_enabled = indent_on;
-		output_range(out.indent_off_text.s, out.indent_off_text.len);
+		write_range(out.indent_off_text.s, out.indent_off_text.len);
 		out.indent_off_text.len = 0;
 	}
 
@@ -416,3 +400,19 @@ prepare_next_line:
 
 	out.line_kind = lk_other;
 }
+
+void
+output_finish(void)
+{
+	if (lab.len > 0 || code.len > 0 || com.len > 0)
+		output_line();
+	if (indent_enabled == indent_on) {
+		if (buffered_blank_lines > 1)
+			buffered_blank_lines = 1;
+		write_buffered_blank_lines();
+	} else {
+		indent_enabled = indent_last_off_line;
+		output_line();
+	}
+	fflush(output);
+}

Reply via email to