bug-coreutils
[Top][All Lists]
Advanced

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

FYI, uniq: remove redundant test


From: Jim Meyering
Subject: FYI, uniq: remove redundant test
Date: Fri, 27 Jun 2008 19:21:03 +0200

It doesn't change semantics,
but does change the generated code:

>From 922c5eff0c03e79714041f3602ce79cceabd5261 Mon Sep 17 00:00:00 2001
From: Jim Meyering <address@hidden>
Date: Fri, 27 Jun 2008 18:52:56 +0200
Subject: [PATCH] uniq: remove redundant test

* src/uniq.c (find_field): Remove redundant test in outer loop-
termination expression.
---
 src/uniq.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/uniq.c b/src/uniq.c
index 5af5271..bfb85f1 100644
--- a/src/uniq.c
+++ b/src/uniq.c
@@ -210,7 +210,7 @@ find_field (const struct linebuffer *line)
   size_t size = line->length - 1;
   size_t i = 0;

-  for (count = 0; count < skip_fields && i < size; count++)
+  for (count = 0; count < skip_fields; count++)
     {
       while (i < size && isblank (to_uchar (lp[i])))
        i++;
--
1.5.6.1.91.gc2a16




reply via email to

[Prev in Thread] Current Thread [Next in Thread]