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

Removing guava #437

Merged
merged 2 commits into from
Nov 25, 2015
Merged

Removing guava #437

merged 2 commits into from
Nov 25, 2015

Conversation

dnalves
Copy link
Contributor

@dnalves dnalves commented Nov 25, 2015

Removed Google Guava, only one class of it was being used. Changed the ConcurrentMap, which was a map with weakKeys created by the Guava's MapMaker, to a synchronized WeakHashMap.

@skhaz
Copy link

skhaz commented Nov 25, 2015

Great work!

sibelius added a commit that referenced this pull request Nov 25, 2015
@sibelius sibelius merged commit 520b030 into chennaione:master Nov 25, 2015
@sibelius
Copy link
Contributor

@dnalves thanks

@sibelius sibelius mentioned this pull request Nov 25, 2015
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.

3 participants