Skip to content

LostInstance false-positive #2

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.

Sign up for GitHub

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

Jump to bottom
Closed
boris-petrov opened this issue Oct 8, 2019 · 4 comments
Closed

LostInstance false-positive #2

boris-petrov opened this issue Oct 8, 2019 · 4 comments
Labels
bug

Comments

@boris-petrov
Copy link 

String leftOverText = "";
...
leftOverText += new String(buffer, 0, readBytes);

The last line of this example gives a LostInstance error which is wrong.
@tsjensen
Copy link
Member

tsjensen commented Oct 8, 2019

Dang, thanks for the report! I'll look into it asap.

@tsjensen
Copy link
Member

tsjensen commented Oct 9, 2019

Reproduced - it's a bug. 😇
Stay tuned for a fix. It will probably not be before the weekend, though.

@tsjensen tsjensen added the bug label Oct 9, 2019
@boris-petrov
Copy link
Author

No worries, there is no hurry, thanks for the support! :)

tsjensen added a commit that referenced this issue Oct 10, 2019
@tsjensen
Fix LostInstance check to handle compound assignment operators #2
57216ca
@tsjensen
Copy link
Member

Released as v6.0.1.
Sync to Maven Central is currently slow, but jcenter should already be good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@boris-petrov @tsjensen