procyberian/emacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

* bidi.c (bidi_mirror_char): Don't possibly truncate the integer

Browse source 
before checking whether it's out of range.  Put the check inside
eassert.  See
<http://lists.gnu.org/archive/html/emacs-devel/2012-05/msg00485.html>.
This commit is contained in:
Paul Eggert 2012-05-27 00:32:46 -07:00
parent 1d5b82efa7
commit 81899c9138
2 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/ChangeLog
View file

@ -1,3 +1,10 @@
2012-05-27 Paul Eggert <eggert@cs.ucla.edu>
* bidi.c (bidi_mirror_char): Don't possibly truncate the integer
before checking whether it's out of range. Put the check inside
eassert. See
<http://lists.gnu.org/archive/html/emacs-devel/2012-05/msg00485.html>.
2012-05-27 Ken Brown <kbrown@cornell.edu>
* callproc.c (Fcall_process): Restore a line that was accidentally

8
src/bidi.c
View file

@ -204,12 +204,8 @@ bidi_mirror_char (int c)
val = CHAR_TABLE_REF (bidi_mirror_table, c);
if (INTEGERP (val))
{
int v = XINT (val);
if (v < 0 || v > MAX_CHAR)
abort ();
return v;
eassert (CHAR_VALID_P (XINT (val)));
return XINT (val);
}
return c;