Skip to content

Commit ac00459

Browse files
committed
Change requirement of image extensions in ImageSuffix class' Javadoc to a possibility
Some services may provide different image formats using the same suffix, without we know what format the service provide. Enforcing an image extension could so lead to provide invalid image URLs, like for SoundCloud PNG images currently. With this documentation change, it is now clear that users of this class decide of whether they want to include image extensions in the suffix. The previous behavior described in the Javadoc was not enforced.
1 parent 917554a commit ac00459

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

extractor/src/main/java/org/schabi/newpipe/extractor/utils/ImageSuffix.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@
77
import java.util.Objects;
88

99
/**
10-
* Serializable class representing a suffix (including its format extension, such as {@code .jpg})
11-
* which needs to be added to get an image/thumbnail URL with its corresponding height, width and
12-
* estimated resolution level.
10+
* Serializable class representing a suffix (which may include its format extension, such as
11+
* {@code .jpg}) which needs to be added to get an image/thumbnail URL with its corresponding
12+
* height, width and estimated resolution level.
1313
*
1414
* <p>
1515
* This class is used to construct {@link org.schabi.newpipe.extractor.Image Image}

0 commit comments

Comments
 (0)