forked from StarOpenSource/Engine
Fix warnings and duplicate check
This commit is contained in:
parent
6785ec64d3
commit
0252776195
1 changed files with 16 additions and 9 deletions
|
@ -177,14 +177,30 @@ public final class DependencyResolver {
|
||||||
for (char character : dependency.toCharArray()) {
|
for (char character : dependency.toCharArray()) {
|
||||||
switch (character) {
|
switch (character) {
|
||||||
case '=' -> {
|
case '=' -> {
|
||||||
|
// Check for duplicate
|
||||||
|
if (duplicateCheck[0])
|
||||||
|
throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '=' characters");
|
||||||
|
|
||||||
|
// Update mode
|
||||||
mode = 1;
|
mode = 1;
|
||||||
|
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
case '<' -> {
|
case '<' -> {
|
||||||
|
// Check for duplicate
|
||||||
|
if (duplicateCheck[1])
|
||||||
|
throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '<' characters");
|
||||||
|
|
||||||
|
// Update mode
|
||||||
mode = 2;
|
mode = 2;
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
case '>' -> {
|
case '>' -> {
|
||||||
|
// Check for duplicate
|
||||||
|
if (duplicateCheck[2])
|
||||||
|
throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '>' characters");
|
||||||
|
|
||||||
|
// Update mode
|
||||||
mode = 3;
|
mode = 3;
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
@ -195,9 +211,6 @@ public final class DependencyResolver {
|
||||||
case 0 -> identifier.append(character);
|
case 0 -> identifier.append(character);
|
||||||
// Version equal
|
// Version equal
|
||||||
case 1 -> {
|
case 1 -> {
|
||||||
// Check for duplicate
|
|
||||||
if (character == '=' && duplicateCheck[0])
|
|
||||||
throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '=' characters");
|
|
||||||
duplicateCheck[0] = true;
|
duplicateCheck[0] = true;
|
||||||
|
|
||||||
// Check for smaller and bigger than
|
// Check for smaller and bigger than
|
||||||
|
@ -208,9 +221,6 @@ public final class DependencyResolver {
|
||||||
}
|
}
|
||||||
// Version smaller
|
// Version smaller
|
||||||
case 2 -> {
|
case 2 -> {
|
||||||
// Check for duplicate
|
|
||||||
if (character == '<' && duplicateCheck[1])
|
|
||||||
throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '<' characters");
|
|
||||||
duplicateCheck[1] = true;
|
duplicateCheck[1] = true;
|
||||||
|
|
||||||
// Check for equal
|
// Check for equal
|
||||||
|
@ -221,9 +231,6 @@ public final class DependencyResolver {
|
||||||
}
|
}
|
||||||
// Version bigger
|
// Version bigger
|
||||||
case 3 -> {
|
case 3 -> {
|
||||||
// Check for duplicate
|
|
||||||
if (character == '>' && duplicateCheck[2])
|
|
||||||
throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '>' characters");
|
|
||||||
duplicateCheck[2] = true;
|
duplicateCheck[2] = true;
|
||||||
|
|
||||||
// Check for equal
|
// Check for equal
|
||||||
|
|
Loading…
Reference in a new issue