-
Notifications
You must be signed in to change notification settings - Fork 580
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
Fix issue #758 #760
Fix issue #758 #760
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add regression test.
@iloveeclipse thank you for your review. I tried to add test to reproduce the issue, but i can not reproduce... I try to debug the issue using reproduction repository. |
I reproduced this issue in test code. I will push. |
ce08fa9
to
7a45e8a
Compare
@iloveeclipse Could you review this PR? |
40659a2
to
6e2a42e
Compare
0f33afc
to
bf3c2d4
Compare
bf3c2d4
to
25ae4d7
Compare
@iloveeclipse I added regression test. Could you review this PR? |
I'm sorry, no free time at all. |
OK. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
@wreulicke could you propose the same PR for |
@KengoTODA OK. i will propose. |
@wreulicke if possible, propose #751 too. |
This PR fixes #758. This PR do not treat root cause.
checkcast
opcodeMake sure these boxes are checked before submitting your PR -- thank you!
CHANGELOG.md
if you have changed SpotBugs code