a9/70e0e3ba67bacd0ade6b95e3faeb5e74dd70e0

   1 Return-Path: <cworth@cworth.org>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 36E07431FC2;\r
   6         Wed,  2 Dec 2009 13:42:08 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id LU6r3q37I1Po; Wed,  2 Dec 2009 13:42:07 -0800 (PST)\r
  11 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  12         by olra.theworths.org (Postfix) with ESMTP id 4E87A431FAE;\r
  13         Wed,  2 Dec 2009 13:42:07 -0800 (PST)\r
  14 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  15         id 0830B2542A9; Wed,  2 Dec 2009 13:42:07 -0800 (PST)\r
  16 From: Carl Worth <cworth@cworth.org>\r
  17 To: Jan Janak <jan@ryngle.com>, Bart Trojanowski <bart@jukie.net>\r
  18 In-Reply-To: <20091125215008.GG25119@ryngle.com>\r
  19 References: <1259100630-13673-1-git-send-email-jan@ryngle.com>\r
  20         <1259100630-13673-2-git-send-email-jan@ryngle.com>\r
  21         <1259100630-13673-3-git-send-email-jan@ryngle.com>\r
  22         <20091125205639.GJ29312@jukie.net> <20091125215008.GG25119@ryngle.com>\r
  23 Date: Wed, 02 Dec 2009 13:42:06 -0800\r
  24 Message-ID: <87vdgp834h.fsf@yoom.home.cworth.org>\r
  25 MIME-Version: 1.0\r
  26 Content-Type: multipart/signed; boundary="=-=-=";\r
  27         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  28 Cc: notmuch@notmuchmail.org\r
  29 Subject: Re: [notmuch] [PATCH 2/4] notmuch: Config option to specify tags       to\r
  30         be applied by 'notmuch new'.\r
  31 X-BeenThere: notmuch@notmuchmail.org\r
  32 X-Mailman-Version: 2.1.12\r
  33 Precedence: list\r
  34 List-Id: "Use and development of the notmuch mail system."\r
  35         <notmuch.notmuchmail.org>\r
  36 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  37         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  38 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  39 List-Post: <mailto:notmuch@notmuchmail.org>\r
  40 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  41 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  42         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  43 X-List-Received-Date: Wed, 02 Dec 2009 21:42:08 -0000\r
  44 \r
  45 --=-=-=\r
  46 Content-Transfer-Encoding: quoted-printable\r
  47 \r
  48 On Wed, 25 Nov 2009 22:50:08 +0100, Jan Janak <jan@ryngle.com> wrote:\r
  49 > On 25-11 15:56, Bart Trojanowski wrote:\r
  50 > > I really want this feature to get in, so I am going to do my best to\r
  51 > > review your code :)\r
  52 > >=20\r
  53 > > Here are some more sticking points...\r
  54 > >=20\r
  55 > > > +char **\r
  56 > > > +notmuch_config_get_new_tags (notmuch_config_t *config, size_t *lengt=\r
  57 h);\r
  58 > >=20\r
  59 > > If you are not giving over control of the pointer to the caller please\r
  60 > > return const char * const *.\r
  61 >=20\r
  62 > I followed Carl's style there, in particular the following function:\r
  63 > notmuch_config_get_user_other_email\r
  64 >\r
  65 > I can, of course, change that. But maybe we should wait for Carl to see\r
  66 > which way he prefers.\r
  67 \r
  68 Call me stupid. Anything more complex than "const char *" and I can\r
  69 never figure out where to put the const to express what's desired.\r
  70 \r
  71 If "const char * const *" means what we want it to, then feel free to\r
  72 return that, (and fix up existing cases). But maybe add a little block\r
  73 of example code to the documentation of the function so that idiots like\r
  74 me can still figure out how to declare the right kind of variable to\r
  75 call the function.\r
  76 \r
  77 =2DCarl\r
  78 \r
  79 --=-=-=\r
  80 Content-Type: application/pgp-signature\r
  81 \r
  82 -----BEGIN PGP SIGNATURE-----\r
  83 Version: GnuPG v1.4.10 (GNU/Linux)\r
  84 \r
  85 iD8DBQFLFt8u6JDdNq8qSWgRAiVSAJ0WmcYUtE020m1WUBWKPooq94PE+wCeKuPl\r
  86 qfMsQF+RnwRTmGkwq0y9EiQ=\r
  87 =mKYy\r
  88 -----END PGP SIGNATURE-----\r
  89 --=-=-=--\r