JakeWharton/salvage — pull request #2 by eschlenz (2013-09-07 19:55:00+00:00)
Title: Changed references to SparseArray.removeAt() to SparseArray.remove() wit...

Jake,

Great idea (this project). I made a few beneficial changes that I would like you to accept:

1) I believe there was a small bug in the static retrieveFromScrap function. Within the loop, where you break out early, you were removing from the scrapViews using the for loop's index. I believe you meant to remove by key here.

2) I have changed references to removeAt to remove() with a key look up. This makes the code compatible with versions prior to API level 11.

3) I wrapped the setAccessibilityDelegate in an SDK version check, making it compatible with API levels below 14.

Please accept these changes.

Eric Schlenz

P.S. I am using your ViewPagerIndicator library. Love it! Nice stuff you are contributing.