From 045eb8d982fb3e72c71b5819f3fa684ac9e52a23 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Sun, 12 Jun 2011 21:27:45 -0700 Subject: [PATCH] * editfns.c (Ftranslate_region_internal): Omit redundant test. --- src/ChangeLog | 2 ++ src/editfns.c | 7 ++----- 2 files changed, 4 insertions(+), 5 deletions(-) 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; } -- 2.39.2