2d/ea15ce5771f765cc6f41ce590826c921608c8d

   1 Return-Path: <jrollins@finestructure.net>\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 A080A431FAF\r
   6         for <notmuch@notmuchmail.org>; Thu, 17 May 2012 09:45:17 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id GchX7x2KvnRx for <notmuch@notmuchmail.org>;\r
  16         Thu, 17 May 2012 09:45:15 -0700 (PDT)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id C0B73431FAE\r
  20         for <notmuch@notmuchmail.org>; Thu, 17 May 2012 09:45:15 -0700 (PDT)\r
  21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by earth-doxen-postvirus (Postfix) with ESMTP id A411B66E01D3;\r
  23         Thu, 17 May 2012 09:45:08 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new\r
  25 Received: from finestructure.net (rrcs-24-103-26-131.nyc.biz.rr.com\r
  26         [24.103.26.131]) (Authenticated sender: jrollins)\r
  27         by earth-doxen-submit (Postfix) with ESMTP id 927D866E018C;\r
  28         Thu, 17 May 2012 09:45:05 -0700 (PDT)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id B1120340; Thu, 17 May 2012 09:45:03 -0700 (PDT)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Jani Nikula <jani@nikula.org>\r
  33 Subject: Re: [PATCH 4/6] cli: intialize crypto structure in show and reply\r
  34 In-Reply-To:\r
  35  <CAB+hUn9DdeaFj-hUNb_c1V3QLsbWjsE7_hpuOpDqWseayASdKQ@mail.gmail.com>\r
  36 References: <1337205359-2444-1-git-send-email-jrollins@finestructure.net>\r
  37         <1337205359-2444-2-git-send-email-jrollins@finestructure.net>\r
  38         <1337205359-2444-3-git-send-email-jrollins@finestructure.net>\r
  39         <1337205359-2444-4-git-send-email-jrollins@finestructure.net>\r
  40         <1337205359-2444-5-git-send-email-jrollins@finestructure.net>\r
  41         <8762bvi70k.fsf@nikula.org>\r
  42         <877gwaeve1.fsf@servo.finestructure.net>\r
  43         <CAB+hUn9DdeaFj-hUNb_c1V3QLsbWjsE7_hpuOpDqWseayASdKQ@mail.gmail.com>\r
  44 User-Agent: Notmuch/0.12+183~g9d5ff3c (http://notmuchmail.org) Emacs/23.4.1\r
  45         (x86_64-pc-linux-gnu)\r
  46 Date: Thu, 17 May 2012 09:45:01 -0700\r
  47 Message-ID: <87aa16daeq.fsf@servo.finestructure.net>\r
  48 MIME-Version: 1.0\r
  49 Content-Type: multipart/signed; boundary="=-=-=";\r
  50         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  51 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  52 X-BeenThere: notmuch@notmuchmail.org\r
  53 X-Mailman-Version: 2.1.13\r
  54 Precedence: list\r
  55 List-Id: "Use and development of the notmuch mail system."\r
  56         <notmuch.notmuchmail.org>\r
  57 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  59 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  60 List-Post: <mailto:notmuch@notmuchmail.org>\r
  61 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  62 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  64 X-List-Received-Date: Thu, 17 May 2012 16:45:17 -0000\r
  65 \r
  66 --=-=-=\r
  67 \r
  68 On Thu, May 17 2012, Jani Nikula <jani@nikula.org> wrote:\r
  69 > The values are not undefined, they are properly initialized, and we can\r
  70 > count on it. For sure, not maybe. If you want to explicitly set them for\r
  71 > clarity, it's a matter of taste. Personally I find it too verbose, but then\r
  72 > again notmuch code is generally fairly verbose.\r
  73 \r
  74 I want them explicitly set for clarity, as well as safety.  Code is\r
  75 meant to be read by humans, not computers.  Brevity is not always a\r
  76 virtue if it sacrifices clarity.  It's much nicer to have the defaults\r
  77 clearly stated in the initialization, than to force the reader to\r
  78 understand how the initialization works and to interpret what that means\r
  79 for the current case.  I also don't think it's safe to assume that the\r
  80 variables will be always be "properly" initialized in your favor in\r
  81 perpetuity.  It's much safer to explicitly set them to what you want\r
  82 them to be rather than just assume they'll be set correctly.\r
  83 \r
  84 > If you insist on it, please at least drop the extra temp crypto\r
  85 > variable, and initialize the struct in one initializer.\r
  86 \r
  87 I don't see why this matters either.  Again, I think this is just a\r
  88 matter of taste.  I would rather the code be verbose where clarity\r
  89 requires it, rather than always trying to make the code as terse as\r
  90 possible.\r
  91 \r
  92 jamie.\r
  93 \r
  94 --=-=-=\r
  95 Content-Type: application/pgp-signature\r
  96 \r
  97 -----BEGIN PGP SIGNATURE-----\r
  98 Version: GnuPG v1.4.12 (GNU/Linux)\r
  99 \r
 100 iQIcBAEBCAAGBQJPtSsNAAoJEO00zqvie6q8EUAQAJnXcl3gyfVqUNY0c6BCOn1f\r
 101 EIk+M5rz0gXAhJLo/k2yUuk4SuGiF0K20ZeUyy9Vf59y5LnazJsgsirzM4LqdZ0B\r
 102 UxlBmlNz7K7MbIB3X3qOAc/PA2V2zRbSNMuO7yhr8l2Q7/E82tEL+2cIBNpplWVM\r
 103 zAXPp2UI+j7fgHuYjwYt4Rr5qaqb5VR4Ix6qQP5ErjEQFDDm/Q3RAU9no+mP6XHQ\r
 104 8yWHvYBaOaMdPOCAuPIl5UmFfuDsQ1wxBB5sLH9omkzzIQn1VynTqq7Is2PLYwm8\r
 105 0RR/NbblYKJmHpnny3Seabzv+6J3DD7ZmeW7K3fHEBHs582VwaojS+GRvZJrzDJ+\r
 106 UbDDxRDyMvKxr4Q1QH13doqcJ8p4en3HpFanY2mGIR4je/b295b9nUZnuFh2e5Un\r
 107 FoPim91THuUhFabIw6nNWZN7j2qCdcMwalTYql++Xi4Pb3fAWZNSgrXnN+nojxWy\r
 108 9dTsKLFD3Xv/Ae7/CinebSQUV7oBngvfRvmtTR8E3F39zLl9ZN0XD576Y3GwMFT7\r
 109 w6wj1hLP1dTjIvlhaLynwsUcYDK4Veq9nGGe4RKYhEXQAHaNOeAHdBVPIR2JX3+Z\r
 110 Y/rU817v3mU7dR7ddB4b7/K5j8J/65DDQK4FX3XEqacdJpgsbRM806mtjx/CvAG5\r
 111 mRST4o7lxkZoe0UbWUk9\r
 112 =mD9j\r
 113 -----END PGP SIGNATURE-----\r
 114 --=-=-=--\r