search

ivanhk / fastText_java

Java port of c++ version of facebook fasttext
Other
122 stars 80 forks source link

System.exit call #19

Closed sharadkhandelwal closed 7 years ago

sharadkhandelwal commented 7 years ago

System.exit(1) should never be used as it can cause applications running fast text within the same jvm to shutdown unknowingly.

Please throw exceptions instead

ivanhk commented 7 years ago

replaced, enjoy

sharadkhandelwal commented 7 years ago

Thanks a lot !