Tomi Ollila writes:
>
> emacs -q -L $PWD/emacs -l emacs/notmuch.el -f notmuch --eval '(progn (setq
> notmuch-address-command "nottoomuch-addresses.sh")
> (notmuch-address-message-insinuate))'
>
Ah, I missed notmuch-address-message-insinuate; it does work if I run
that. I wonder if this can be
WARNING: this version is very preliminary, and might eat your data.
Unicode has multiple sequences representing what should normally be
considered the same text. For example here's a combining AÌ and a
noncombining Ã.
Depending on the way you view this, you may or may not see a
difference, but
On Sun, Aug 30 2015, David Bremner wrote:
> Tomi Ollila writes:
>
>> +(defsubst notmuch-address--message-insinuated ()
>> + (memq notmuch-address-message-alist-member message-completion-alist))
>> +
>
> Is there some advantage to defsubst other than (maybe?) performance? It
> just seems like o
On Sun, Aug 30 2015, David Bremner wrote:
> Tomi Ollila writes:
>
>> The new function notmuch-show-message-resend re-sends
>> message to new recipients using #'message-resend.
>>
>> Recipients are read from minibuffer as a comma-separated
>> string (with some keyboard support including tab compl
Tomi Ollila writes:
> The new function notmuch-show-message-resend re-sends
> message to new recipients using #'message-resend.
>
> Recipients are read from minibuffer as a comma-separated
> string (with some keyboard support including tab completion).
>
I couldn't get the tab completion to work
Tomi Ollila writes:
> +(defsubst notmuch-address--message-insinuated ()
> + (memq notmuch-address-message-alist-member message-completion-alist))
> +
Is there some advantage to defsubst other than (maybe?) performance? It
just seems like one more construct for people to get up to speed with
th
Jani Nikula writes:
> Consider all variants of an email address as one, and print the most
> common variant.
Other than the quibbles already mentioned, the series looks ok to
me. For production it should have one or two tests I guess. Oh, and man
page updates. But you knew that I guess.
I can l
Jani Nikula writes:
>> I found this use of mailbox as a temporary variable confusing; despite
>> the obvious return I thought it might have something to do with the
>> g_list_append below. Maybe just make a block scope temporary variable?
>
> This is how the function would turn out with that. Be
On Sun, 30 Aug 2015, David Bremner wrote:
> I guess this doesn't make the error handling worse; both old and new
> code silently ignore OOM if I understand correctly.
Oh, and current git will not silently ignore OOM. It will segfault... ;)
BR,
Jani.
__
On Sun, 30 Aug 2015, David Bremner wrote:
> Jani Nikula writes:
>
>>
>> +static int
>> +strcase_equal (const void *a, const void *b)
>> +{
>> +return strcasecmp (a, b) == 0;
>> +}
>> +
>> +static unsigned int
>> +strcase_hash (const void *ptr)
>> +{
>> +const char *s = ptr;
>> +
>> +
On Sun, 30 Aug 2015, David Bremner wrote:
> Jani Nikula writes:
>
>
>> +{ NOTMUCH_OPT_KEYWORD, &ctx->dupe, "deduplicate", 'x',
>
> probably you want 'D' or 'd' here. Not that it makes a practical
> difference at this point.
>
>> + (notmuch_keyword_t []){ { "yes", -1 },
>
> I'm not very e
11 matches
Mail list logo