From: Bruno Haible <bruno@clisp.org>
Date: Mon, 24 May 2010 08:44:42 +0000 (+0200)
Subject: Tiny optimization.
X-Git-Tag: v0.18.1~29
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=41d50d2ca8b6c81e418654921b04bf8cd9242813;p=thirdparty%2Fgettext.git

Tiny optimization.
---

diff --git a/gettext-tools/libgrep/ChangeLog b/gettext-tools/libgrep/ChangeLog
index dd951dbc6..8804b00d7 100644
--- a/gettext-tools/libgrep/ChangeLog
+++ b/gettext-tools/libgrep/ChangeLog
@@ -1,3 +1,8 @@
+2010-05-24  Bruno Haible  <bruno@clisp.org>
+
+	Tiny optimization.
+	* m-fgrep.c (Fexecute): New local variable buflim.
+
 2010-05-24  Bruno Haible  <bruno@clisp.org>
 
 	* m-fgrep.c (Fcompile, Fexecute): Reduce scope of local variables.
diff --git a/gettext-tools/libgrep/m-fgrep.c b/gettext-tools/libgrep/m-fgrep.c
index 5c92365b3..7bb3715eb 100644
--- a/gettext-tools/libgrep/m-fgrep.c
+++ b/gettext-tools/libgrep/m-fgrep.c
@@ -157,6 +157,7 @@ Fexecute (const void *compiled_pattern, const char *buf, size_t buf_size,
 {
   struct compiled_kwset *ckwset = (struct compiled_kwset *) compiled_pattern;
   char eol = ckwset->eolbyte;
+  register const char *buflim = buf + buf_size;
   register const char *beg;
   register size_t len;
 #ifdef MBS_SUPPORT
@@ -165,11 +166,10 @@ Fexecute (const void *compiled_pattern, const char *buf, size_t buf_size,
     mb_properties = check_multibyte_string (buf, buf_size);
 #endif /* MBS_SUPPORT */
 
-  for (beg = buf; beg <= buf + buf_size; ++beg)
+  for (beg = buf; beg <= buflim; ++beg)
     {
       struct kwsmatch kwsmatch;
-      size_t offset =
-        kwsexec (ckwset->kwset, beg, buf + buf_size - beg, &kwsmatch);
+      size_t offset = kwsexec (ckwset->kwset, beg, buflim - beg, &kwsmatch);
       if (offset == (size_t) -1)
         {
 #ifdef MBS_SUPPORT
@@ -197,7 +197,7 @@ Fexecute (const void *compiled_pattern, const char *buf, size_t buf_size,
         {
           if (beg > buf && beg[-1] != eol)
             continue;
-          if (beg + len < buf + buf_size && beg[len] != eol)
+          if (beg + len < buflim && beg[len] != eol)
             continue;
           goto success;
         }
@@ -208,7 +208,7 @@ Fexecute (const void *compiled_pattern, const char *buf, size_t buf_size,
             {
               if (curr > buf && IS_WORD_CONSTITUENT ((unsigned char) curr[-1]))
                 break;
-              if (curr + len < buf + buf_size
+              if (curr + len < buflim
                   && IS_WORD_CONSTITUENT ((unsigned char) curr[len]))
                 {
                   offset = kwsexec (ckwset->kwset, beg, --len, &kwsmatch);
@@ -241,11 +241,11 @@ Fexecute (const void *compiled_pattern, const char *buf, size_t buf_size,
   {
     register const char *end;
 
-    end = (const char *) memchr (beg + len, eol, (buf + buf_size) - (beg + len));
+    end = (const char *) memchr (beg + len, eol, buflim - (beg + len));
     if (end != NULL)
       end++;
     else
-      end = buf + buf_size;
+      end = buflim;
     while (buf < beg && beg[-1] != eol)
       --beg;
     *match_size = end - beg;