Re: [PATCH v4 2/6] config: Check 'config get' arity exactly

Subject: Re: [PATCH v4 2/6] config: Check 'config get' arity exactly

Date: Sat, 28 Apr 2012 10:51:48 -0300

To: Peter Wang, notmuch@notmuchmail.org

Cc:

From: David Bremner

Peter Wang <novalazy@gmail.com> writes:

> Require that 'config get' is passed exactly one additional argument,
> instead of silently ignoring extra arguments. As a side-effect, produce
> more specific error messages for the 'config' command as a whole.

Hi Peter;

I pushed the rest of the series to master. Would you mind making patch
to NEWS describing the new feature?

thanks

d

• Previous message (by thread): Re: [PATCH v4 2/6] config: Check 'config get' arity exactly

Thread:

• Peter Wang—[PATCH 1/2] config: Add 'config list' command [inbox, notmuch::patch, notmuch::wip, unread]
  • Peter Wang—[PATCH 2/2] man: Document 'config list' command [inbox, notmuch::patch, notmuch::wip, unread]
  • Tomi Ollila—Re: [PATCH 1/2] config: Add 'config list' command [inbox, unread]
    • Peter Wang—Re: [PATCH 1/2] config: Add 'config list' command [inbox, unread]
    • Jameson Graef Rollins—Re: [PATCH 1/2] config: Add 'config list' command [inbox, signed, unread]
  • Jameson Graef Rollins—Re: [PATCH 1/2] config: Add 'config list' command [inbox, signed, unread]
  • Peter Wang—[PATCH v2] Add 'config list' command [inbox, unread]
    • Peter Wang—[PATCH v2 1/5] config: Fix free in 'config get' implementation. [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Peter Wang—[PATCH v2 2/5] test: Add tests for 'config' command [inbox, notmuch::obsolete, notmuch::patch, unread]
      • Mark Walters—Re: [PATCH v2 2/5] test: Add tests for 'config' command [inbox, notmuch::review, unread]
        • Jameson Graef Rollins—Re: [PATCH v2 2/5] test: Add tests for 'config' command [inbox, signed, unread]
    • Peter Wang—[PATCH v2 3/5] test: Add broken test for 'config list' [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Peter Wang—[PATCH v2 4/5] config: Add 'config list' command [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Peter Wang—[PATCH v2 5/5] man: Document 'config list' command [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v3 0/5] Add 'config list' command [inbox, unread]
    • Peter Wang—[PATCH v3 1/5] config: Fix free in 'config get' implementation. [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Peter Wang—[PATCH v3 2/5] test: Add tests for 'config' command [inbox, notmuch::obsolete, notmuch::patch, unread]
      • Jameson Graef Rollins—Re: [PATCH v3 2/5] test: Add tests for 'config' command [inbox, signed, unread]
    • Peter Wang—[PATCH v3 3/5] test: Add broken test for 'config list' [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Peter Wang—[PATCH v3 4/5] config: Add 'config list' command [inbox, notmuch::obsolete, notmuch::patch, unread]
      • Jameson Graef Rollins—Re: [PATCH v3 4/5] config: Add 'config list' command [inbox, signed, unread]
        • Peter Wang—Re: [PATCH v3 4/5] config: Add 'config list' command [inbox, unread]
    • Peter Wang—[PATCH v3 5/5] man: Document 'config list' command [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Peter Wang—[PATCH v4 0/6] Config-related patches [inbox, unread]
    • Peter Wang—[PATCH v4 1/6] config: Fix free in 'config get' implementation. [inbox, notmuch::patch, notmuch::pushed, notmuch::trivial, unread]
      • David Bremner—Re: [PATCH v4 1/6] config: Fix free in 'config get' implementation. [inbox, unread]
    • Peter Wang—[PATCH v4 2/6] config: Check 'config get' arity exactly [inbox, notmuch::patch, notmuch::pushed, unread]
      • David Bremner—Re: [PATCH v4 2/6] config: Check 'config get' arity exactly [inbox, unread]
    • Peter Wang—[PATCH v4 3/6] test: Add tests for 'config' command [inbox, notmuch::patch, notmuch::pushed, unread]
    • Peter Wang—[PATCH v4 4/6] test: Add broken test for 'config list' [inbox, notmuch::patch, notmuch::pushed, unread]
    • Peter Wang—[PATCH v4 5/6] config: Add 'config list' command [inbox, notmuch::patch, notmuch::pushed, unread]
    • Peter Wang—[PATCH v4 6/6] man: Document 'config list' command [inbox, notmuch::patch, notmuch::pushed, unread]
    • Mark Walters—Re: [PATCH v4 0/6] Config-related patches [inbox, notmuch::review, unread]
      • Jameson Graef Rollins—Re: [PATCH v4 0/6] Config-related patches [inbox, signed, unread]
      • Mark Walters—Re: [PATCH v4 0/6] Config-related patches [inbox, unread]
    • Jameson Graef Rollins—Re: [PATCH v4 0/6] Config-related patches [inbox, signed, unread]