norohind/vanilla
1
0
Fork 0
Code Issues 1 Pull Requests Projects Releases Wiki Activity

fix setSortMode range check

Browse Source 
Modes are inverted indexes, so we can not use Math.abs()
This commit is contained in:
Adrian Ulrich 2017-02-03 21:36:10 +01:00
parent e5e9ba3022
commit c0854e2740
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/ch/blinkenlights/android/vanilla/MediaAdapter.java
View File

@ -560,9 +560,10 @@ public class MediaAdapter
* is negative, the inverse of the index will be used and sort order will * is negative, the inverse of the index will be used and sort order will
* be reversed. * be reversed.
*/ */
public void setSortMode(int i) public void setSortMode(int mode)
{ {
mSortMode = (Math.abs(i) < mSortEntries.length ? i : 0); int index = ( mode < 0 ? ~mode : mode); // 'negative' modes are actually inverted indexes
mSortMode = index < mSortEntries.length ? mode : 0;
} }
/** /**