Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/usr.bin/xlint/lint1 lint: remove redundant comment



details:   https://anonhg.NetBSD.org/src/rev/def209993ced
branches:  trunk
changeset: 959468:def209993ced
user:      rillig <rillig%NetBSD.org@localhost>
date:      Mon Feb 15 07:40:18 2021 +0000

description:
lint: remove redundant comment

That comment was useful when there was no function is_null_pointer.
Back then, the code for testing a null pointer was written in-line,
which made it really hard to see what's going on.  This is no longer the
case.

diffstat:

 usr.bin/xlint/lint1/tree.c |  5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diffs (26 lines):

diff -r ac253e6455e4 -r def209993ced usr.bin/xlint/lint1/tree.c
--- a/usr.bin/xlint/lint1/tree.c        Mon Feb 15 07:36:40 2021 +0000
+++ b/usr.bin/xlint/lint1/tree.c        Mon Feb 15 07:40:18 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: tree.c,v 1.206 2021/02/15 07:36:40 rillig Exp $        */
+/*     $NetBSD: tree.c,v 1.207 2021/02/15 07:40:18 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.206 2021/02/15 07:36:40 rillig Exp $");
+__RCSID("$NetBSD: tree.c,v 1.207 2021/02/15 07:40:18 rillig Exp $");
 #endif
 
 #include <float.h>
@@ -1049,7 +1049,6 @@
        if (lt == UNION && rt == UNION && ltp->t_str == rtp->t_str)
                return true;
 
-       /* combination of any pointer and null pointer is ok */
        if (lt == PTR && is_null_pointer(rn))
                return true;
        if (rt == PTR && is_null_pointer(ln))



Home | Main Index | Thread Index | Old Index