f9/ccf77a9847e64c5eaf832d475281f3008e0016

   1 Return-Path: <tomi.ollila@nixu.com>\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 4B9B5429E5D\r
   6         for <notmuch@notmuchmail.org>; Tue, 17 Jan 2012 02:46:29 -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\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 ijxcjlqsrn0Y for <notmuch@notmuchmail.org>;\r
  16         Tue, 17 Jan 2012 02:46:28 -0800 (PST)\r
  17 Received: from mail-gw3.nixu.fi (mail-gw3.nixu.fi [193.209.237.7])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 4E165429E5C\r
  21         for <notmuch@notmuchmail.org>; Tue, 17 Jan 2012 02:46:28 -0800 (PST)\r
  22 Received: from pps.filterd (mail-gw3 [127.0.0.1])\r
  23         by mail-gw3.nixu.fi (8.14.4/8.14.4) with SMTP id q0HAkK9A022375;\r
  24         Tue, 17 Jan 2012 12:46:24 +0200\r
  25 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  26         by mail-gw3.nixu.fi with ESMTP id 114cs17p1b-1\r
  27         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  28         Tue, 17 Jan 2012 12:46:24 +0200\r
  29 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  30         by taco2.nixu.fi (8.14.3/8.14.3/Debian-5+lenny1) with ESMTP id\r
  31         q0HAkNK5008204; Tue, 17 Jan 2012 12:46:23 +0200\r
  32 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  33 To: Austin Clements <amdragon@mit.edu>\r
  34 Subject: Re: [PATCH 2/3] emacs: whitespace-cleanup and indent-region for\r
  35         emacs/*.el files\r
  36 In-Reply-To: <20120117043200.GJ16740@mit.edu>\r
  37 References: <cun7h0v7ete.fsf@hotblack-desiato.hh.sledj.net>\r
  38         <1326704656-26102-1-git-send-email-tomi.ollila@iki.fi>\r
  39         <1326704656-26102-2-git-send-email-tomi.ollila@iki.fi>\r
  40         <20120117043200.GJ16740@mit.edu>\r
  41 User-Agent: Notmuch/0.11+64~g42e8f66 (http://notmuchmail.org) Emacs/23.3.1\r
  42         (i686-pc-linux-gnu)\r
  43 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  44         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  45         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  46 Date: Tue, 17 Jan 2012 12:46:23 +0200\r
  47 Message-ID: <yf6obu261z4.fsf@taco2.nixu.fi>\r
  48 MIME-Version: 1.0\r
  49 Content-Type: text/plain; charset=us-ascii\r
  50 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.5.7110,\r
  51  1.0.211,       0.0.0000        definitions=2012-01-17_02:2012-01-17, 2012-01-17,\r
  52         1970-01-01 signatures=0\r
  53 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0\r
  54         ipscore=0 suspectscore=0\r
  55         phishscore=0 bulkscore=0 adultscore=0 classifier=spam adjust=0\r
  56         reason=mlx\r
  57         scancount=1 engine=6.0.2-1012030000 definitions=main-1201170044\r
  58 Cc: notmuch@notmuchmail.org\r
  59 X-BeenThere: notmuch@notmuchmail.org\r
  60 X-Mailman-Version: 2.1.13\r
  61 Precedence: list\r
  62 List-Id: "Use and development of the notmuch mail system."\r
  63         <notmuch.notmuchmail.org>\r
  64 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  66 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  67 List-Post: <mailto:notmuch@notmuchmail.org>\r
  68 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  69 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  71 X-List-Received-Date: Tue, 17 Jan 2012 10:46:29 -0000\r
  72 \r
  73 On Mon, 16 Jan 2012 23:32:00 -0500, Austin Clements <amdragon@mit.edu> wrote:\r
  74 > Cleanup is the type of pain that should only be suffered once, so I'd\r
  75 > be much happier with this if there was an accompanying git hook that\r
  76 > prevented more mis-formatted code from slipping in.\r
  77 \r
  78 We'd need a script to be called from .git/hooks/pre-commit to do\r
  79 extra checking; developer needs first activate this pre-commit\r
  80 and then add call to our checking routine. Imagine the amount\r
  81 of false positives this hook starts to generate...\r
  82 \r
  83 ... but. developer can run 'git commit --no-verify' ...aargh no;\r
  84 I guess if pre-commit hook fails, commit-msg hook is not run\r
  85 and this is bypassed; maybe NO_FORMATCHECK_HOOK=1 git commit ...\r
  86 is the answer.\r
  87 \r
  88 But what we at least need is Guidelines document that states \r
  89 these formatting issues clearly and precicely. Surely \r
  90 self-respecting programmers understands to follow there (and soon \r
  91 adjusts their workflow -- i.e. activate/run these checkers\r
  92 after list response).\r
  93 \r
  94 > Quoth Tomi Ollila on Jan 16 at 11:04 am:\r
  95 > > diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el\r
  96 > > index 00ed589..7b63348 100644\r
  97 > > --- a/emacs/notmuch-lib.el\r
  98 > > +++ b/emacs/notmuch-lib.el\r
  99 > > @@ -143,7 +143,7 @@ The result is a new function which does the same as FUN, except that\r
 100 > >  the first N arguments are fixed at the values with which this function\r
 101 > >  was called."\r
 102 > >     (lexical-let ((fun fun) (args1 args))\r
 103 > > -     (lambda (&rest args2) (apply fun (append args1 args2))))))\r
 104 > > +           (lambda (&rest args2) (apply fun (append args1 args2))))))\r
 105 > \r
 106 > Ack.  What happened here?  This shows up in at least one other place,\r
 107 > too.  I'm betting you need to load cl in some form to get the\r
 108 > indentation rule for lexical-let.\r
 109 \r
 110 Yes, thanks. (require 'cl) fixes that. I'll mark this and the RFC patch\r
 111 (of batch-tools.el) obsolete. \r
 112 \r
 113 Tomi\r