uniq: remove redundant test

* src/uniq.c (find_field): Remove redundant test in outer loop-
termination expression. Also, add a "const" attribute.

diff --git a/src/uniq.c b/src/uniq.c
index 5af52710c6f1a714eb5e4d5a6d1b86cce4a8857b..f2aeb77d9b62b2b2142cfea005a1d5e0f0646dbd 100644 (file)
--- a/src/uniq.c
+++ b/src/uniq.c
@@ -203,14 +203,14 @@ size_opt (char const *opt, char const *msgid)
    return a pointer to the beginning of the line's field to be compared. */
 
 static char *
-find_field (const struct linebuffer *line)
+find_field (struct linebuffer const *line)
 {
   size_t count;
-  char *lp = line->buffer;
+  char const *lp = line->buffer;
   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++;
@@ -221,7 +221,7 @@ find_field (const struct linebuffer *line)
   for (count = 0; count < skip_chars && i < size; count++)
     i++;
 
-  return lp + i;
+  return line->buffer + i;
 }
 
 /* Return false if two strings OLD and NEW match, true if not.