Module Name:    src
Committed By:   rillig
Date:           Fri Apr  2 16:38:08 UTC 2021

Modified Files:
        src/usr.bin/xlint/lint1: tree.c

Log Message:
lint: rename local variable in build_colon

The name 'rtp' was confusing since it did not correspond to 'rn'.  The
'r' meant 'result', not 'right-hand side'.

No functional change.


To generate a diff of this commit:
cvs rdiff -u -r1.261 -r1.262 src/usr.bin/xlint/lint1/tree.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/xlint/lint1/tree.c
diff -u src/usr.bin/xlint/lint1/tree.c:1.261 src/usr.bin/xlint/lint1/tree.c:1.262
--- src/usr.bin/xlint/lint1/tree.c:1.261	Fri Apr  2 16:17:19 2021
+++ src/usr.bin/xlint/lint1/tree.c	Fri Apr  2 16:38:08 2021
@@ -1,4 +1,4 @@
-/*	$NetBSD: tree.c,v 1.261 2021/04/02 16:17:19 rillig Exp $	*/
+/*	$NetBSD: tree.c,v 1.262 2021/04/02 16:38:08 rillig Exp $	*/
 
 /*
  * Copyright (c) 1994, 1995 Jochen Pohl
@@ -37,7 +37,7 @@
 
 #include <sys/cdefs.h>
 #if defined(__RCSID) && !defined(lint)
-__RCSID("$NetBSD: tree.c,v 1.261 2021/04/02 16:17:19 rillig Exp $");
+__RCSID("$NetBSD: tree.c,v 1.262 2021/04/02 16:38:08 rillig Exp $");
 #endif
 
 #include <float.h>
@@ -2854,7 +2854,7 @@ static tnode_t *
 build_colon(tnode_t *ln, tnode_t *rn)
 {
 	tspec_t	lt, rt, pdt;
-	type_t	*rtp;
+	type_t	*tp;
 	tnode_t	*ntn;
 
 	lt = ln->tn_type->t_tspec;
@@ -2866,11 +2866,11 @@ build_colon(tnode_t *ln, tnode_t *rn)
 	 * still need to be handled.
 	 */
 	if (is_arithmetic(lt) && is_arithmetic(rt)) {
-		rtp = ln->tn_type;
+		tp = ln->tn_type;
 	} else if (lt == BOOL && rt == BOOL) {
-		rtp = ln->tn_type;
+		tp = ln->tn_type;
 	} else if (lt == VOID || rt == VOID) {
-		rtp = gettyp(VOID);
+		tp = gettyp(VOID);
 	} else if (lt == STRUCT || lt == UNION) {
 		/* Both types must be identical. */
 		lint_assert(rt == STRUCT || rt == UNION);
@@ -2880,27 +2880,27 @@ build_colon(tnode_t *ln, tnode_t *rn)
 			error(138, op_name(COLON));
 			return NULL;
 		}
-		rtp = ln->tn_type;
+		tp = ln->tn_type;
 	} else if (lt == PTR && is_integer(rt)) {
 		if (rt != pdt) {
 			rn = convert(NOOP, 0, gettyp(pdt), rn);
 			rt = pdt;
 		}
-		rtp = ln->tn_type;
+		tp = ln->tn_type;
 	} else if (rt == PTR && is_integer(lt)) {
 		if (lt != pdt) {
 			ln = convert(NOOP, 0, gettyp(pdt), ln);
 			lt = pdt;
 		}
-		rtp = rn->tn_type;
+		tp = rn->tn_type;
 	} else if (lt == PTR && ln->tn_type->t_subt->t_tspec == VOID) {
 		lint_assert(rt == PTR);
-		rtp = rn->tn_type;
-		merge_qualifiers(&rtp, ln->tn_type, rn->tn_type);
+		tp = rn->tn_type;
+		merge_qualifiers(&tp, ln->tn_type, rn->tn_type);
 	} else if (rt == PTR && rn->tn_type->t_subt->t_tspec == VOID) {
 		lint_assert(lt == PTR);
-		rtp = ln->tn_type;
-		merge_qualifiers(&rtp, ln->tn_type, rn->tn_type);
+		tp = ln->tn_type;
+		merge_qualifiers(&tp, ln->tn_type, rn->tn_type);
 	} else {
 		lint_assert(lt == PTR);
 		lint_assert(rt == PTR);
@@ -2910,11 +2910,11 @@ build_colon(tnode_t *ln, tnode_t *rn)
 		 * and the other one, at the same place, only an old style
 		 * declaration.
 		 */
-		rtp = ln->tn_type;
-		merge_qualifiers(&rtp, ln->tn_type, rn->tn_type);
+		tp = ln->tn_type;
+		merge_qualifiers(&tp, ln->tn_type, rn->tn_type);
 	}
 
-	ntn = new_tnode(COLON, rtp, ln, rn);
+	ntn = new_tnode(COLON, tp, ln, rn);
 
 	return ntn;
 }

Reply via email to