diff options
author | Mark H Weaver <mhw@netris.org> | 2013-12-25 05:10:19 -0500 |
---|---|---|
committer | Mark H Weaver <mhw@netris.org> | 2014-01-08 21:42:24 -0500 |
commit | 793e8a9317d24298c82389bdf86b8ca17b4ee2f0 (patch) | |
tree | 44df01f67907fdbd39260111a2baafdc8beadc8b /libguile/srfi-13.c | |
parent | 900a897cd31df98df06b84a478b77a7438739b54 (diff) | |
download | guile-793e8a9317d24298c82389bdf86b8ca17b4ee2f0.tar.gz |
Fix 'string-copy!' to work properly with overlapping src/dest.
* libguile/srfi-13.c (scm_string_copy_x): Fix to work properly with
overlapping src/dest.
* test-suite/tests/srfi-13.test ("string-copy!"): Add tests.
Diffstat (limited to 'libguile/srfi-13.c')
-rw-r--r-- | libguile/srfi-13.c | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/libguile/srfi-13.c b/libguile/srfi-13.c index 4e5d5725f..5c30dfe20 100644 --- a/libguile/srfi-13.c +++ b/libguile/srfi-13.c @@ -546,10 +546,17 @@ SCM_DEFINE (scm_string_copy_x, "string-copy!", 3, 2, 0, SCM_ASSERT_RANGE (3, s, len <= scm_i_string_length (target) - ctstart); target = scm_i_string_start_writing (target); - for (i = 0; i < cend - cstart; i++) + if (ctstart < cstart) { - scm_i_string_set_x (target, ctstart + i, - scm_i_string_ref (s, cstart + i)); + for (i = 0; i < len; i++) + scm_i_string_set_x (target, ctstart + i, + scm_i_string_ref (s, cstart + i)); + } + else + { + for (i = len; i--;) + scm_i_string_set_x (target, ctstart + i, + scm_i_string_ref (s, cstart + i)); } scm_i_string_stop_writing (); scm_remember_upto_here_1 (target); |