[PATCH] contrib: pick: tidy up pick-format-field

Tomi Ollila tomi.ollila at iki.fi
Mon Sep 16 13:15:29 PDT 2013


On Sun, Sep 15 2013, Mark Walters <markwalters1009 at gmail.com> wrote:

> Previously this function used a temporary variable to store the return
> value but we can just use the return value of the cond statement
> directly.
>
> The only tiny subtlety is that in one case (subject) we need to
> slightly reorder the logic to make sure the formatted-field is the
> last thing computed.
> ---

> This follows Tomi's suggestion in id:m2ob7vg24n.fsf at guru.guru-group.fi

LGTM

Tomi

> and simplifies the code here slightly. This patch applies on top of the
> series at id:1378092488-32050-1-git-send-email-markwalters1009 at gmail.com
>
> (It also includes one small whitespace fix-up for the face for the tags)
>
>
>  contrib/notmuch-pick/notmuch-pick.el |   49 ++++++++++++++-------------------
>  1 files changed, 21 insertions(+), 28 deletions(-)
>


More information about the notmuch mailing list