6b/5899fcb2a6eda017a9c0cc817e94479e108313

   1 Return-Path: <amdragon@mit.edu>\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 320D8429E47\r
   6         for <notmuch@notmuchmail.org>; Fri, 13 Jan 2012 14:54:55 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 CAYIdw8fZxIR for <notmuch@notmuchmail.org>;\r
  16         Fri, 13 Jan 2012 14:54:54 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-8.mit.edu (DMZ-MAILSEC-SCANNER-8.MIT.EDU\r
  18         [18.7.68.37])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 95A87429E3F\r
  20         for <notmuch@notmuchmail.org>; Fri, 13 Jan 2012 14:54:54 -0800 (PST)\r
  21 X-AuditID: 12074425-b7f4a6d0000008e0-5f-4f10b63d784b\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-8.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id FC.32.02272.D36B01F4; Fri, 13 Jan 2012 17:54:54 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id q0DMsrnc007316; \r
  27         Fri, 13 Jan 2012 17:54:53 -0500\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q0DMspwZ026059\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Fri, 13 Jan 2012 17:54:52 -0500 (EST)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1Rlq1L-00009w-BT; Fri, 13 Jan 2012 17:54:51 -0500\r
  37 Date: Fri, 13 Jan 2012 17:54:51 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Xavier Maillard <xma@gnu.org>\r
  40 Subject: Re: [PATCH] Set fill column to 70 in .dir-locals.el and refactor\r
  41         other settings.\r
  42 Message-ID: <20120113225451.GK18625@mit.edu>\r
  43 References: <1326410243-22277-1-git-send-email-amdragon@mit.edu>\r
  44         <m2lipb6alj.fsf@kcals.intra.maillard.im>\r
  45         <m27h0v5imm.fsf@kcals.intra.maillard.im>\r
  46 MIME-Version: 1.0\r
  47 Content-Type: text/plain; charset=us-ascii\r
  48 Content-Disposition: inline\r
  49 In-Reply-To: <m27h0v5imm.fsf@kcals.intra.maillard.im>\r
  50 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  51 X-Brightmail-Tracker:\r
  52  H4sIAAAAAAAAA+NgFupjleLIzCtJLcpLzFFi42IR4hTV1rXbJuBv0HWR0eL6zZnMFo/a97M7\r
  53         MHm0TTPzeLbqFnMAUxSXTUpqTmZZapG+XQJXxubvfxkLXvNVTNz8i6mB8QJ3FyMnh4SAicT5\r
  54         Q2+YIGwxiQv31rN1MXJxCAnsY5R43reMEcLZwCixbeMeqMxJJomdEw6xQjhLGCUmzXkI5HBw\r
  55         sAioSpy7wQcyik1AQ2Lb/uWMILaIgKLEgU372EBsZgFpiW+/m5lAyoUFoiW6tmmDhHkFdCRe\r
  56         vLzEDDFyGqPE3P09LBAJQYmTM5+wQPRqSdz49xKsF2TO8n8cIGFOAVOJb+sOgI0XFVCRmHJy\r
  57         G9sERqFZSLpnIemehdC9gJF5FaNsSm6Vbm5iZk5xarJucXJiXl5qka6FXm5miV5qSukmRlBQ\r
  58         s7uo7mCccEjpEKMAB6MSD69kloC/EGtiWXFl7iFGSQ4mJVFelq1AIb6k/JTKjMTijPii0pzU\r
  59         4kOMEhzMSiK8UqZAOd6UxMqq1KJ8mJQ0B4uSOK+m1js/IYH0xJLU7NTUgtQimKwMB4eSBO9O\r
  60         kKGCRanpqRVpmTklCGkmDk6Q4TxAw7eA1PAWFyTmFmemQ+RPMepynFx75RyjEEtefl6qlDhv\r
  61         G0iRAEhRRmke3BxYMnrFKA70ljBvJUgVDzCRwU16BbSECWhJWQofyJKSRISUVAPjspqYmf+m\r
  62         pnw+8Cg2WHHitEOz+eJUnlxc4CCl9VJo0cZtjRt7Tcs/f77l+vjCTR/TOU1hD6xfBgkd6voy\r
  63         7feh6T29P0/dfbqxqWoPMKw/G9/ckJireGNuZKvD6VKXj08zd6Vc2apm++PKvD8B2z/es5ht\r
  64         O6Fa79a0BKlVblNiS6MlnI+su1pirsRSnJFoqMVcVJwIAKkjO+whAwAA\r
  65 Cc: notmuch@notmuchmail.org\r
  66 X-BeenThere: notmuch@notmuchmail.org\r
  67 X-Mailman-Version: 2.1.13\r
  68 Precedence: list\r
  69 List-Id: "Use and development of the notmuch mail system."\r
  70         <notmuch.notmuchmail.org>\r
  71 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  72         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  73 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  74 List-Post: <mailto:notmuch@notmuchmail.org>\r
  75 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  76 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  77         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  78 X-List-Received-Date: Fri, 13 Jan 2012 22:54:55 -0000\r
  79 \r
  80 Quoth Xavier Maillard on Jan 13 at 11:42 pm:\r
  81 > \r
  82 > This controls where comments and other text wraps.  70 is the default\r
  83 > value, so this simply returns it to the default for people who have\r
  84 > overridden it.  Most notmuch code already adheres to this.\r
  85 > ---\r
  86 > SO here is the patch (still sorry if I did it wrongly and very badly).\r
  87 > \r
  88 >  .dir-locals.el |   19 ++++++-------------\r
  89 >  1 files changed, 6 insertions(+), 13 deletions(-)\r
  90 > \r
  91 > diff --git a/.dir-locals.el b/.dir-locals.el\r
  92 > index aea630b..27f01c0 100644\r
  93 > --- a/.dir-locals.el\r
  94 > +++ b/.dir-locals.el\r
  95 > @@ -1,23 +1,16 @@\r
  96 > -; emacs local configuration settings for notmuch source\r
  97 > -; surmised by dkg on 2010-11-23 13:43:18-0500\r
  98 > -; amended by amdragon on 2011-06-06\r
  99 > +;; emacs local configuration settings for notmuch source\r
 100 > +;; surmised by dkg on 2010-11-23 13:43:18-0500\r
 101 > +;; amended by amdragon on 2011-06-06\r
 102 >  \r
 103 > -((c-mode\r
 104 > +((nil\r
 105 >    (indent-tabs-mode . t)\r
 106 >    (tab-width . 8)\r
 107 >    (c-basic-offset . 4)\r
 108 > +  (fill-column . 70))\r
 109 > + (c-mode\r
 110 >    (c-file-style . "linux"))\r
 111 >   (c++-mode\r
 112 > -  (indent-tabs-mode . t)\r
 113 > -  (tab-width . 8)\r
 114 > -  (c-basic-offset . 4)\r
 115 >    (c-file-style . "linux"))\r
 116 > - (emacs-lisp-mode\r
 117 > -  (indent-tabs-mode . t)\r
 118 > -  (tab-width . 8))\r
 119 >   (shell-mode\r
 120 > -  (indent-tabs-mode . t)\r
 121 > -  (tab-width . 8)\r
 122 > -  (sh-basic-offset . 4)\r
 123 >    (sh-indentation . 4))\r
 124 >   )\r
 125 \r
 126 You ammended it right, but actually the configuration is intentionally\r
 127 repeated for the various modes.  For some programming languages\r
 128 (notably Python), it's inappropriate to mess with the tab settings\r
 129 (especially indent-tabs-mode), hence I took a "whitelist" approach to\r
 130 be on the safe side, rather than using global settings and then\r
 131 enumerating the modes I happened to know would break.\r
 132 \r
 133 This is admittedly subtle.  There should probably be a comment in the\r
 134 file explaining this.\r