baalajimaestro/NewPipe
1
0
Fork 0
Code Issues Pull requests Projects Releases 7 Packages Wiki Activity

Merge pull request #1835 from ritiek/fix-res-crash

Browse source 
Fix crash with default resolution set to best and limited mobile data resolution
This commit is contained in:
Christian Schabesberger 2018-11-01 20:22:44 +01:00 committed by GitHub
commit 45fea983b9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
app/src/main/java/org/schabi/newpipe/util/ListHelper.java
View file

@ -110,7 +110,8 @@ public final class ListHelper {
: context.getString(R.string.best_resolution_key); : context.getString(R.string.best_resolution_key);
String maxResolution = getResolutionLimit(context); String maxResolution = getResolutionLimit(context);
if (maxResolution != null && compareVideoStreamResolution(maxResolution, resolution) < 1){ if (maxResolution != null && (resolution.equals(context.getString(R.string.best_resolution_key))
|| compareVideoStreamResolution(maxResolution, resolution) < 1)) {
resolution = maxResolution; resolution = maxResolution;
} }
return resolution; return resolution;