On Sun, Sep 15 2013, Mark Walters <[email protected]> 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:[email protected]

LGTM

Tomi

> and simplifies the code here slightly. This patch applies on top of the
> series at id:[email protected]
>
> (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(-)
>
_______________________________________________
notmuch mailing list
[email protected]
http://notmuchmail.org/mailman/listinfo/notmuch

Reply via email to