diff options
author | Paul Eggert <eggert@cs.ucla.edu> | 2011-06-12 21:27:45 -0700 |
---|---|---|
committer | Paul Eggert <eggert@cs.ucla.edu> | 2011-06-12 21:27:45 -0700 |
commit | 045eb8d982fb3e72c71b5819f3fa684ac9e52a23 (patch) | |
tree | e11b99adad9de900fc537e092c9a742e2277cfeb | |
parent | c1f134b59b870f6385364a611911a1344cfef7a2 (diff) |
* editfns.c (Ftranslate_region_internal): Omit redundant test.
-rw-r--r-- | src/ChangeLog | 2 | ||||
-rw-r--r-- | src/editfns.c | 7 |
2 files changed, 4 insertions, 5 deletions
diff --git a/src/ChangeLog b/src/ChangeLog index f5b81cff88..22e64fb409 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@ -1,5 +1,7 @@ 2011-06-13 Paul Eggert <eggert@cs.ucla.edu> + * 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 bcf3b751aa..12e82c428a 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; } |