Peertube
/
NewPipe-app-android
mirror of https://github.com/TeamNewPipe/NewPipe/
1
0
Fork 0
Code Issues Releases Wiki Activity

Merge pull request #232 from DevFactory/release/null-pointer-should-not-be-dereferenced-fix-1 

Code quality fix - Null pointers should not be dereferenced.
This commit is contained in:
Christian Schabesberger 2016-03-11 14:15:41 +01:00
parent 31dd68be1d 1a3aaf86ee
commit c6ccc2b20d
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/src/main/java/org/schabi/newpipe/extractor/AudioStream.java
View File

@ -40,7 +40,7 @@ public class AudioStream {
// revelas wether two streams are equal
public boolean equals(AudioStream cmp) {
return equalStats(cmp)
return cmp != null && equalStats(cmp)
&& url == cmp.url;
}
}

2
app/src/main/java/org/schabi/newpipe/extractor/VideoStream.java
View File

@ -38,7 +38,7 @@ public class VideoStream {
// revelas wether two streams are equal
public boolean equals(VideoStream cmp) {
return equalStats(cmp)
return cmp != null && equalStats(cmp)
&& url == cmp.url;
}
}

2
app/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubeStreamExtractor.java
View File

@ -599,7 +599,7 @@ public class YoutubeStreamExtractor extends StreamExtractor {
public String getAverageRating() throws ParsingException {
try {
if (playerArgs == null) {
videoInfoPage.get("avg_rating");
return videoInfoPage.get("avg_rating");
}
return playerArgs.getString("avg_rating");
} catch (JSONException e) {