GNU bug report logs - #44329
28.0.50; Crash in bidi_mirror_char on view-hello-file when no bidi-display-reordering (harfbuzz)

Previous Next

Full log

View this message in rfc822 format

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>, Kevin Brubeck Unhammer <unhammer <at> fsfe.org>
Cc: 44329 <at> debbugs.gnu.org
Subject: bug#44329: 28.0.50; Crash in bidi_mirror_char on view-hello-file when no bidi-display-reordering (harfbuzz)
Date: Fri, 30 Oct 2020 23:29:58 +0000

Eli Zaretskii <eliz <at> gnu.org> writes:

>> Is there something like make-obsolete-variable for this case that would
>> give a warning in init.el on this variable?
>
> The doc string already does warn.  The variable is not obsolete, it is
> useful for debugging the display code.

It sure would be helpful if the byte-compiler could warn about it
though.  Not sure if we have the facilities for that.

> You assume people who do this read the manual...

Some kind of warning does seem to be warranted, though.  The text now
seems to almost encourage users to set it:

       The buffer-local variable ‘bidi-display-reordering’ controls whether
    text in the buffer is reordered for display.  If its value is non-‘nil’,
    Emacs reorders characters that have right-to-left directionality when
    they are displayed.  The default value is ‘t’.

Should we just remove it from the manual, if it's only useful for
debugging?  Or add a warning similar to the one in the docstring?

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.