Merge pull request #143 from jsenellart/master

invalid test causing all pieces to be identified as non valid
google · Jul 27, 2018 · a0ba0b9 · a0ba0b9
2 parents fd96572 + fbcaba9
commit a0ba0b9
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/trainer_interface.cc b/src/trainer_interface.cc
@@ -123,7 +123,7 @@ bool TrainerInterface::IsValidSentencePiece(
       }
       // Do not allow a piece to include multiple Unicode scripts
       // when split_by_unicode_script() is true (default = true).
-      if (prev_script != -1 && prev_script != s &&
+      if (prev_script != static_cast<unicode_script::ScriptType>(-1) && prev_script != s &&
           trainer_spec_.split_by_unicode_script()) {
         return false;
       }