]> granicus.if.org Git - fribidi/commitdiff
Remove skipping tests and fixed two test errors.
authorDov Grobgeld <dov.grobgeld@gmail.com>
Sun, 22 Jul 2018 18:55:19 +0000 (21:55 +0300)
committerDov Grobgeld <dov.grobgeld@gmail.com>
Sun, 22 Jul 2018 18:55:19 +0000 (21:55 +0300)
- BidiCharTest was erroneously skipping RTL tests. This has been
  fixed.
- This exposed 18 errors. Two have been fixed in this commit, but
  16 remains.

lib/fribidi-bidi.c
test/unicode-conformance/BidiCharacterTest.c

index 9d700d3bea4f82c0a8de20ef1998e57af63ba1f9..4966369aba06a5caa2a5b512fc966c9736a15e5a 100644 (file)
@@ -1175,7 +1175,7 @@ fribidi_get_par_embedding_levels_ex (
         const FriBidiBracketType NoBracket = FRIBIDI_NO_BRACKET;
         for_run_list (pp, main_run_list)
           pp->bracket_type = NoBracket;
-        compact_list (main_run_list);
+        compact_neutrals (main_run_list);
       }
     }
 
index 791f911dc7f4f0b396b5d6d043828dac2fe59f1a..cf3f6766748e42f1e11c75f9463ca1b778b514e4 100644 (file)
@@ -253,7 +253,7 @@ main (int argc, char **argv)
   FriBidiChar *code_points = NULL;
   int code_points_len = 0;
   int expected_ltor_len = 0;
-  int base_dir_mode = 0, paragraph_dir;
+  int paragraph_dir = 0;
   FriBidiLevel *expected_levels = NULL;
   int *expected_ltor = NULL;
   int resolved_paragraph_embedding_level;
@@ -345,9 +345,6 @@ main (int argc, char **argv)
                 bracket_types[i] = NoBracket;
           }
 
-        if ((paragraph_dir & (1<<base_dir_mode)) == 0)
-          continue;
-
         switch (paragraph_dir)
           {
           case 0: base_dir = FRIBIDI_PAR_LTR; break;
@@ -430,17 +427,8 @@ main (int argc, char **argv)
 
             if (debug)
               {
-                FriBidiParType base_dir;
-
                 fribidi_set_debug (1);
 
-                switch (base_dir_mode)
-                  {
-                  case 0: base_dir = FRIBIDI_PAR_ON;  break;
-                  case 1: base_dir = FRIBIDI_PAR_LTR; break;
-                  case 2: base_dir = FRIBIDI_PAR_RTL; break;
-                  }
-
                 if (fribidi_get_par_embedding_levels_ex (types,
                                                          bracket_types,
                                                          types_len,