New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
IntervalTree#remove null pointer exception #1405
Comments
Gah, this class is terrible. Zero documentation and apparently even the most basic functionality isn't tested. Thanks so much for uncovering it |
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? |
…htmost, or rootmost nodes. I'm the guy who tests his code. You must be the other guy. Addresses #1405
…htmost, or rootmost nodes. I'm the guy who tests his code. You must be the other guy. Addresses #1405
@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. |
Now released in 4,.5.6 |
The following program causes a null pointer exception to be thrown. I would expect it to complete successfully and output
false
.The text was updated successfully, but these errors were encountered: