From: Paul Eggert Date: Mon, 13 Jun 2011 04:27:45 +0000 (-0700) Subject: * editfns.c (Ftranslate_region_internal): Omit redundant test. X-Git-Tag: emacs-pretest-24.0.90~104^2~548^2~33 X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=045eb8d982fb3e72c71b5819f3fa684ac9e52a23;p=emacs.git * editfns.c (Ftranslate_region_internal): Omit redundant test. --- diff --git a/src/ChangeLog b/src/ChangeLog index f5b81cff88e..22e64fb409a 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@ -1,5 +1,7 @@ 2011-06-13 Paul Eggert + * editfns.c (Ftranslate_region_internal): Omit redundant test. + * fns.c (concat): Minor tuning based on overflow analysis. This doesn't fix any bugs. Use int to hold character, instead of constantly refetching from Emacs object. Use XFASTINT, not diff --git a/src/editfns.c b/src/editfns.c index bcf3b751aa9..12e82c428ad 100644 --- a/src/editfns.c +++ b/src/editfns.c @@ -3088,14 +3088,11 @@ It returns the number of characters changed. */) } else { - int c; - nc = oc; val = CHAR_TABLE_REF (table, oc); - if (CHARACTERP (val) - && (c = XFASTINT (val), CHAR_VALID_P (c, 0))) + if (CHARACTERP (val)) { - nc = c; + nc = XFASTINT (val); str_len = CHAR_STRING (nc, buf); str = buf; }