Skip to content
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

squid:S1319 - Declarations should use Java collection interfaces such… #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

squid:S1319 - Declarations should use Java collection interfaces such… #1

wants to merge 1 commit into from

Conversation

m-ezzat
Copy link

@m-ezzat m-ezzat commented May 24, 2016

This pull request is focused on resolving occurrences of Sonar rule squid:S1319 - Declarations should use Java collection interfaces such as List rather than specific implementation classes such as LinkedList

You can find more information about the issue here: https://dev.eclipse.org/sonar/coding_rules#q=squid:1319

Please let me know if you have any questions.

M-Ezzat

… as List rather than specific implementation classes such as LinkedList
@m-ezzat
Copy link
Author

m-ezzat commented Jun 28, 2016

I am not sure whether you do not agree with these changes or not. Please close this PR if not.

@benniaobuguai
Copy link
Owner

I just see this right now, so sorry. I also use the Map before, I looked at the Android performance advice directly using the class object is faster than the interface.

Statement HashMap map is more efficient than Map map, even if both are HashMap. About 6% of the performance loss.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants