From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rasmus Subject: Re: Avoiding `intangible' Date: Thu, 16 Apr 2015 11:50:31 +0200 Message-ID: <87r3rk9zbs.fsf@gmx.us> References: Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:50962) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YigRd-0006qP-Q4 for emacs-orgmode@gnu.org; Thu, 16 Apr 2015 05:50:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YigRb-0003dF-2Y for emacs-orgmode@gnu.org; Thu, 16 Apr 2015 05:50:49 -0400 Received: from plane.gmane.org ([80.91.229.3]:38987) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YigRa-0003d0-S6 for emacs-orgmode@gnu.org; Thu, 16 Apr 2015 05:50:47 -0400 Received: from list by plane.gmane.org with local (Exim 4.69) (envelope-from ) id 1YigRS-0001yL-SC for emacs-orgmode@gnu.org; Thu, 16 Apr 2015 11:50:38 +0200 Received: from tsn109-201-154-150.dyn.nltelcom.net ([109.201.154.150]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Thu, 16 Apr 2015 11:50:38 +0200 Received: from rasmus by tsn109-201-154-150.dyn.nltelcom.net with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Thu, 16 Apr 2015 11:50:38 +0200 List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org To: emacs-orgmode@gnu.org Cc: monnier@iro.umontreal.ca Hi Stefan, Thanks for the patch. Stefan Monnier writes: > I'm on a crusade to eradicate the `intangible' property, which happens > to rub me the wrong way because it's implemented at too-low a level (it > affects every point movement) which incurs a significant performance > penalty (even when not used) and affects a lot of code which then > needs to be fixed by binding inhibit-point-motion-hooks (which I'm > considering defaulting to t). OK. In addition to the places you've identified, it's also mentioned here: ./contrib/lisp/org-colview-xemacs.el:396: (overlay-put ov 'intangible t) ./contrib/lisp/org-eval.el:154: (list 'display value 'intangible t 'font-lock-multiline t ./lisp/org.el:11165: (unless (get-text-property (match-end group) 'intangible) ; Emacs 21 > So I'd like to install the patch below. Any objection? The `invisible' > property already ensures that the command loop moves point away from > those overlays, so the `intangible' property should not be needed there > (tho I don't use Org enough to know how to test it). I can't even make the first modified function do what it is documented to do. From the special text properties documentation and the documentation of the changed I don't see a reason why it would not be OK. I'll install it in master and it will eventually get back to Emacs-core once 8.3 released assuming no unforeseen breakages are associated with the change. Thanks, Rasmus -- Not everything that goes around comes back around, you know