Re: [PATCH] cli: factor out config handling code to get/set lists.

Subject: Re: [PATCH] cli: factor out config handling code to get/set lists.

Date: Mon, 12 Dec 2011 14:26:21 -0400

To: Dmitry Kurochkin, notmuch@notmuchmail.org

Cc:

From: David Bremner

On Sun, 11 Dec 2011 20:41:53 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
> Hi David.
> 
> On Sun, 11 Dec 2011 12:07:51 -0400, David Bremner <david@tethera.net> wrote:
> > From: David Bremner <bremner@debian.org>
> > 
> > Two new internal routines are created _config_get_list and
> > _config_set_list; the notmuch_config_get_* functions that deal with
> > lists are simply wrappers for these functions.

Updated version pushed. I'm not completely happy with _config_get_list's
weird API, but I don't have a better idea now.

d

• Previous message (by thread): Re: [PATCH] test: cli: getting/setting/removing config values

Thread:

• David Bremner—[RFC PATCH] cli: factor out config handling code to get/set lists. [inbox, notmuch::obsolete, notmuch::patch, notmuch::wip, unread]
  • Austin Clements—Re: [RFC PATCH] cli: factor out config handling code to get/set lists. [inbox, unread]
    • David Bremner—Re: [RFC PATCH] cli: factor out config handling code to get/set lists. [inbox, unread]
      • David Bremner—[PATCH] cli: factor out config handling code to get/set lists. [inbox, notmuch::patch, notmuch::pushed, unread]
        • Dmitry Kurochkin—Re: [PATCH] cli: factor out config handling code to get/set lists. [inbox, unread]
          • David Bremner—Re: [PATCH] cli: factor out config handling code to get/set lists. [inbox, unread]
            • Pieter Praet—Re: [PATCH] cli: factor out config handling code to get/set lists. [inbox, notmuch::review, unread]
              • Pieter Praet—[PATCH] test: cli: getting/setting/removing config values [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
                • David Bremner—Re: [PATCH] test: cli: getting/setting/removing config values [inbox, unread]
                  • Pieter Praet—Re: [PATCH] test: cli: getting/setting/removing config values [inbox, unread]
                    • Pieter Praet—[PATCH] test: cli: getting/setting/removing config values [inbox, notmuch::patch, notmuch::test, notmuch::wip, unread]
                      • David Bremner—Re: [PATCH] test: cli: getting/setting/removing config values [inbox, unread]
                        • Pieter Praet—Re: [PATCH] test: cli: getting/setting/removing config values [inbox, unread]
                          • David Bremner—Re: [PATCH] test: cli: getting/setting/removing config values [inbox, unread]
                            • Pieter Praet—Re: [PATCH] test: cli: getting/setting/removing config values [inbox, unread]