CypherpunkSamurai/NewPipeExtractor
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Remove length check from StringUtils.matchToClosingParenthesis

Browse source
This commit is contained in:
XiangRongLin 2021-07-31 16:05:24 +02:00
parent 852a65ff18
commit 37df225556
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
extractor/src/main/java/org/schabi/newpipe/extractor/utils/StringUtils.java
View file

@ -9,10 +9,11 @@ public class StringUtils {
} }
/** /**
* @param string The string to search in * @param string The string to search in.
* @param start A string from which to start searching. * @param start A string from which to start searching.
* @return A substring where each '{' matches a '}' * @return A substring where each '{' matches a '}'.
* @throws IndexOutOfBoundsException If {@code string} does not contain {@code start} * @throws IndexOutOfBoundsException If {@code string} does not contain {@code start}
* or parenthesis could not be matched .
*/ */
@Nonnull @Nonnull
public static String matchToClosingParenthesis(@Nonnull final String string, @Nonnull final String start) { public static String matchToClosingParenthesis(@Nonnull final String string, @Nonnull final String start) {
@ -29,8 +30,7 @@ public class StringUtils {
++endIndex; ++endIndex;
int openParenthesis = 1; int openParenthesis = 1;
int length = string.length(); while (openParenthesis > 0) {
while (openParenthesis > 0 && endIndex < length) {
switch (string.charAt(endIndex)) { switch (string.charAt(endIndex)) {
case '{': case '{':
++openParenthesis; ++openParenthesis;