24/91aa9f408ef5eac1d7d6fbea6ba50da3f50791

   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 05E34431FAF\r
   6         for <notmuch@notmuchmail.org>; Thu, 17 May 2012 07:26:45 -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 XnQYg95OrMkj for <notmuch@notmuchmail.org>;\r
  16         Thu, 17 May 2012 07:26:43 -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 4F528431FAE\r
  20         for <notmuch@notmuchmail.org>; Thu, 17 May 2012 07:26:43 -0700 (PDT)\r
  21 Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by fire-doxen-postvirus (Postfix) with ESMTP id B6A362E50D4C;\r
  23         Thu, 17 May 2012 07:26:38 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on fire-doxen by amavisd-new\r
  25 Received: from finestructure.net (cpe-98-14-81-93.nyc.res.rr.com\r
  26  [98.14.81.93]) (Authenticated sender: jrollins)        by fire-doxen-submit\r
  27  (Postfix) with ESMTP id D977A2E50CC1;  Thu, 17 May 2012 07:26:35 -0700 (PDT)\r
  28 Received: by finestructure.net (Postfix, from userid 1000)\r
  29         id D03404FD; Thu, 17 May 2012 07:26:32 -0700 (PDT)\r
  30 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  31 To: Jani Nikula <jani@nikula.org>, Notmuch Mail <notmuch@notmuchmail.org>\r
  32 Subject: Re: [PATCH 4/6] cli: intialize crypto structure in show and reply\r
  33 In-Reply-To: <8762bvi70k.fsf@nikula.org>\r
  34 References: <1337205359-2444-1-git-send-email-jrollins@finestructure.net>\r
  35         <1337205359-2444-2-git-send-email-jrollins@finestructure.net>\r
  36         <1337205359-2444-3-git-send-email-jrollins@finestructure.net>\r
  37         <1337205359-2444-4-git-send-email-jrollins@finestructure.net>\r
  38         <1337205359-2444-5-git-send-email-jrollins@finestructure.net>\r
  39         <8762bvi70k.fsf@nikula.org>\r
  40 User-Agent: Notmuch/0.12+183~g9d5ff3c (http://notmuchmail.org) Emacs/23.4.1\r
  41         (x86_64-pc-linux-gnu)\r
  42 Date: Thu, 17 May 2012 07:26:30 -0700\r
  43 Message-ID: <877gwaeve1.fsf@servo.finestructure.net>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: multipart/signed; boundary="=-=-=";\r
  46         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  47 X-BeenThere: notmuch@notmuchmail.org\r
  48 X-Mailman-Version: 2.1.13\r
  49 Precedence: list\r
  50 List-Id: "Use and development of the notmuch mail system."\r
  51         <notmuch.notmuchmail.org>\r
  52 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  54 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  55 List-Post: <mailto:notmuch@notmuchmail.org>\r
  56 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  57 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  59 X-List-Received-Date: Thu, 17 May 2012 14:26:45 -0000\r
  60 \r
  61 --=-=-=\r
  62 \r
  63 On Thu, May 17 2012, Jani Nikula <jani@nikula.org> wrote:\r
  64 > On Thu, 17 May 2012, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
  65 >> This makes sure it has proper initialization values when it's created.\r
  66 >\r
  67 > Please don't do this. It's unnecessary; if one field is initialized with\r
  68 > a designated initializer, the rest are initialized to zero (or NULL).\r
  69 \r
  70 It may be technically unnecessary, but why is that a reason to not do\r
  71 it?  I intentionally did this to make it clear what the defaults are.\r
  72 Otherwise the defaults are essentially undefined, which is not good.\r
  73 Maybe the structure initializes to the correct defaults, but why count\r
  74 on that when we can set them to the correct default, and have it clear\r
  75 to readers of the code?\r
  76 \r
  77 jamie.\r
  78 \r
  79 --=-=-=\r
  80 Content-Type: application/pgp-signature\r
  81 \r
  82 -----BEGIN PGP SIGNATURE-----\r
  83 Version: GnuPG v1.4.12 (GNU/Linux)\r
  84 \r
  85 iQIcBAEBCAAGBQJPtQqWAAoJEO00zqvie6q87lMP/1dNXTVFfvyNJVT65AVie7bj\r
  86 rinWFVmi2qY/biIM2JUtAa9HDgD8CdVgG+oA/DNrlgpm72Ommexgl8CvvW2StZVX\r
  87 IMnFGMXiwjmToLPJq0jL8b0Bq2IVNUi7K7yLDhh04Fv64EMWCYF58aYj5tJ9dA8N\r
  88 HkKEyuYZoNc3xNqgjDTVSSKI0tjmGUFNaEQAvFvucdXJ4Ca8dHsYrpg7HD35d/0J\r
  89 Q1sRj6wlEkmTgmxbkNBx6NRnSCGVv/QhnK2aRpjIk0EL7vdE4ZA1IzuY73OQWLvn\r
  90 QMQRacmAAHZkyVmJlaKmTdUx4Z7AdeGrYLTvf/LDNME9bfJmIbykDTRMqZ7k+KZ6\r
  91 vqqfvvP2HZJePWSAhTnB7Rb9Fl0lDylgRqdq6wtu/L12rmwhOGVvAQqUmTerKe1E\r
  92 ugEnb/rhZ/KcNYXFS4G9dV6I4SD4sIyW14380ZiKur6vwpcA3rPDwrFNeX+C6bEH\r
  93 Jmob8Q9WfqjhPUDuG6ngSOSx0+D1pABPALCimSdalKtGj/hhas/L8kkj8MMHCN00\r
  94 S98fwle+fo8uJFAMjeaE3XQ0WJPD8AdH/nBGL6EzNb9liUlH2cygTs0zD8wRutUv\r
  95 Z717CXKeoolz0ls8hm3a7Poab1DIw6Oi5OzYxXNy6TmOmoI7dJXUemSfUTiR77mk\r
  96 BbH8yDE64UlYFNYyExkF\r
  97 =hE/w\r
  98 -----END PGP SIGNATURE-----\r
  99 --=-=-=--\r