Skip to content

Commit 6b1a6d2

Browse files
committed
Better naming
1 parent 79540a8 commit 6b1a6d2

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

app/src/main/java/org/schabi/newpipe/util/StreamItemAdapter.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -47,15 +47,16 @@ public class StreamItemAdapter<T extends Stream, U extends Stream> extends BaseA
4747
* has no audio ({@link VideoStream#isVideoOnly()} returns true) and has no secondary stream
4848
* associated with it.
4949
*/
50-
private final boolean hasVideoOnlyWithNoSecondaryStream;
50+
private final boolean hasAnyVideoOnlyStreamWithNoSecondaryStream;
5151

5252
public StreamItemAdapter(final Context context, final StreamSizeWrapper<T> streamsWrapper,
5353
final SparseArray<SecondaryStreamHelper<U>> secondaryStreams) {
5454
this.context = context;
5555
this.streamsWrapper = streamsWrapper;
5656
this.secondaryStreams = secondaryStreams;
5757

58-
this.hasVideoOnlyWithNoSecondaryStream = checkHasVideoOnlyWithNoSecondaryStream();
58+
this.hasAnyVideoOnlyStreamWithNoSecondaryStream =
59+
checkHasAnyVideoOnlyStreamWithNoSecondaryStream();
5960
}
6061

6162
public StreamItemAdapter(final Context context, final StreamSizeWrapper<T> streamsWrapper) {
@@ -119,7 +120,7 @@ private View getCustomView(final int position, final View view, final ViewGroup
119120
final VideoStream videoStream = ((VideoStream) stream);
120121
qualityString = videoStream.getResolution();
121122

122-
if (hasVideoOnlyWithNoSecondaryStream) {
123+
if (hasAnyVideoOnlyStreamWithNoSecondaryStream) {
123124
if (videoStream.isVideoOnly()) {
124125
woSoundIconVisibility = hasSecondaryStream(position)
125126
// It has a secondary stream associated with it, so check if it's a
@@ -186,9 +187,9 @@ private boolean hasSecondaryStream(final int position) {
186187

187188
/**
188189
* @return if there are any video-only streams with no secondary stream associated with them.
189-
* @see #hasVideoOnlyWithNoSecondaryStream
190+
* @see #hasAnyVideoOnlyStreamWithNoSecondaryStream
190191
*/
191-
private boolean checkHasVideoOnlyWithNoSecondaryStream() {
192+
private boolean checkHasAnyVideoOnlyStreamWithNoSecondaryStream() {
192193
for (int i = 0; i < streamsWrapper.getStreamsList().size(); i++) {
193194
final T stream = streamsWrapper.getStreamsList().get(i);
194195
if (stream instanceof VideoStream) {

0 commit comments

Comments
 (0)