[Top][All Lists]

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

Re: bug#60936: 30.0.50; ERC >5.5: Add erc-fill style based on visual-lin

From: J.P.
Subject: Re: bug#60936: 30.0.50; ERC >5.5: Add erc-fill style based on visual-line-mode
Date: Tue, 17 Oct 2023 06:48:21 -0700
User-agent: Gnus/5.13 (Gnus v5.13)

v2 (erc-display-line redux). Fix initial bug involving missing text
props on multi-buffer calls to `erc-display-line'. Convert latter to
internal function and reimplement interface as high-level wrapper around

"J.P." <jp@neverwas.me> writes:

> "J.P." <jp@neverwas.me> writes:
>> These changes introduced a(t least one) bug. To reproduce, call
>> `erc-display-line' with a list of buffers, and notice only the first
>> sees its message inserted with the correct text properties. A quick way
>> to simulate this is by having two clients join the same two channels
>> and then having one quit. The expected text props will be missing from
>> one of the inserted
>>   *** someuser (n!~u@h) has quit
>> messages. Verify by going to the first asterisk and doing C-u C-x =.
>> Fix forthcoming.
> The second of the attached patches should hopefully do the trick.

Actually, merely hoping folks will use `erc-display-message' instead of
`erc-display-line' is surely delusional. There's likely far too much
code out there doing stuff like:

  (erc-display-line (erc-make-notice "foo") my-buffer)

So I've instead converted `erc-display-line' into a high-level insertion
function more aligned with the manner in which it's used in practice.
It's now more or less a thin wrapper around `erc-display-message' with a
bit of special casing to intercept instances of the `erc-make-notice'
pattern above for rewriting as:

  (erc-display-message nil 'notice my-buffer "foo")

Hopefully, this is an acceptable compromise.

Attachment: 0000-v1-v2.diff
Description: Text Data

Attachment: 0001-Mark-erc-log-test-as-unstable.patch
Description: Text Data

Attachment: 0002-5.6-Restore-missing-metadata-props-in-erc-display-li.patch
Description: Text Data

reply via email to

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