Merge pull request #1242 from TeamNewPipe/revert-1205-feature-branch
Revert "Refactored Identifiers"
This commit is contained in:
commit
ea1a1d1375
|
@ -28,7 +28,7 @@ public abstract class Extractor {
|
|||
@Nullable
|
||||
private ContentCountry forcedContentCountry = null;
|
||||
|
||||
private boolean isPageFetched = false;
|
||||
private boolean pageFetched = false;
|
||||
// called like this to prevent checkstyle errors about "hiding a field"
|
||||
private final Downloader downloader;
|
||||
|
||||
|
@ -54,21 +54,21 @@ public abstract class Extractor {
|
|||
* @throws ExtractionException if the pages content is not understood
|
||||
*/
|
||||
public void fetchPage() throws IOException, ExtractionException {
|
||||
if (isPageFetched) {
|
||||
if (pageFetched) {
|
||||
return;
|
||||
}
|
||||
onFetchPage(downloader);
|
||||
isPageFetched = true;
|
||||
pageFetched = true;
|
||||
}
|
||||
|
||||
protected void assertPageFetched() {
|
||||
if (!isPageFetched) {
|
||||
if (!pageFetched) {
|
||||
throw new IllegalStateException("Page is not fetched. Make sure you call fetchPage()");
|
||||
}
|
||||
}
|
||||
|
||||
protected boolean isPageFetched() {
|
||||
return isPageFetched;
|
||||
return pageFetched;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -35,7 +35,7 @@ public final class AudioStream extends Stream {
|
|||
|
||||
// Fields for DASH
|
||||
private int itag = ITAG_NOT_AVAILABLE_OR_NOT_APPLICABLE;
|
||||
private int bitRate;
|
||||
private int bitrate;
|
||||
private int initStart;
|
||||
private int initEnd;
|
||||
private int indexStart;
|
||||
|
@ -351,7 +351,7 @@ public final class AudioStream extends Stream {
|
|||
this.itagItem = itagItem;
|
||||
this.itag = itagItem.id;
|
||||
this.quality = itagItem.getQuality();
|
||||
this.bitRate = itagItem.getBitrate();
|
||||
this.bitrate = itagItem.getBitrate();
|
||||
this.initStart = itagItem.getInitStart();
|
||||
this.initEnd = itagItem.getInitEnd();
|
||||
this.indexStart = itagItem.getIndexStart();
|
||||
|
@ -369,8 +369,8 @@ public final class AudioStream extends Stream {
|
|||
* {@inheritDoc}
|
||||
*/
|
||||
@Override
|
||||
public boolean areStatsEqual(final Stream cmp) {
|
||||
return super.areStatsEqual(cmp) && cmp instanceof AudioStream
|
||||
public boolean equalStats(final Stream cmp) {
|
||||
return super.equalStats(cmp) && cmp instanceof AudioStream
|
||||
&& averageBitrate == ((AudioStream) cmp).averageBitrate
|
||||
&& Objects.equals(audioTrackId, ((AudioStream) cmp).audioTrackId)
|
||||
&& audioTrackType == ((AudioStream) cmp).audioTrackType
|
||||
|
@ -405,8 +405,8 @@ public final class AudioStream extends Stream {
|
|||
*
|
||||
* @return the bitrate set from the {@link ItagItem} passed in the constructor of the stream.
|
||||
*/
|
||||
public int getBitRate() {
|
||||
return bitRate;
|
||||
public int getBitrate() {
|
||||
return bitrate;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -74,7 +74,7 @@ public abstract class Stream implements Serializable {
|
|||
return false;
|
||||
}
|
||||
for (final Stream cmpStream : streamList) {
|
||||
if (stream.areStatsEqual(cmpStream)) {
|
||||
if (stream.equalStats(cmpStream)) {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
@ -97,7 +97,7 @@ public abstract class Stream implements Serializable {
|
|||
* @param other the stream object to be compared to this stream object
|
||||
* @return whether the stream have the same stats or not, based on the criteria above
|
||||
*/
|
||||
public boolean areStatsEqual(@Nullable final Stream other) {
|
||||
public boolean equalStats(@Nullable final Stream other) {
|
||||
if (other == null || mediaFormat == null || other.mediaFormat == null) {
|
||||
return false;
|
||||
}
|
||||
|
|
|
@ -265,8 +265,8 @@ public final class SubtitlesStream extends Stream {
|
|||
* {@inheritDoc}
|
||||
*/
|
||||
@Override
|
||||
public boolean areStatsEqual(final Stream cmp) {
|
||||
return super.areStatsEqual(cmp)
|
||||
public boolean equalStats(final Stream cmp) {
|
||||
return super.equalStats(cmp)
|
||||
&& cmp instanceof SubtitlesStream
|
||||
&& code.equals(((SubtitlesStream) cmp).code)
|
||||
&& autoGenerated == ((SubtitlesStream) cmp).autoGenerated;
|
||||
|
|
|
@ -326,8 +326,8 @@ public final class VideoStream extends Stream {
|
|||
* {@inheritDoc}
|
||||
*/
|
||||
@Override
|
||||
public boolean areStatsEqual(final Stream cmp) {
|
||||
return super.areStatsEqual(cmp)
|
||||
public boolean equalStats(final Stream cmp) {
|
||||
return super.equalStats(cmp)
|
||||
&& cmp instanceof VideoStream
|
||||
&& resolution.equals(((VideoStream) cmp).resolution)
|
||||
&& isVideoOnly == ((VideoStream) cmp).isVideoOnly;
|
||||
|
|
Loading…
Reference in New Issue