[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

master 6cc3582: Correct 'concat' manual entry (bug#42296)

From: Mattias Engdegård
Subject: master 6cc3582: Correct 'concat' manual entry (bug#42296)
Date: Sat, 11 Jul 2020 09:13:49 -0400 (EDT)

branch: master
commit 6cc35829e4772479abed1f46f0c51daa24d070e0
Author: Mattias Engdegård <mattiase@acm.org>
Commit: Mattias Engdegård <mattiase@acm.org>

    Correct 'concat' manual entry (bug#42296)
    * doc/lispref/strings.texi (Creating Strings): 'concat' does not
    necessarily return a newly allocated string.  This has been the case
    at least since 1997 (Emacs 20.3).
 doc/lispref/strings.texi | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/doc/lispref/strings.texi b/doc/lispref/strings.texi
index 2ef88b9..8de6255 100644
--- a/doc/lispref/strings.texi
+++ b/doc/lispref/strings.texi
@@ -248,7 +248,7 @@ properties removed.
 @defun concat &rest sequences
 @cindex copying strings
 @cindex concatenating strings
-This function returns a new string consisting of the characters in the
+This function returns a string consisting of the characters in the
 arguments passed to it (along with their text properties, if any).  The
 arguments may be strings, lists of numbers, or vectors of numbers; they
 are not themselves changed.  If @code{concat} receives no arguments, it
@@ -269,9 +269,14 @@ returns an empty string.
 @end example
-This function always constructs a new string that is not @code{eq} to
-any existing string, except when the result is the empty string (to
-save space, Emacs makes only one empty multibyte string).
+This function does not always allocate a new string.  Callers are
+advised not rely on the result being a new string nor on it being
+@code{eq} to an existing string.
+In particular, mutating the returned value may inadvertently change
+another string, alter a constant string in the program, or even raise
+an error.  To obtain a string that you can safely mutate, use
+@code{copy-sequence} on the result.
 For information about other concatenation functions, see the
 description of @code{mapconcat} in @ref{Mapping Functions},

reply via email to

[Prev in Thread] Current Thread [Next in Thread]