Skip to content

Commit c4e1a03

Browse files
committed
remove useless comments
1 parent d5705b4 commit c4e1a03

16 files changed

Lines changed: 1 addition & 120 deletions

File tree

DEVELOPMENT.md

Lines changed: 0 additions & 42 deletions
This file was deleted.

build.gradle

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ def hasHudElementRegistry = stonecutter.eval(mcVersion, '>=1.21.6') // 1.21.6+
1414

1515
apply plugin: isLegacy ? 'net.fabricmc.fabric-loom-remap' : 'net.fabricmc.fabric-loom'
1616

17-
// Per-MC Java version (21 for 1.21.x, 25 for 26.1+).
1817
def javaVersionInt = Integer.parseInt(project.java_version)
1918
def javaLang = JavaVersion.toVersion(javaVersionInt)
2019
def mixinJavaCompat = "JAVA_${javaVersionInt}"
@@ -31,12 +30,8 @@ repositories {
3130
maven { url 'https://maven.terraformersmc.com/releases/' }
3231
}
3332

34-
// Adds the Modrinth maven and modmenuDependency(), a fallback for when the
35-
// TerraformersMC maven is unreachable.
3633
apply from: "${rootDir}/gradle/modrinth-fallback.gradle"
3734

38-
// MC 26+ uses fabric-rendering-v1's transitive deps but rejects ASM coming from
39-
// annotation processors.
4035
if (!isLegacy) {
4136
configurations.configureEach {
4237
if (name.toLowerCase().contains('annotationprocessor')) {
@@ -45,9 +40,6 @@ if (!isLegacy) {
4540
}
4641
}
4742

48-
// Every variant declares fabric-api, modmenu, and cloth-config as runtime deps
49-
// (not bundled). 1.21.x uses modImplementation for intermediary remapping; 26.x
50-
// uses plain implementation against unobfuscated MC.
5143
dependencies {
5244
minecraft "com.mojang:minecraft:${project.minecraft_version}"
5345

gradle.properties

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,8 @@
1-
# Done to increase the memory available to gradle.
21
org.gradle.jvmargs=-Xmx2G
32
org.gradle.parallel=true
43

5-
# IntelliJ IDEA is not yet fully compatible with configuration cache, see: https://github.com/FabricMC/fabric-loom/issues/1349
64
org.gradle.configuration-cache=false
75

8-
# Acknowledge using Groovy DSL with Stonecutter (Kotlin DSL is recommended but Groovy works).
96
dev.kikugie.stonecutter.hard_mode=true
107

118
# Single loom version covers all MC variants. The `fabric-loom-remap` plugin id

launchtest/src/fallback/java/dsns/betterhud/gametest/LaunchTestClient.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,10 @@
2121
public class LaunchTestClient implements ClientModInitializer {
2222
private static final boolean EXPECT_WORLD = Boolean.getBoolean("betterhud.launchtest.expectWorld");
2323

24-
// 200 ticks (10s) gives the software renderer on CI time to draw chunks.
2524
private static final int WORLD_SCREENSHOT_TICK = 200;
2625
private static final int WORLD_STOP_TICK = 240;
2726
private static final int TITLE_SCREENSHOT_TICK = 40;
2827
private static final int TITLE_STOP_TICK = 80;
29-
// 2400 ticks (2min) is plenty to join the pre-generated flat world; bail
30-
// out with a failure instead of hanging until the CI job timeout.
3128
private static final int WORLD_JOIN_TIMEOUT_TICKS = 2400;
3229

3330
private int totalTicks;

launchtest/src/gametest-legacy/java/dsns/betterhud/gametest/BetterHudClientGameTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ public void runTest(ClientGameTestContext context) {
2626
.create()) {
2727
singleplayer.getClientWorld().waitForChunksRender();
2828

29-
// Let the world tick a little with the HUD rendering before capturing evidence.
3029
context.waitTicks(40);
3130
assertScreenshotSaved(context.takeScreenshot("betterhud-survival-world"));
3231
}

settings.gradle

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -21,28 +21,16 @@ plugins {
2121
id 'dev.kikugie.stonecutter' version '0.9.3'
2222
}
2323

24-
// Every supported Minecraft version, and the build variant that covers it,
25-
// lives in supported-versions.json — the single source of truth also used by
26-
// the launch tests, the CI matrices, and the modrinthBundle task.
2724
def supportedVersions = new groovy.json.JsonSlurper().parse(file('supported-versions.json'))
2825
def variants = supportedVersions.values()*.variant.unique()
2926

3027
stonecutter {
3128
create(rootProject) {
32-
// Each variant is one buildable subproject; the string is both the
33-
// subproject name and the MC version it compiles against (see
34-
// versions/<variant>/gradle.properties).
3529
versions(*variants)
36-
37-
// vcsVersion is the variant whose form the sources are committed in
38-
// (the "active" version when nothing else is selected).
3930
vcsVersion = variants.last()
4031
}
4132
}
4233

4334
rootProject.name = 'betterhud'
4435

45-
// Not a Stonecutter variant: a sourceless harness that launches a production
46-
// Fabric client for any supported MC version (-PtestMcVersion) with the
47-
// matching variant's built jar. Used by the CI launch-test matrix.
4836
include 'launchtest'

src/gametest/java/dsns/betterhud/gametest/BetterHudClientGameTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ public void runTest(ClientGameTestContext context) {
2121
.create()) {
2222
singleplayer.getClientLevel().waitForChunksRender();
2323

24-
// Let the world tick a little with the HUD rendering before capturing evidence.
2524
context.waitTicks(40);
2625
assertScreenshotSaved(context.takeScreenshot("betterhud-survival-world"));
2726
}

src/main/java/dsns/betterhud/BetterHUD.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,6 @@
2323

2424
@Environment(EnvType.CLIENT)
2525
public class BetterHUD implements ClientModInitializer {
26-
27-
// This logger is used to write text to the console and the log file.
28-
// It is considered best practice to use your mod id as the logger's name.
29-
// That way, it's clear which mod wrote info, warnings, and errors.
3026
public static final Logger LOGGER = LoggerFactory.getLogger("betterhud");
3127

3228
public static ArrayList<BaseMod> mods = new ArrayList<>(

src/main/java/dsns/betterhud/ModMenu.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ public ConfigScreenFactory<?> getModConfigScreenFactory() {
2424
.setParentScreen(parent)
2525
.setTitle(Component.literal("BetterHUD Settings"));
2626

27-
// same as builder.setSavingRunnable(() -> Config.serialize());
2827
builder.setSavingRunnable(Config::serialize);
2928

3029
ConfigEntryBuilder entryBuilder = builder.entryBuilder();

src/main/java/dsns/betterhud/mods/Biome.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,8 @@ public CustomText onStartTick(Minecraft client) {
3636

3737
if (player == null) return null;
3838

39-
// have to specify this because name of class is Biome
4039
//? if >=26 {
40+
// must use long type because name of class is Biome
4141
Holder<net.minecraft.world.level.biome.Biome> biome = client.level.getBiome(player.blockPosition());
4242

4343
if (!biome.unwrapKey().isPresent()) return null;
@@ -99,17 +99,14 @@ public CustomText onStartTick(Minecraft client) {
9999
}
100100

101101
public String formatSnakeCase(String biomeName) {
102-
// Split the string by underscores
103102
String[] words = biomeName.split("_");
104103

105-
// Capitalize each word
106104
for (int i = 0; i < words.length; i++) {
107105
words[i] =
108106
words[i].substring(0, 1).toUpperCase() +
109107
words[i].substring(1).toLowerCase();
110108
}
111109

112-
// Join the words with spaces
113110
return String.join(" ", words);
114111
}
115112
}

0 commit comments

Comments
 (0)