utf8-srfi-13 string-map! does not update in-place

[Caolan McMahon]

(use srfi-13)
(define str "foo")
(string-map! (lambda (c) #\x) str) ;; => "xxx"
str ;; => "xxx"

(use utf8-srfi-13)
(define str "foo")
(string-map! (lambda (c) #\x) str) ;; => "xxx"
str ;; => "foo"

[Alex Shinn]

This is intentional - it would be super slow to do otherwise.

[evhan]

Would it be better to signal an error indicating that in-place mapping is unsupported? The current behaviour leads to more subtle bugs.

[Alex Shinn]

I really hate this API, but since the sole purpose is SRFI-13 compatibility I've gone ahead and fixed it to update in place.

If you know you might want to use the utf8 egg, you should never use string-map! in programs.