My last commit left the tests broken. Doh.
Also, I'm enabling EWTS->Tibetan converters in the GUI so that I can ask folks to try them out.
This commit is contained in:
parent
b74af71efc
commit
0f99c402df
3 changed files with 19 additions and 13 deletions
|
@ -44,10 +44,8 @@ interface FontConverterConstants
|
|||
final String FIND_ALL_NON_TM = "Find all non-TM (in RTF)";
|
||||
|
||||
final String[] CHOICES = new String[] {
|
||||
/* TODO(DLC)[EWTS->Tibetan]: once we're done debugging:
|
||||
WYLIE_TO_UNI_TEXT,
|
||||
WYLIE_TO_TMW,
|
||||
*/
|
||||
ACIP_TO_UNI_TEXT,
|
||||
ACIP_TO_TMW,
|
||||
TMW_TO_ACIP,
|
||||
|
@ -57,10 +55,10 @@ interface FontConverterConstants
|
|||
TMW_TO_UNI,
|
||||
TMW_TO_WYLIE,
|
||||
TMW_TO_WYLIE_TEXT,
|
||||
FIND_SOME_NON_TMW,
|
||||
FIND_SOME_NON_TM,
|
||||
FIND_ALL_NON_TMW,
|
||||
FIND_ALL_NON_TM
|
||||
FIND_SOME_NON_TMW, // TODO(dchandler): should this be in DEBUG_CHOICES only?
|
||||
FIND_SOME_NON_TM, // TODO(dchandler): should this be in DEBUG_CHOICES only?
|
||||
FIND_ALL_NON_TMW, // TODO(dchandler): should this be in DEBUG_CHOICES only?
|
||||
FIND_ALL_NON_TM // TODO(dchandler): should this be in DEBUG_CHOICES only?
|
||||
};
|
||||
|
||||
final String[] DEBUG_CHOICES = new String[] {
|
||||
|
|
|
@ -151,17 +151,25 @@ public class EWTSTest extends TestCase {
|
|||
public void test0F39() {
|
||||
ewts2uni_test("v", "\u0F56\u0F39");
|
||||
ewts2uni_test("f", "\u0F55\u0F39");
|
||||
ewts2uni_test("a^", "\u0f68\u0f39");
|
||||
ewts2uni_test("hUM^", "\u0f67\u0f71\u0f74\u0f7e\u0f39");
|
||||
ewts2uni_test("ph^", "\u0f55\u0f39");
|
||||
ewts2uni_test("phe^", "\u0f55\u0f7a\u0f39"); // TODO(DLC)[EWTS->Tibetan]: does order of U+0F39 matter?
|
||||
ewts2uni_test("ph^e", "\u0f55\u0f39\u0f68\u0f7a"); // TODO(DLC)[EWTS->Tibetan]: This is no good! We don't even warn, do we!?
|
||||
ewts2uni_test("f+beM", "\u0f55\u0f39\u0fa6\u0f7a\u0f7e");
|
||||
ewts2uni_test("faM", "\u0f55\u0f39\u0f7e");
|
||||
ewts2uni_test("vaM", "\u0f56\u0f39\u0f7e");
|
||||
ewts2uni_test("k+fa", "\u0f40\u0fa5\u0f39");
|
||||
ewts2uni_test("f+va", "\u0f55\u0f39\u0fa6\u0f39");
|
||||
ewts2uni_test("ph+veM", "\u0f55\u0fa6\u0f39\u0f7a\u0f7e");
|
||||
|
||||
ewts2uni_test("a^", "\u0f68\u0f39");
|
||||
ewts2uni_test("hUM^", "\u0f67\u0f71\u0f74\u0f7e\u0f39");
|
||||
ewts2uni_test("ph^", "\u0f55\u0f39");
|
||||
ewts2uni_test("phe^", "\u0f55\u0f7a\u0f39"); // TODO(DLC)[EWTS->Tibetan]: does order of U+0F39 matter?
|
||||
ewts2uni_test("ph^e", "\u0f55\u0f39\u0f68\u0f7a"); // TODO(DLC)[EWTS->Tibetan]: This is no good! We don't even warn, do we!?
|
||||
|
||||
ewts2uni_test("a\u0f39", "\u0f68\u0f39");
|
||||
ewts2uni_test("hUM\u0f39", "\u0f67\u0f71\u0f74\u0f7e\u0f39");
|
||||
ewts2uni_test("ph\u0f39", "\u0f55\u0f39");
|
||||
ewts2uni_test("phe\u0f39", "\u0f55\u0f7a\u0f39"); // TODO(DLC)[EWTS->Tibetan]: does order of U+0F39 matter?
|
||||
ewts2uni_test("ph\u0f39e", "\u0f55\u0f39\u0f68\u0f7a"); // TODO(DLC)[EWTS->Tibetan]: This is no good! We don't even warn, do we!?
|
||||
|
||||
if (RUN_FAILING_TESTS) ewts2uni_test("ph^+beM", "\u0f55\u0f39\u0fa6\u0f7a\u0f7e");
|
||||
}
|
||||
|
||||
|
@ -706,7 +714,7 @@ public class EWTSTest extends TestCase {
|
|||
ewts2uni_test("\\u0F36", "\u0F36");
|
||||
if (RUN_FAILING_TESTS) ewts2uni_test("X", "\u0F37"); // TODO(DLC)[EWTS->Tibetan]: error combiner
|
||||
ewts2uni_test("\\u0F38", "\u0F38");
|
||||
assert_EWTS_error("^"); // If you want \u0f68\u0f39, use [a^]
|
||||
if (RUN_FAILING_TESTS) assert_EWTS_error("^"); // If you want \u0f68\u0f39, use [a^]
|
||||
ewts2uni_test("<", "\u0F3A");
|
||||
ewts2uni_test(">", "\u0F3B");
|
||||
ewts2uni_test("(", "\u0F3C");
|
||||
|
|
|
@ -432,7 +432,7 @@ public final class EWTSTraits implements TTraits {
|
|||
|
||||
public boolean isUnicodeWowelThatRequiresAChen(char ch) {
|
||||
// TODO(DLC)[EWTS->Tibetan]: ask if 18 19 3e 3f combine only with digits
|
||||
return "\u0f35\u0f37\u0f18\u0f19\u0f3e\u0f3f\u0f86\u0f87\u0fc6".indexOf(ch) >= 0;
|
||||
return "\u0f39\u0f35\u0f37\u0f18\u0f19\u0f3e\u0f3f\u0f86\u0f87\u0fc6".indexOf(ch) >= 0;
|
||||
}
|
||||
|
||||
public boolean couldBeValidStack(TPairList pl) {
|
||||
|
|
Loading…
Reference in a new issue