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

>From 922c5eff0c03e79714041f3602ce79cceabd5261 Mon Sep 17 00:00:00 2001
From: Jim Meyering <[EMAIL PROTECTED]>
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


_______________________________________________
Bug-coreutils mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/bug-coreutils

Reply via email to