[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3] Allow applying filters to summary consecutively
From: |
Andrea Monaco |
Subject: |
Re: [PATCH v3] Allow applying filters to summary consecutively |
Date: |
Thu, 20 Oct 2022 17:45:56 +0200 |
> Iʼd call it `rmail-summary-stack-filters' and say
The metaphor of a stack is not perfect, as we don't have any "pop", that
is there's no command to cancel the last filter.
> I donʼt remember the details of rmailʼs summary line format, but I
> think youʼd be better served with (thing-at-point 'number)
There is some padding with whitespace at the beginning of each line, so
you'd still need a jump command.
> I think you have this same lambda about 4 times in the code, perhaps
> `defun' it (using "rmail-summary--" as a prefix to indicate that
> itʼs intended for internal usage)
The five lambdas I added (one for each rmail-summary-by-* command) are
similar but not identical. Maybe I could write a general function, but
I find it too much of a hassle for a little line saving.
I agree on the other objections.
Andrea Monaco
- Re: [PATCH v2] Allow applying filters to summary consecutively (was: Summary by thread in rmail), (continued)
- Re: [PATCH v2] Allow applying filters to summary consecutively (was: Summary by thread in rmail), Andrea Monaco, 2022/10/11
- Re: [PATCH v2] Allow applying filters to summary consecutively, Robert Pluim, 2022/10/11
- Re: [PATCH v2] Allow applying filters to summary consecutively, Andrea Monaco, 2022/10/12
- Re: [PATCH v2] Allow applying filters to summary consecutively, Robert Pluim, 2022/10/12
- Re: [PATCH v2] Allow applying filters to summary consecutively, Eli Zaretskii, 2022/10/12
- Re: [PATCH v2] Allow applying filters to summary consecutively, Andrea Monaco, 2022/10/12
- Re: [PATCH v2] Allow applying filters to summary consecutively, Eli Zaretskii, 2022/10/12
- Re: [PATCH v2] Allow applying filters to summary consecutively, Richard Stallman, 2022/10/12
- Re: [PATCH v3] Allow applying filters to summary consecutively, Andrea Monaco, 2022/10/19
- Re: [PATCH v3] Allow applying filters to summary consecutively, Robert Pluim, 2022/10/19
- Re: [PATCH v3] Allow applying filters to summary consecutively,
Andrea Monaco <=
- Re: [PATCH v3] Allow applying filters to summary consecutively, Robert Pluim, 2022/10/20
- Re: [PATCH v3] Allow applying filters to summary consecutively, Andrea Monaco, 2022/10/20
- Re: [PATCH v3] Allow applying filters to summary consecutively, Richard Stallman, 2022/10/21
- Re: [PATCH v3] Allow applying filters to summary consecutively, Richard Stallman, 2022/10/21
- Re: [PATCH v3] Allow applying filters to summary consecutively, Eli Zaretskii, 2022/10/27
- Re: [PATCH v3] Allow applying filters to summary consecutively, Andrea Monaco, 2022/10/27
- Re: [PATCH v3] Allow applying filters to summary consecutively, Andrea Monaco, 2022/10/28
- Re: [PATCH v2] Allow applying filters to summary consecutively (was: Summary by thread in rmail), Eli Zaretskii, 2022/10/27
- Re: [PATCH] Allow applying filters to summary consecutively (was: Summary by thread in rmail), Richard Stallman, 2022/10/10