search

zendesk / android-db-commons

Some common utilities for ContentProvider/ContentResolver/Cursor and other db-related android stuff
Apache License 2.0
222 stars 28 forks source link

Usecase of IdentityLinkedMap (question) #72

Closed ghost closed 9 years ago

ghost commented 9 years ago

Why don't you just use IdentityHashMap provided by JDK instead of IdentityLinkedMap?

chalup commented 9 years ago

I can't for the life of me remember the reason. Most likely explanation: brainfart. Second most likely explanation: someone decided to go perfnazi and implemented it instead of wrapping IdentityHashMap into Collections.synchronizedMap().

Either way, we don't have 100% code coverage in ComposedCursorLoader so I'm going to apply "if it ain't broke don't fix it" rule and keep it in place.