GNU bug report logs - #76018
31.0.50; wrap-prefix properties from visual-wrap-prefix-mode proliferate

Previous Next

Full log

Message #74 received at 76018 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Jim Porter <jporterbugs <at> gmail.com>
Cc: luangruo <at> yahoo.com, 76018 <at> debbugs.gnu.org, monnier <at> iro.umontreal.ca,
 kevin.legouguec <at> gmail.com
Subject: Re: bug#76018: 31.0.50; wrap-prefix properties from
 visual-wrap-prefix-mode proliferate
Date: Mon, 02 Jun 2025 08:58:27 +0300

> Date: Sun, 1 Jun 2025 12:24:36 -0700
> Cc: luangruo <at> yahoo.com, 76018 <at> debbugs.gnu.org, monnier <at> iro.umontreal.ca,
>  kevin.legouguec <at> gmail.com
> From: Jim Porter <jporterbugs <at> gmail.com>
> 
> I've reworded the commit message to explain things better (I hope):
> 
>    When calling 'add-display-text-property' on a region of text that
>    already contains PROP, we delete the old property from the list of
>    values.  If the region's 'display' property is a list of display
>    specifications, we need to avoid destructively modifying the list; other
>    regions of text could be using the same list object.  Instead, if
>    deleting a property from a list, use 'remove'.  (For a 'display'
>    properties that's a vector or a single display spec, this doesn't matter
>    since we first make a new list in the code.)

This is more clear, but please use consistent terminology.  In the
above, you use "list of values" and "list of display specifications"
to mean the same thing (AFAIU), and "property" to mean "display
specification".

Thanks.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.