From db6848fe1b77bc7f2a657231307d2a4271d41482 Mon Sep 17 00:00:00 2001 From: Reto Brunner Date: Mon, 8 Feb 2021 08:40:07 +0100 Subject: notmuch: don't read the full file into memory --- worker/notmuch/message.go | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) (limited to 'worker/notmuch') diff --git a/worker/notmuch/message.go b/worker/notmuch/message.go index 82d00f3..10fbabc 100644 --- a/worker/notmuch/message.go +++ b/worker/notmuch/message.go @@ -3,10 +3,9 @@ package notmuch import ( - "bytes" + "bufio" "fmt" "io" - "io/ioutil" "os" "git.sr.ht/~sircmpwn/aerc/models" @@ -22,7 +21,7 @@ type Message struct { db *notmuch.DB } -// NewReader reads a message into memory and returns an io.Reader for it. +// NewReader returns a reader for a message func (m *Message) NewReader() (io.Reader, error) { name, err := m.Filename() if err != nil { @@ -32,12 +31,7 @@ func (m *Message) NewReader() (io.Reader, error) { if err != nil { return nil, err } - defer f.Close() - b, err := ioutil.ReadAll(f) - if err != nil { - return nil, err - } - return bytes.NewReader(b), nil + return bufio.NewReader(f), nil } // MessageInfo populates a models.MessageInfo struct for the message. -- cgit 1.4.1-2-gfad0