uni-mirrored.el and uni-bidi.el

uni-mirrored.el and uni-bidi.el

[Chong Yidong]

Was there any resolution on the question of replacing the internal bidi
tables with those in uni-mirrored.el and uni-bidi.el (or vice versa)?
It would be nice not to have two independent copies of these tables.

This is immediately relevant if we want to change some C code to be bidi
aware, like generate-new-buffer-name.  Should such code use the internal
tables, or uni-*.el?

Re: uni-mirrored.el and uni-bidi.el

[Eli Zaretskii]

> From: Chong Yidong <cyd@stupidchicken.com>
> Cc: emacs-devel@gnu.org
> Date: Fri, 12 Aug 2011 13:43:20 -0400
> 
> Was there any resolution on the question of replacing the internal bidi
> tables with those in uni-mirrored.el and uni-bidi.el (or vice versa)?

Yes.  I will make the necessary changes, probably next week, to drop
the private copy in biditype.h and use the tables from uni-bidi.el via
the APIs defined on chartab.c.

> This is immediately relevant if we want to change some C code to be bidi
> aware, like generate-new-buffer-name.  Should such code use the internal
> tables, or uni-*.el?

It should use the uniprop tables, as described in the commentary
before uniprop_table_uncompress (in chartab.c).  Those tables are
populated by loading uni-*.el as needed.

Re: uni-mirrored.el and uni-bidi.el

[Eli Zaretskii]

> From: Chong Yidong <cyd@stupidchicken.com>
> Date: Fri, 12 Aug 2011 13:43:20 -0400
> Cc: emacs-devel@gnu.org
> 
> This is immediately relevant if we want to change some C code to be bidi
> aware, like generate-new-buffer-name.

Are you thinking about using directional controls in buffer names only
if the buffer name needs it, btw?  If so, I'm not sure that's the best
idea.  For starters, code that deals with buffer names would need to
make non-trivial tests to know whether to remove these controls or
not, if that code needs the original undecorated buffer name, e.g. to
compare it with the base names of the files visited in buffers.  It is
IMO better to always decorate with directional controls.