From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.bugs Subject: bug#22250: 25.0.50; Eww fails to break RTL paragraph Date: Tue, 29 Dec 2015 18:47:43 +0200 Message-ID: <83io3hdwwg.fsf@gnu.org> References: <8760zj3dvr.fsf@justinian.i-did-not-set--mail-host-address--so-tickle-me> <83twn3g0ac.fsf@gnu.org> <8760zjbi9s.fsf@justinian.turtle-trading.net> <83k2nzfdt9.fsf@gnu.org> <87twn2il10.fsf@justinian.turtle-trading.net> <83bn9afqfz.fsf@gnu.org> <87k2nyigmh.fsf@justinian.turtle-trading.net> <8360zifmsk.fsf@gnu.org> <8737umi7xw.fsf@justinian.turtle-trading.net> Reply-To: Eli Zaretskii NNTP-Posting-Host: plane.gmane.org X-Trace: ger.gmane.org 1451407705 7164 80.91.229.3 (29 Dec 2015 16:48:25 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 29 Dec 2015 16:48:25 +0000 (UTC) Cc: 22250@debbugs.gnu.org, larsi@gnus.org To: Benjamin Riefenstahl Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Tue Dec 29 17:48:15 2015 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 1aDxRV-0003XI-Jt for geb-bug-gnu-emacs@m.gmane.org; Tue, 29 Dec 2015 17:48:13 +0100 Original-Received: from localhost ([::1]:49227 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aDxRV-0001Ak-40 for geb-bug-gnu-emacs@m.gmane.org; Tue, 29 Dec 2015 11:48:13 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:35686) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aDxRO-00015I-G2 for bug-gnu-emacs@gnu.org; Tue, 29 Dec 2015 11:48:10 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aDxRK-0006tV-Ed for bug-gnu-emacs@gnu.org; Tue, 29 Dec 2015 11:48:06 -0500 Original-Received: from debbugs.gnu.org ([208.118.235.43]:41217) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aDxRK-0006tR-BD for bug-gnu-emacs@gnu.org; Tue, 29 Dec 2015 11:48:02 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84) (envelope-from ) id 1aDxRK-0000gP-77 for bug-gnu-emacs@gnu.org; Tue, 29 Dec 2015 11:48:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Eli Zaretskii Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 29 Dec 2015 16:48:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 22250 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: confirmed Original-Received: via spool by 22250-submit@debbugs.gnu.org id=B22250.14514076392569 (code B ref 22250); Tue, 29 Dec 2015 16:48:02 +0000 Original-Received: (at 22250) by debbugs.gnu.org; 29 Dec 2015 16:47:19 +0000 Original-Received: from localhost ([127.0.0.1]:48819 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84) (envelope-from ) id 1aDxQa-0000fK-AF for submit@debbugs.gnu.org; Tue, 29 Dec 2015 11:47:19 -0500 Original-Received: from eggs.gnu.org ([208.118.235.92]:44208) by debbugs.gnu.org with esmtp (Exim 4.84) (envelope-from ) id 1aDxQV-0000f4-IJ for 22250@debbugs.gnu.org; Tue, 29 Dec 2015 11:47:14 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aDxQM-0006dg-66 for 22250@debbugs.gnu.org; Tue, 29 Dec 2015 11:47:06 -0500 Original-Received: from fencepost.gnu.org ([2001:4830:134:3::e]:58443) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aDxQB-0006cW-5l; Tue, 29 Dec 2015 11:46:51 -0500 Original-Received: from 84.94.185.246.cable.012.net.il ([84.94.185.246]:4531 helo=HOME-C4E4A596F7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_128_CBC_SHA1:128) (Exim 4.82) (envelope-from ) id 1aDxQ9-0000LC-EN; Tue, 29 Dec 2015 11:46:50 -0500 In-reply-to: <8737umi7xw.fsf@justinian.turtle-trading.net> (message from Benjamin Riefenstahl on Mon, 28 Dec 2015 22:23:23 +0100) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] 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:110962 Archived-At: > From: Benjamin Riefenstahl > Cc: 22250@debbugs.gnu.org, larsi@gnus.org > Date: Mon, 28 Dec 2015 22:23:23 +0100 > > I see that you found something with that other file, that's good. My > script-based test case contains no diacritics though, so it is still > another problem. I have uploaded this now to my private webserver as > . > This URL reproduces the problem for me after "G RET". Not for me, it doesn't. I tried "G RET" quite a few times, it always displays correctly. > In the bad case, for the first line, everything looks the same, > vertical-motion gets called with the same parameter, but when it returns > point is at 161. Which is not good. What does window-hscroll return in each of these two cases? > In the good case, it->w->hscroll is 0, in the bad case it->w->hscroll is > 68. Experimentation tells me that the interpretation of window-hscroll > (whether it refers to the left or the right margin) depends on > bidi-paragraph-direction, is that right? Yes and no. It depends on what you mean by "interpretation". > Note that at the point when vertical-motion is called and gives > different answers, bidi-paragraph-direction is always right-to-left, so > it looks like some window parameter that depends on > bidi-paragraph-direction is cached somewhere? The value of bidi-paragraph-direction shouldn't matter when bidi-display-reordering is nil (I've just went through the entire code and didn't see any place where we use that value when bidi-display-reordering is nil). But just in case I missed something, try bindings bidi-paragraph-direction to nil or left-to-right where I bind bidi-display-reordering, and see if that helps. Thanks. P.S. I'm going to commit my patch, as it definitely improves things and is clearly TRT to do (and I'm tired of stashing it ;-).