~sircmpwn/aerc

Strip trailing newline from address book entries without names v1 PROPOSED

Ben Burwell: 1
 Strip trailing newline from address book entries without names

 1 files changed, 11 insertions(+), 9 deletions(-)
Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~sircmpwn/aerc/patches/9483/mbox | git am -3
Learn more about email & git

[PATCH] Strip trailing newline from address book entries without names Export this patch

When the list of completions from the external command doesn't have
associated contact names, the email address we attempt to parse was
being terminated with a newline. Now, we strip the trailing newline if
present.
---
 completer/completer.go | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

diff --git a/completer/completer.go b/completer/completer.go
index f6900ee..bc6c96f 100644
--- a/completer/completer.go
+++ b/completer/completer.go
@@ -138,16 +138,18 @@ func readCompletions(r io.Reader) ([]string, error) {
			return nil, err
		}
		parts := strings.SplitN(line, "\t", 3)
		if addr, err := mail.ParseAddress(parts[0]); err == nil {
			if len(parts) > 1 {
				addr.Name = strings.TrimSpace(parts[1])
			}
			decoded, err := decodeMIME(addr.String())
			if err != nil {
				return nil, fmt.Errorf("could not decode MIME string: %w", err)
			}
			completions = append(completions, decoded)
		addr, err := mail.ParseAddress(strings.TrimSpace(parts[0]))
		if err != nil {
			return nil, err
		}
		if len(parts) > 1 {
			addr.Name = strings.TrimSpace(parts[1])
		}
		decoded, err := decodeMIME(addr.String())
		if err != nil {
			return nil, fmt.Errorf("could not decode MIME string: %w", err)
		}
		completions = append(completions, decoded)
	}
}

-- 
2.24.1
Thanks!

To git.sr.ht:~sircmpwn/aerc
   a40959c..598e39f  master -> master
View this thread in the archives