all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Stefan Kangas <stefan@marxist.se>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 48740@debbugs.gnu.org, Ihor Radchenko <yantar92@gmail.com>
Subject: bug#48740: 28.0.50; Composition text property is not always honoured
Date: Sun, 24 Oct 2021 00:42:56 -0700	[thread overview]
Message-ID: <CADwFkmmg1Fjo0W2U2ZHf+pBfj0Mp-eGgKpP8DfLB9xVz=Zfh1w@mail.gmail.com> (raw)
In-Reply-To: <8335t7tffy.fsf@gnu.org> (Eli Zaretskii's message of "Thu, 24 Jun 2021 19:06:41 +0300")

[-- Attachment #1: Type: text/plain, Size: 379 bytes --]

Eli Zaretskii <eliz@gnu.org> writes:

>> I understand. However, the problem is quite unexpected. I do not see how
>> application can anticipate it. Probably, adding a note to docstring
>> would be useful? Something like in the attached patch.
>
> Fine with me, thanks.

The patch here was never installed.  Should it be?

I've attached a version below with some minor touch-ups.

[-- Attachment #2: 0001-Doc-fix-for-concat.patch --]
[-- Type: text/x-diff, Size: 935 bytes --]

From d2b1f2125b98ac8b496712ed133cda27757437b5 Mon Sep 17 00:00:00 2001
From: Ihor Radchenko <yantar92@gmail.com>
Date: Thu, 24 Jun 2021 22:33:08 +0800
Subject: [PATCH] Doc fix for concat

* src/fns.c (Fconcat): Note that composition values may not remain eq
in return value of concat.  (Bug#48740)
---
 src/fns.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/fns.c b/src/fns.c
index a72e41aee5..6f358dd1ba 100644
--- a/src/fns.c
+++ b/src/fns.c
@@ -672,6 +672,9 @@ DEFUN ("concat", Fconcat, Sconcat, 0, MANY, 0,
        doc: /* Concatenate all the arguments and make the result a string.
 The result is a string whose elements are the elements of all the arguments.
 Each argument may be a string or a list or vector of characters (integers).
+
+Values of the `composition' property of the result are not guaranteed
+to be `eq'.
 usage: (concat &rest SEQUENCES)  */)
   (ptrdiff_t nargs, Lisp_Object *args)
 {
-- 
2.30.2


  reply	other threads:[~2021-10-24  7:42 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-30  7:35 bug#48740: 28.0.50; Composition text property is not always honoured Ihor Radchenko
2021-05-30 11:10 ` Eli Zaretskii
2021-05-30 13:08   ` Ihor Radchenko
2021-05-30 13:52     ` Eli Zaretskii
2021-05-30 15:13       ` Ihor Radchenko
2021-05-30 15:58         ` Eli Zaretskii
2021-05-31 13:00           ` Ihor Radchenko
2021-05-31 14:32             ` Eli Zaretskii
2021-06-01 14:28               ` Ihor Radchenko
2021-06-01 15:25                 ` Eli Zaretskii
2021-06-01 16:25                   ` Eli Zaretskii
2021-06-20 11:24                   ` Ihor Radchenko
2021-06-20 11:53                     ` Eli Zaretskii
2021-06-20 13:16                       ` Ihor Radchenko
2021-06-21 12:18                         ` Eli Zaretskii
2021-06-24 14:35                           ` Ihor Radchenko
2021-06-24 16:06                             ` Eli Zaretskii
2021-10-24  7:42                               ` Stefan Kangas [this message]
2021-10-24 12:04                                 ` Eli Zaretskii
2021-10-24 12:27                                   ` Stefan Kangas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CADwFkmmg1Fjo0W2U2ZHf+pBfj0Mp-eGgKpP8DfLB9xVz=Zfh1w@mail.gmail.com' \
    --to=stefan@marxist.se \
    --cc=48740@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=yantar92@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.