chrisbanes / Android-BitmapCache

Android-BitmapCache is a specialised cache, for use with Android Bitmap objects.
834 stars 224 forks source link

RecyclePolicy not interpreted as expected #42

Open elroid opened 10 years ago

elroid commented 10 years ago

I'm having issues with the implementation of the BitmapLruCache$RecyclePolicy.canInBitmap method. Here is how it stands:

boolean canInBitmap() {
    switch (this) {
        case PRE_HONEYCOMB_ONLY:
        case DISABLED:
            return Build.VERSION.SDK_INT >= Build.VERSION_CODES.HONEYCOMB;
    }
    return false;
}

This implies that it will return true (e.g. bitmaps can be re-used) when the policy is set to DISABLED (and you are running on a post-honeycomb device), which i believe is incorrect. May i suggest the following:

boolean canInBitmap() {
    switch (this) {
        case PRE_HONEYCOMB_ONLY:
            return Build.VERSION.SDK_INT < Build.VERSION_CODES.HONEYCOMB;
        case DISABLED:
            return false;
    }
    return true;
}