From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Karl Fogel Newsgroups: gmane.emacs.devel Subject: Re: Odd behavior when moving point over invisible text Date: Mon, 05 Jun 2023 12:51:20 -0500 Message-ID: <874jnlpzl3.fsf@red-bean.com> References: <87cz2ac390.fsf@gnu.org> <87y1kybd1a.fsf@red-bean.com> <83h6rm3vqc.fsf@gnu.org> Reply-To: Karl Fogel Mime-Version: 1.0 Content-Type: text/plain; format=flowed Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="24587"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Cc: matsl@gnu.org, emacs-devel@gnu.org To: Eli Zaretskii Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Mon Jun 05 19:52:25 2023 Return-path: Envelope-to: ged-emacs-devel@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1q6ENU-0006A2-HM for ged-emacs-devel@m.gmane-mx.org; Mon, 05 Jun 2023 19:52:25 +0200 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1q6EMa-0003OT-0c; Mon, 05 Jun 2023 13:51:28 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1q6EMX-0003O5-PY for emacs-devel@gnu.org; Mon, 05 Jun 2023 13:51:25 -0400 Original-Received: from sanpietro.red-bean.com ([45.79.25.59]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1q6EMW-00009t-7m; Mon, 05 Jun 2023 13:51:25 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=red-bean.com; s=202005newsp; h=Content-Type:MIME-Version:Message-ID:Date: Reply-To:References:In-Reply-To:Subject:Cc:To:From:Sender: Content-Transfer-Encoding:Content-ID:Content-Description; bh=jfdRfoPkala5Nj7qUAK3GmHDXR/bxKTob20tHRs5Hg0=; t=1685987482; x=1687197082; b=MZ/7FZryrZTmaWqOTuAC80tKuGeZjewPUOzgM49vP3dYf7+n/XW9RuI0ZnL5N82lKyCZAYqCvhW f2L0iKhg8a3PXy/0SbnTm4s+8t7FxNqulPhjt+BPNiU+n34fbTHyhPuVPDSBt/tGfCWqgVeyrG/O2 uvy2XAIEGstc/KJRV8nKg+AzFqHJYO/Gl+JV+DJYXxqfo/ar6ijgwDKLW5vBKx2kTkk3Ot18DkHLV I70jYgeHnaxEDLBZPAVUrAV7Peux61MGcIXzi3vbPcC6NVv26L/nmz9J6tGsaqaG0kY50cd7/zN9i ouKtW/pousUNPDVas1tdKPZY7DB5UJ4TD5fA==; Original-Received: from [12.106.183.66] (port=48613 helo=hummy) by sanpietro.red-bean.com with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1q6EMT-00AqZA-Ir; Mon, 05 Jun 2023 17:51:21 +0000 In-Reply-To: <83h6rm3vqc.fsf@gnu.org> (Eli Zaretskii's message of "Mon, 05 Jun 2023 16:05:31 +0300") Received-SPF: pass client-ip=45.79.25.59; envelope-from=kfogel@red-bean.com; helo=sanpietro.red-bean.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.devel:306654 Archived-At: On 05 Jun 2023, Eli Zaretskii wrote: >The point-adjustment feature needs to record the previous >position of point, to know in which direction to move it in >order to exit the invisible area: if point moved to a larger >buffer position and entered invisible text, point-adjustment >wants to move it further forward; if it moved towards lower >position, point-adjustment wants to move it back. The bug was >that last point position was recorded in a global variable, and >M-x enters recursive-edit, which clobbered that variable's value >with the value from the minibuffer. Thank you for the clear explanation, Eli. That all makes sense, as does your fix in commit 9a28600a9735. Best regards, -Karl