[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