Skip to content

Commit 9fbf232

Browse files
committed
Move LVT naming into a separate module
This makes is possible to be used in other programs as a library.
1 parent 40b8361 commit 9fbf232

27 files changed

Lines changed: 62 additions & 24 deletions

build.gradle.kts

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,11 @@ plugins {
99
dependencies {
1010
api(libs.checker)
1111

12+
implementation(projects.codebookLvt)
13+
1214
implementation(libs.guice)
1315
implementation(libs.inject)
16+
implementation(libs.guava)
1417

1518
implementation(libs.gson)
1619
implementation(libs.bytes)
@@ -20,7 +23,7 @@ dependencies {
2023
implementation(libs.unpick.cli)
2124

2225
implementation(platform(libs.hypo.platform))
23-
implementation(libs.bundles.hypo)
26+
implementation(libs.bundles.hypo.full)
2427

2528
implementation(libs.lorenz)
2629

@@ -38,6 +41,12 @@ dependencies {
3841
testImplementation(libs.mockito.junit)
3942
}
4043

44+
tasks.jar {
45+
manifest {
46+
attributes("Automatic-Module-Name" to "io.papermc.codebook")
47+
}
48+
}
49+
4150
publishing {
4251
publications {
4352
codebook {

codebook-lvt/build.gradle.kts

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
plugins {
2+
`java-library`
3+
id("codebook")
4+
}
5+
6+
dependencies {
7+
implementation(platform(libs.hypo.platform))
8+
9+
api(libs.checker)
10+
api(libs.bundles.hypo.base)
11+
api(libs.lorenz)
12+
13+
implementation(libs.guice)
14+
implementation(libs.inject)
15+
implementation(libs.guava)
16+
17+
implementation(libs.bundles.hypo.impl)
18+
implementation(libs.bundles.asm)
19+
}
20+
21+
tasks.jar {
22+
manifest {
23+
attributes("Automatic-Module-Name" to "io.papermc.codebook.lvt")
24+
}
25+
}
26+
27+
publishing {
28+
publications {
29+
codebook {
30+
pom {
31+
name.set("codebook-lvt")
32+
description.set("LVT naming tool for PaperMC")
33+
}
34+
}
35+
}
36+
}

src/main/java/io/papermc/codebook/lvt/LvtNamer.java renamed to codebook-lvt/src/main/java/io/papermc/codebook/lvt/LvtNamer.java

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
package io.papermc.codebook.lvt;
2424

2525
import dev.denwav.hypo.asm.AsmClassData;
26-
import dev.denwav.hypo.asm.AsmConstructorData;
2726
import dev.denwav.hypo.asm.AsmMethodData;
2827
import dev.denwav.hypo.core.HypoContext;
2928
import dev.denwav.hypo.hydrate.generic.HypoHydration;
@@ -122,13 +121,7 @@ private void fillNames0(final MethodData method) throws IOException {
122121
}
123122
}
124123

125-
final MethodNode node;
126-
if (method instanceof AsmMethodData) {
127-
node = ((AsmMethodData) method).getNode();
128-
} else {
129-
node = ((AsmConstructorData) method).getNode();
130-
}
131-
124+
final MethodNode node = ((AsmMethodData) method).getNode();
132125
final ClassData parentClass = method.parentClass();
133126
// A method cannot be both a lambda expression and a local class, so if we've already determined an outer
134127
// method, there's nothing to do here.

src/main/java/io/papermc/codebook/lvt/LvtTypeSuggester.java renamed to codebook-lvt/src/main/java/io/papermc/codebook/lvt/LvtTypeSuggester.java

File renamed without changes.

src/main/java/io/papermc/codebook/lvt/LvtUtil.java renamed to codebook-lvt/src/main/java/io/papermc/codebook/lvt/LvtUtil.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424

2525
import dev.denwav.hypo.asm.HypoAsmUtil;
2626
import dev.denwav.hypo.model.data.types.JvmType;
27-
import java.util.Arrays;
2827
import java.util.List;
2928
import java.util.function.Predicate;
3029
import org.checkerframework.checker.nullness.qual.Nullable;
@@ -54,7 +53,14 @@ public static String capitalize(final String name, final int index) {
5453
}
5554

5655
public static Predicate<String> equalsAny(final String... strings) {
57-
return s -> Arrays.stream(strings).anyMatch(Predicate.isEqual(s));
56+
return s -> {
57+
for (final String string : strings) {
58+
if (string.equals(s)) {
59+
return true;
60+
}
61+
}
62+
return false;
63+
};
5864
}
5965

6066
public static String findNextWord(final int start, final String str) {

src/main/java/io/papermc/codebook/lvt/RootLvtSuggester.java renamed to codebook-lvt/src/main/java/io/papermc/codebook/lvt/RootLvtSuggester.java

File renamed without changes.

src/main/java/io/papermc/codebook/lvt/package-info.java renamed to codebook-lvt/src/main/java/io/papermc/codebook/lvt/package-info.java

File renamed without changes.

src/main/java/io/papermc/codebook/lvt/suggestion/GenericSuggester.java renamed to codebook-lvt/src/main/java/io/papermc/codebook/lvt/suggestion/GenericSuggester.java

File renamed without changes.

src/main/java/io/papermc/codebook/lvt/suggestion/LvtSuggester.java renamed to codebook-lvt/src/main/java/io/papermc/codebook/lvt/suggestion/LvtSuggester.java

File renamed without changes.

src/main/java/io/papermc/codebook/lvt/suggestion/MathSuggester.java renamed to codebook-lvt/src/main/java/io/papermc/codebook/lvt/suggestion/MathSuggester.java

File renamed without changes.

0 commit comments

Comments
 (0)