search

danielnaber / jwordsplitter

small Java library for splitting German compound words
Other
62 stars 11 forks source link

Removed an unnecessary toLowerCase() #2

Closed kraftb closed 10 years ago

kraftb commented 10 years ago

This change happens to also resolve an issue with one of the test cases.

kraftb commented 10 years ago

Those two patches fix issues with inproper upper/lowercase in some of the unit tests.

The first one results in even less code being executed because an "toLowerCase()" gets removed.

The second one addds a somewhat costy handling of exception split but results in proper upper/lower case. By just assuming that exception splits are rather an exception this doesn't have much impact as if the exception split returns "null" no additional code gets executed.