IntervalTree#remove null pointer exception about corenlp

[AngledLuffa]

Gah, this class is terrible. Zero documentation and apparently even the most basic functionality isn't tested. Thanks so much for uncovering it

[AngledLuffa]

Testing on other trees of size two, it also crashes if you insert things in such an order that the left child of the balanced tree is being deleted or if the root of the balanced tree is deleted. You would think such a thing would be found before the other 500 lines of code got written.

What's debugging got to do with it, anyway?
What's debugging but a sweet old fashioned notion?
Who needs a unit test when a test can be broken?

[Sirius902]

@AngledLuffa Thanks for the quick reply! I found the issue using a larger set of test data but cf11603 seems to have addressed the issue there as well.

[AngledLuffa]

Now released in 4,.5.6