* coding.c (encode_coding_raw_text): Avoid unnecessary test

the first time through the loop, since we know p0 < p1 then.
This also avoids a gcc -Wstrict-overflow warning.

diff --git a/src/ChangeLog b/src/ChangeLog
index a341d1b467831c54c8ab1d11d6f5a22e49133678..45982f607f4f89457a8ba52d680cbccc06db32f2 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,5 +1,9 @@
 2011-03-22  Paul Eggert  <eggert@cs.ucla.edu>
 
+       * coding.c (encode_coding_raw_text): Avoid unnecessary test
+       the first time through the loop, since we know p0 < p1 then.
+       This also avoids a gcc -Wstrict-overflow warning.
+
        * lisp.h (SAFE_ALLOCA, SAFE_ALLOCA_LISP): Avoid 'int' overflow
        leading to a memory leak, possible in functions like
        load_charset_map_from_file that can allocate an unbounded number

diff --git a/src/coding.c b/src/coding.c
index 0c2836c19f6897b718bed5cd818c82e4cc32e99d..0596d16bf463177235d1a3096787bb9ee32758bc 100644 (file)
--- a/src/coding.c
+++ b/src/coding.c
@@ -5266,11 +5266,12 @@ encode_coding_raw_text (struct coding_system *coding)
                unsigned char str[MAX_MULTIBYTE_LENGTH], *p0 = str, *p1 = str;
 
                CHAR_STRING_ADVANCE (c, p1);
-               while (p0 < p1)
+               do
                  {
                    EMIT_ONE_BYTE (*p0);
                    p0++;
                  }
+               while (p0 < p1);
              }
          }
       else