51/d7401cc27310ee3201f7f8b26426bc21d505d7

   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 C99FE431FB6\r
   6         for <notmuch@notmuchmail.org>; Mon, 27 Aug 2012 17:22:20 -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: -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 SMHoPbFlRzFJ for <notmuch@notmuchmail.org>;\r
  16         Mon, 27 Aug 2012 17:22:20 -0700 (PDT)\r
  17 Received: from dmz-mailsec-scanner-7.mit.edu (DMZ-MAILSEC-SCANNER-7.MIT.EDU\r
  18         [18.7.68.36])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 09ABE431FAE\r
  20         for <notmuch@notmuchmail.org>; Mon, 27 Aug 2012 17:22:19 -0700 (PDT)\r
  21 X-AuditID: 12074424-b7fa06d0000008c2-63-503c0f3a8b35\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-7.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 4C.C2.02242.A3F0C305; Mon, 27 Aug 2012 20:22:18 -0400 (EDT)\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 q7S0MHB8013804; \r
  27         Mon, 27 Aug 2012 20:22:18 -0400\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 q7S0MGe2007371\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Mon, 27 Aug 2012 20:22:17 -0400 (EDT)\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 1T69ZP-0001UV-Sw; Mon, 27 Aug 2012 20:22:15 -0400\r
  37 Date: Mon, 27 Aug 2012 20:22:15 -0400\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Michal Nazarewicz <mpn@google.com>\r
  40 Subject: Re: [PATCH] emacs: notmuch-search: fix faces\r
  41 Message-ID: <20120828002215.GB11179@mit.edu>\r
  42 References:\r
  43  <83882f4c8de5317e0184dc091a197b2341e5f015.1345725028.git.mina86@mina86.com>\r
  44         <87ipc4jcfv.fsf@servo.finestructure.net>        <xa1t3938ue0m.fsf@mina86.com>\r
  45 MIME-Version: 1.0\r
  46 Content-Type: text/plain; charset=us-ascii\r
  47 Content-Disposition: inline\r
  48 In-Reply-To: <xa1t3938ue0m.fsf@mina86.com>\r
  49 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  50 X-Brightmail-Tracker:\r
  51  H4sIAAAAAAAAA+NgFuphleLIzCtJLcpLzFFi42IR4hTV1rXitwkw6PqpZ7Fnn5fFq+997BbX\r
  52         b85kdmD2uHuay2PBplKPZ6tuMQcwR3HZpKTmZJalFunbJXBlnP18mLHgCX/FoXsz2RoYr/B0\r
  53         MXJySAiYSPxZ388KYYtJXLi3nq2LkYtDSGAfo8SmOSeYIJwNjBI7971lh3BOMkmcWtvGCOEs\r
  54         YZSYe2cKI0g/i4CqRMPtB+wgNpuAhsS2/cvB4iICahIX/zxmArGZBSIkpsz4CGYLC5hKzJu5\r
  55         mgXE5hXQkfj6egnU7kWMErtvfGeHSAhKnJz5hAWiWUvixr+XQM0cQLa0xPJ/HCBhTqDw/N83\r
  56         wHaJCqhITDm5jW0Co9AsJN2zkHTPQuhewMi8ilE2JbdKNzcxM6c4NVm3ODkxLy+1SNdcLzez\r
  57         RC81pXQTIyjQ2V1UdjA2H1I6xCjAwajEw/tipXWAEGtiWXFl7iFGSQ4mJVFeQx6bACG+pPyU\r
  58         yozE4oz4otKc1OJDjBIczEoivLEvgMp5UxIrq1KL8mFS0hwsSuK811Nu+gsJpCeWpGanphak\r
  59         FsFkZTg4lCR4g/mAhgoWpaanVqRl5pQgpJk4OEGG8wANNwGp4S0uSMwtzkyHyJ9iVJQS57UG\r
  60         SQiAJDJK8+B6YYnoFaM40CvCvN4gVTzAJAbX/QpoMBPQ4P2KIFcXlyQipKQaGON6xEzT7+rv\r
  61         EJZS59z+5c90S05v/assrzNiJTujPvBXTW9WNjRv76i7U9DYu9/GdIlXvdPqOQ6pGfPXLYjs\r
  62         tfR70rHH7+yUd0Xf369Sqfsj0rurwrziUOvZh/eD2+W35N22E5/pdrSd5ZDgGQM1Xx/jWtPc\r
  63         +DajBAHv81wq8Zsm/n8fZDVfiaU4I9FQi7moOBEA8Kwbzh8DAAA=\r
  64 Cc: notmuch@notmuchmail.org\r
  65 X-BeenThere: notmuch@notmuchmail.org\r
  66 X-Mailman-Version: 2.1.13\r
  67 Precedence: list\r
  68 List-Id: "Use and development of the notmuch mail system."\r
  69         <notmuch.notmuchmail.org>\r
  70 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  71         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  72 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  73 List-Post: <mailto:notmuch@notmuchmail.org>\r
  74 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  75 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  76         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  77 X-List-Received-Date: Tue, 28 Aug 2012 00:22:20 -0000\r
  78 \r
  79 Quoth Michal Nazarewicz on Aug 28 at  1:11 am:\r
  80 > > On Thu, Aug 23 2012, Michal Nazarewicz <mpn@google.com> wrote:\r
  81 > >> For some reason the faces do not get applied when 'face property is\r
  82 > >> used, but they work correctly with 'font-lock-face property.  This\r
  83 > >> commit changes notmuch-search to use the latter.\r
  84 > \r
  85 > Jameson Graef Rollins <jrollins@finestructure.net> writes:\r
  86 > > Hi, Michal.  Can you say a bit more about what's motivating this?  This\r
  87 > > feature seems to work fine for me, so I would like to understand what's\r
  88 > > not working for you.  I'm certainly no expert on emacs font handling,\r
  89 > > though, so maybe this is the proper thing to do.  Thanks.\r
  90 > \r
  91 > I'm not an expert either, but with this patch applied I see colours,\r
  92 > without this patch, I don't see colours, ie. everything is rendered\r
  93 > using the default face.\r
  94 > \r
  95 > I'm also not entirely sure if that's the correct way of doing things\r
  96 > since 'face seems to be working in other modes (most notably\r
  97 > notmuch-show).  By posting, I'm also hoping that someone more\r
  98 > experienced will maybe take a look to see what's going on here... ;)\r
  99 > \r
 100 > For some more info, in notmuch-show mode, I get-text-property of nil for\r
 101 > both 'face and 'font-lock-face, but in message-mode or in *scratch*\r
 102 > buffer I'm getting nil for 'font-lock-face but non-nil for 'face.  With\r
 103 > the patch, I'm getting the same non-nil for both 'face and\r
 104 > 'font-lock-face; without the patch, I'm getting nil for both.\r
 105 \r
 106 This is odd.  Could you give more details about your environment?  In\r
 107 particular, what Emacs version are you running and might you have any\r
 108 unusual Emacs packages installed or customizations set?\r
 109 \r
 110 The unusual thing about font-lock-face is that it only gets applied if\r
 111 font-lock is enabled; otherwise it's ignored.  One theory is that the\r
 112 nil font-lock-face (without your patch) is causing font-lock to\r
 113 *override* the face property that we set.  I don't think font-lock\r
 114 usually does that, but perhaps in some situations, it will?\r