unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Richard Stallman <rms@gnu.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 9571@debbugs.gnu.org
Subject: bug#9571: 24.0.50; user option to turn off bidi, please
Date: Sat, 24 Sep 2011 13:30:41 -0400	[thread overview]
Message-ID: <E1R7W3l-0001aI-5u@fencepost.gnu.org> (raw)
In-Reply-To: <834o02t4tg.fsf@gnu.org> (message from Eli Zaretskii on Sat, 24 Sep 2011 17:04:11 +0300)


    The way I understand this request is: make it so that the result of
    reordering is the text in its original logical (i.e. buffer or string)
    order.  The code that is involved in reordering, largely in bidi.c and
    xdisp.c, will still work as it normally does.

Exactly.

    Again, I don't think this is what the original proponents of the
    option wanted, because implementing what you suggest will not bypass
    any of the code in the new display.  IOW, I think it's an entirely
    different feature.

Maybe you're right.  I didn't follow the whole discussion.  I am not
sure why they particularly want to bypass code in the new display;
that seems like an internal implementation detail, and I don't see why
a user would care how the job gets done as long as the output is what
he wants.

    But simply overwriting default_type with STRONG_L isn't right, either.
    That's because as long as we run the code in bidi.c unaltered, there
    are some characters whose bidirectional properties are still needed
    for the code to work: newlines, paragraph separators, line separators,
    and a few others.

Ok, my proposed patch won't do the job.  But wouldn't a slightly more
complicated patch in the same place do the job?

The method of replacing the unicode property table seems to have
several drawbacks:

1. Creating the modified table is more work.

2. It is a big data structure, so having another one would be a waste.

3. It feels wrong to alter the information describing the characters.
This is a matter of choosing a different way to display some characters,
not a matter of redefining what they mean.

I think an easy implementation in bidi_get_type is to have an if
statement choose between the existing switch statement and a new
switch statement.  The new switch statement would return different
values that would avoid reordering and give the Emacs 23 behavior.

-- 
Dr Richard Stallman
President, Free Software Foundation
51 Franklin St
Boston MA 02110
USA
www.fsf.org  www.gnu.org
Skype: No way! That's nonfree (freedom-denying) software.
  Use free telephony http://directory.fsf.org/category/tel/





  reply	other threads:[~2011-09-24 17:30 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-22  4:18 bug#9571: 24.0.50; user option to turn off bidi, please Drew Adams
2011-09-22  5:49 ` Eli Zaretskii
2011-09-22 12:31 ` Stefan Monnier
2011-09-22 13:13   ` Drew Adams
2011-09-22 21:44     ` Richard Stallman
2011-09-23  4:13       ` Stefan Monnier
2011-09-23 12:31         ` Richard Stallman
2011-09-23 14:46           ` Eli Zaretskii
2011-09-23 14:55             ` Lars Magne Ingebrigtsen
2011-09-23 15:06               ` Eli Zaretskii
2011-09-23 17:36                 ` Stefan Monnier
2011-09-23 18:23                 ` Lars Magne Ingebrigtsen
2011-09-23 19:44             ` Richard Stallman
2011-09-23 20:09               ` Eli Zaretskii
2011-09-24  3:56           ` Jason Rumney
2011-09-24 12:28             ` Richard Stallman
2011-09-23  9:13       ` Eli Zaretskii
2011-09-23 12:31         ` Richard Stallman
2011-09-23 14:40           ` Eli Zaretskii
2011-09-23 19:44             ` Richard Stallman
2011-09-23 19:48               ` Eli Zaretskii
2011-09-24 12:28                 ` Richard Stallman
2011-09-24 14:04                   ` Eli Zaretskii
2011-09-24 17:30                     ` Richard Stallman [this message]
2011-09-24 19:15                       ` Eli Zaretskii
2011-09-24 23:52                         ` Richard Stallman
2011-09-23  4:11     ` Stefan Monnier
2011-09-23  8:01       ` Štěpán Němec
2011-09-23  9:21         ` Juanma Barranquero
2011-09-23 10:39           ` Štěpán Němec
2011-09-23 11:01             ` Eli Zaretskii
2011-09-23 16:09               ` Drew Adams
2011-09-23 17:48                 ` Eli Zaretskii
2011-09-23 19:03                   ` Drew Adams
2011-09-23 19:46                     ` Eli Zaretskii
2011-09-23 21:23                       ` Drew Adams
2011-09-23 23:21                         ` Juanma Barranquero
2011-09-24  0:32                           ` Drew Adams
2011-09-24  1:13                             ` Juanma Barranquero
2011-09-24  3:46                               ` Drew Adams
2011-09-24  8:44                                 ` Juanma Barranquero
2012-02-22  2:34                                   ` Glenn Morris
2011-09-24  6:49                             ` Eli Zaretskii
2011-09-24  8:46                               ` Juanma Barranquero
2011-09-23 20:00                     ` Stefan Monnier
2011-09-23 10:18         ` Eli Zaretskii
2011-09-23 11:09           ` Štěpán Němec
2011-09-23 11:45             ` Juanma Barranquero
2011-09-23 13:01               ` Štěpán Němec
2011-09-23 15:03                 ` Eli Zaretskii
2011-09-23 17:46                   ` Stefan Monnier
2011-09-23 18:24                     ` Eli Zaretskii
2011-09-23 19:10                       ` Eli Zaretskii
2011-09-23 11:50             ` Eli Zaretskii
2011-09-24 12:28               ` Richard Stallman
2011-09-24 14:13                 ` Eli Zaretskii
2011-09-24  3:53 ` Jason Rumney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=E1R7W3l-0001aI-5u@fencepost.gnu.org \
    --to=rms@gnu.org \
    --cc=9571@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).