[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#53989: 29.0.50; Gnus searches broken
From: |
Eric Abrahamsen |
Subject: |
bug#53989: 29.0.50; Gnus searches broken |
Date: |
Fri, 18 Feb 2022 08:01:03 -0800 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) |
On 02/18/22 03:10 AM, Michael Heerdegen wrote:
> Eric Abrahamsen <eric@ericabrahamsen.net> writes:
>
>> Okay, it turns out that also lets us drop most of the regexp silliness
>> from the parsing stuff. Would you mind trying the attached patch and
>> confirm that it doesn't break anything? I'll try to make this the last
>> time I mess with search results parsing...
>
> Works so far for me.
>
> | +(cl-defmethod gnus-search-indexed-extract :around
> | + ((_engine gnus-search-indexed))
> | + (let ((ret (cl-call-next-method)))
>
> Why do you need an :around method here - why don't you just add that to
> the primary method?
Hmm... I think I was originally trying to do more in this method --
prefix removal and all that. You're right this isn't necessary.
> | + ;; We run `expand-file-name' here in order to collapse multiple
> | + ;; consecutive directory separators.
> | + (cl-callf expand-file-name (car ret))
> | + ret))
>
> If I were you I would add to the comment that mairix may return such
> multi-separator file names - else it's not clear why that collapsing is
> necessary at that point.
I think I'd rather leave it like this. We don't know if the behavior
might appear under other circumstances, and in the end the real problem
is multiple separators, not mairix.
- bug#53989: 29.0.50; Gnus searches broken, (continued)
- bug#53989: 29.0.50; Gnus searches broken, Eric Abrahamsen, 2022/02/14
- bug#53989: 29.0.50; Gnus searches broken, Michael Heerdegen, 2022/02/14
- bug#53989: 29.0.50; Gnus searches broken, Eric Abrahamsen, 2022/02/14
- bug#53989: 29.0.50; Gnus searches broken, Andreas Schwab, 2022/02/14
- bug#53989: 29.0.50; Gnus searches broken, Eric Abrahamsen, 2022/02/14
- bug#53989: 29.0.50; Gnus searches broken, Michael Heerdegen, 2022/02/14
- bug#53989: 29.0.50; Gnus searches broken, Eric Abrahamsen, 2022/02/14
- bug#53989: 29.0.50; Gnus searches broken, Michael Heerdegen, 2022/02/14
- bug#53989: 29.0.50; Gnus searches broken, Eric Abrahamsen, 2022/02/15
- bug#53989: 29.0.50; Gnus searches broken, Michael Heerdegen, 2022/02/17
- bug#53989: 29.0.50; Gnus searches broken,
Eric Abrahamsen <=
- bug#53989: 29.0.50; Gnus searches broken, Michael Heerdegen, 2022/02/18
- bug#53989: 29.0.50; Gnus searches broken, Eric Abrahamsen, 2022/02/18
- bug#53989: 29.0.50; Gnus searches broken, Michael Heerdegen, 2022/02/18
- bug#53989: 29.0.50; Gnus searches broken, Eric Abrahamsen, 2022/02/14