Skip to content

Commit 67c0cee

Browse files
committed
Error handling in subscribe() in DownloadDialog
1 parent c12ac64 commit 67c0cee

1 file changed

Lines changed: 15 additions & 13 deletions

File tree

app/src/main/java/org/schabi/newpipe/download/DownloadDialog.java

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -312,25 +312,27 @@ public void onStopTrackingTouch(final SeekBar p1) { }
312312

313313
private void fetchStreamsSize() {
314314
disposables.clear();
315-
316315
disposables.add(StreamSizeWrapper.fetchSizeForWrapper(wrappedVideoStreams)
317316
.subscribe(result -> {
318-
if (dialogBinding.videoAudioGroup.getCheckedRadioButtonId() == R.id.video_button) {
319-
setupVideoSpinner();
320-
}
321-
}));
317+
if (dialogBinding.videoAudioGroup.getCheckedRadioButtonId()
318+
== R.id.video_button) {
319+
setupVideoSpinner();
320+
}
321+
}, throwable -> Log.e(TAG, "Throwable " + throwable.getMessage())));
322322
disposables.add(StreamSizeWrapper.fetchSizeForWrapper(wrappedAudioStreams)
323323
.subscribe(result -> {
324-
if (dialogBinding.videoAudioGroup.getCheckedRadioButtonId() == R.id.audio_button) {
325-
setupAudioSpinner();
326-
}
327-
}));
324+
if (dialogBinding.videoAudioGroup.getCheckedRadioButtonId()
325+
== R.id.audio_button) {
326+
setupAudioSpinner();
327+
}
328+
}, throwable -> Log.e(TAG, "Throwable " + throwable.getMessage())));
328329
disposables.add(StreamSizeWrapper.fetchSizeForWrapper(wrappedSubtitleStreams)
329330
.subscribe(result -> {
330-
if (dialogBinding.videoAudioGroup.getCheckedRadioButtonId() == R.id.subtitle_button) {
331-
setupSubtitleSpinner();
332-
}
333-
}));
331+
if (dialogBinding.videoAudioGroup.getCheckedRadioButtonId()
332+
== R.id.subtitle_button) {
333+
setupSubtitleSpinner();
334+
}
335+
}, throwable -> Log.e(TAG, "Throwable " + throwable.getMessage())));
334336
}
335337

336338
@Override

0 commit comments

Comments
 (0)