[Fribidi-discuss] Bugfix patches: --debug and remove_marks
Beni Cherniavsky
cben at techunix.technion.ac.il
Sun May 25 12:33:01 EST 2003
Beni Cherniavsky wrote on 2003-05-25:
> And while we are at number of levels, the few 63 constants should IMHO
> be replaced by some new FRIBIDI_MAX_LEVEL define...
>
There is one already, of course. I wrote this sentence after seeing
the interface of `fribidi_map_range()`. Here is the patch (the header
also had a typo (3 instead of 63 :-) which was fine because this size
is ingored by the compiler anyway):
--- ../fribidi-cvs/fribidi_utils.c 2003-05-23 13:53:16.000000000
+0300
+++ fribidi_utils.c 2003-05-25 22:24:37.000000000 +0300
@@ -73,7 +73,7 @@ fribidi_map_range (FriBidiEnv *fribidien
const FriBidiLevel *embedding_level_list,
/* output */
int *num_mapped_spans,
- FriBidiStrIndex mapped_spans[63][2])
+ FriBidiStrIndex mapped_spans[UNI_MAX_BIDI_LEVEL+2][2])
{
FriBidiStrIndex ch_idx;
fribidi_boolean in_range = FRIBIDI_FALSE;
--- ../fribidi-cvs/fribidi.h 2003-05-23 13:53:16.000000000 +0300
+++ fribidi.h 2003-05-25 22:28:53.000000000 +0300
@@ -189,7 +189,7 @@
*embedding_level_list,
/* output */
int *num_mapped_spans,
- FriBidiStrIndex spans[3][2]);
+ FriBidiStrIndex spans[UNI_MAX_BIDI_LEVEL+2][2]);
/*======================================================================
* fribidi_is_char_rtl() answers the question whether a character
--
Beni Cherniavsky <cben at users.sf.net>
The Three Laws of Copy-Protechnics:
http://www.technion.ac.il/~cben/threelaws.html
More information about the FriBidi
mailing list