Skip to content

Update StringUtils.java#79

Open
elasticjava wants to merge 1 commit intoHeidelTime:masterfrom
elasticjava:patch-1
Open

Update StringUtils.java#79
elasticjava wants to merge 1 commit intoHeidelTime:masterfrom
elasticjava:patch-1

Conversation

@elasticjava
Copy link

my first contact with your code - i added the maven dependencies and stumpled into the wrong (your) StringUtils class and ended in an infinite loop. You should point out the very experimental state of your project!

my first contact with your code - i added the maven dependencies and stumpled into the wrong (your) StringUtils class and ended in an infinite loop. You should point out the very experimental state of your project!
@kno10
Copy link

kno10 commented Mar 28, 2019

The function countOccurrences is unused, and probably should just be dropped.

No need to be insulting ("very experimental state"). May I point out that errors occur even in the most mature projects, in particular in unused code. For example, the widely known and respected Stanford CoreNLP package used to have an invalid covariance method (that wouldn't compute covariance). As it was unused, it was removed in 2018: stanfordnlp/CoreNLP/pull/592
I would not call CoreNLP "very experimental" though either... mistakes happen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants