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

Fixing squid:S3599 #62

Open

alainlompo wants to merge 1 commit into VerbalExpressions:master from alainlompo:master

alainlompo commented Nov 5, 2017

squid:s3599: Double Brace Initialization should not be used

Because Double Brace Initialization (DBI) creates an anonymous class with a reference to the instance of the owning object,
its use can lead to memory leaks if the anonymous inner class is returned and held by other objects. Even when there's no leak, DBI is so obscure
that it's bound to confuse most maintainers.

Using diamond operator


          Fixing squid:S3599

c0bef77

Using diamnond operator

coveralls commented Nov 5, 2017

Coverage remained the same at 99.338% when pulling c0bef77 on alainlompo:master into f311b15 on VerbalExpressions:master.

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

Labels

None yet