GNU bug report logs - #65632
30.0.50; Proposal to improve `faces--attribute-at-point'.

Previous Next

Package: emacs;

Reported by: David Ponce <da_vid <at> orange.fr>

Date: Wed, 30 Aug 2023 18:06:02 UTC

Severity: wishlist

Tags: moreinfo, patch

Found in version 30.0.50

Done: Stefan Kangas <stefankangas <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


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

From: David Ponce <da_vid <at> orange.fr>
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: Eli Zaretskii <eliz <at> gnu.org>, 65632 <at> debbugs.gnu.org
Subject: Re: bug#65632: 30.0.50; Proposal to improve
 `faces--attribute-at-point'.
Date: Mon, 24 Feb 2025 12:01:41 +0100
On 2025-02-23 02:27, Stefan Kangas wrote:
> David Ponce <da_vid <at> orange.fr> writes:
> 
>> Sorry, my previous patch was buggy.  Here is the correct one.
> 
> (Sorry for the lack of a response for a long time here.)
> 
> Could you please provide a rationale for these changes?  IOW, what are
> they trying to achieve?  Thanks in advance.

Hello Stefan,

Sincerely, I thought this bug was closed a long time ago.

The rationale of this proposal was to handle any possible face spec returned by (get-char-property (point) 'face) in `faces--attribute-at-point' (by using `face-attributes-as-vector'), because in some corner cases the current implementation can return a wrong attribute value, as illustrated by the simple recipe in my initial post.

However, if I correctly understand Eli's objection, I agree that the face spec in my recipe can be considered as invalid, and not supported by `faces--attribute-at-point', even if a such face spec is an accepted text property, probably for backward compatibility.

Thanks




This bug report was last modified 85 days ago.

Previous Next


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