From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Michael Heerdegen Newsgroups: gmane.emacs.bugs Subject: bug#23079: 25.0.92; Movement commands leave cursor in invisible line Date: Tue, 22 Mar 2016 17:48:24 +0100 Message-ID: <87zitqihlz.fsf@web.de> References: <87oaa77fev.fsf@web.de> <8737rj79cp.fsf@web.de> <871t72cxhh.fsf@web.de> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1458665366 17286 80.91.229.3 (22 Mar 2016 16:49:26 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 22 Mar 2016 16:49:26 +0000 (UTC) Cc: 23079@debbugs.gnu.org, jonas@bernoul.li To: Stefan Monnier Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Tue Mar 22 17:49:14 2016 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1aiPUY-0006JJ-8F for geb-bug-gnu-emacs@m.gmane.org; Tue, 22 Mar 2016 17:49:14 +0100 Original-Received: from localhost ([::1]:38428 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aiPUX-0000at-Rs for geb-bug-gnu-emacs@m.gmane.org; Tue, 22 Mar 2016 12:49:13 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:48558) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aiPUR-0000aj-EZ for bug-gnu-emacs@gnu.org; Tue, 22 Mar 2016 12:49:11 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aiPUM-0001eL-FU for bug-gnu-emacs@gnu.org; Tue, 22 Mar 2016 12:49:07 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:34870) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aiPUM-0001eH-BR for bug-gnu-emacs@gnu.org; Tue, 22 Mar 2016 12:49:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1aiPUM-0002km-1U for bug-gnu-emacs@gnu.org; Tue, 22 Mar 2016 12:49:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Michael Heerdegen Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 22 Mar 2016 16:49:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 23079 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: notabug Original-Received: via spool by 23079-submit@debbugs.gnu.org id=B23079.145866531910553 (code B ref 23079); Tue, 22 Mar 2016 16:49:01 +0000 Original-Received: (at 23079) by debbugs.gnu.org; 22 Mar 2016 16:48:39 +0000 Original-Received: from localhost ([127.0.0.1]:60230 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1aiPTz-0002k9-Ig for submit@debbugs.gnu.org; Tue, 22 Mar 2016 12:48:39 -0400 Original-Received: from mout.web.de ([212.227.15.4]:53318) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1aiPTy-0002jy-8N for 23079@debbugs.gnu.org; Tue, 22 Mar 2016 12:48:38 -0400 Original-Received: from drachen.dragon ([94.218.210.27]) by smtp.web.de (mrweb004) with ESMTPSA (Nemesis) id 0MNx4F-1agTOi1UNP-007R2c; Tue, 22 Mar 2016 17:48:27 +0100 In-Reply-To: (Stefan Monnier's message of "Tue, 22 Mar 2016 12:16:12 -0400") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.92 (gnu/linux) X-Provags-ID: V03:K0:JYQKjhQFUmSh5K0xLSzddfL1a+Jx7t0tJnWS+IU466gjLjsA+N7 Bwr2NQzK3vW4lrOLUIu9qwTnYrU8+gxKL5dGWWoPpkpP6oCetrDcsdtSzBQXg/MT7+rTV3o Bo2EbN9IoniZRmpxKy4uuc5HIZjoDoNUb5kOh1xIgNIKOyYfurQu2AQQO35zYckOv5tcdQ0 rdAk1AQyd+xLEQpTN1XRQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:UEVPje12GcU=:J1cy76blfb6rgdBXtnUSiG pJT3Yx+57UrX16MBT9SXyx3Y7ub/ZcphWrGM20CxDRLCJIEktCon7lkMR6KNec2ymhSQB4q9K eZnc3NGixtVAMKs6m02Y4hMUbi1eyKnvZqFNcBFfvCiakk9CtoQ8SP9XCxv6xtYVX2khNm0Ee RQs/wT0eXF9xcF2U3c0OsRhA/SEIQ03yk9VtVCES6X3BXeeiDFUpLdTd5Ltg+h2IPptnw5T31 TGVwWDJ4rtcKVe8GwqBu8dJTAqd3YrJLwfNFUThvrxNxFqpIHrbkduvWQkY2bZ23PUIngY5g3 F43mZ9UEMldcUnMRKc0/n5Cydzi6/qAzTqFiCnxJ3eGLRy/5G7qYK2HvrqW5Uh4JGzS593iSU Y94VCy9QuG927g1faPhVLTuh8CDh0I9kQ6QXWGcOXfjbed5i0jf2cCxYi7RTsAgdweYTRDfCN UXmVNLCNnoJIiqewBf7dcpyx/Cvqs8kAB0oN7e7qe7I//Q+U17sZYh9aTm52srXOcyF0rr3AT 0Utz5zSPYpP9SOdXucY8lo27hhPf4vDA/fOs0fJF3CMrrKpZVRYG1nvGXEDVxec2s30UJ6/Wk l46DfIFSaDfZ2/D8WjSyX4/cPMlwrM6q4rFMYSPVq6aDw0Plmjb2l/Kxdwi1bQmOj4I6HzD/u G7/19aTcE9o71XQZoHLevxzR8vWq/gCQ2YatU//O6sR3ufRBzPtXUM7VTL5MMpyc4kv3oYI0N LJjBC+o6GJR0Hwek086FNgI4ni9gEYdlv9rZfIa1l73hzCLUv9Th4H9BPm2LDx6mtHiF6AFj X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:115335 Archived-At: Stefan Monnier writes: > The default "stickiness" of overlay boundaries is the inverse of that if > text-properties, for some reason. So, this test above suffers from the > bug#19200, but other than that, it seems to work correctly for me > (i.e. it prefers putting point in front of "333" rather than in front > of "555"). > > And if I remove the "nil t t" args to make-overlay, then > point-adjustment correctly tries to put point in front of "555" instead > of putting it in front of "333". Ok, we are coming closer... They use the default stickiness for their overlays. Nonetheless, the behavior is the same as with "nil t t". _But_ their local binding of post-command-hook looks like (t magit-section-update-highlight). When I set it to (t) only, the issue is fixed (but the highlighting update doesn't work anymore, of course)! magit-section-update-highlight is #+begin_src emacs-lisp (defun magit-section-update-highlight () (let ((section (magit-current-section))) (unless (eq section magit-section-highlighted-section) (let ((inhibit-read-only t) (deactivate-mark nil) (selection (magit-region-sections))) (mapc #'delete-overlay magit-section-highlight-overlays) (setq magit-section-unhighlight-sections magit-section-highlighted-sections magit-section-highlighted-sections nil) (unless (eq section magit-root-section) (run-hook-with-args-until-success 'magit-section-highlight-hook section selection)) (--each magit-section-unhighlight-sections (run-hook-with-args-until-success 'magit-section-unhighlight-hook it selection)) (restore-buffer-modified-p nil) (unless (eq magit-section-highlighted-section section) (setq magit-section-highlighted-section (unless (magit-section-hidden section) section)))) (setq deactivate-mark nil)))) #+end_src This function doesn't move point, but it somehow interferes with point adjustment in another way. Michael