Skip to content

Commit 6b5814d

Browse files
Russell Harlandclaude
andcommitted
Fix infinite config correction loop for scarfOpacity
TOML parses all floating-point values as Double, but scarfOpacity was defined as Float (0.4f). NeoForge's config validator sees the type mismatch (Double != Float), rewrites the file, the FileWatcher detects the write, re-validates, finds it wrong again — infinite loop every 2s. Changed scarfOpacity from Float to Double to match TOML's native type. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 41e076c commit 6b5814d

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/main/java/com/sammy/malum/config/ClientConfig.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,9 @@ public class ClientConfig extends LodestoneConfig {
2828
public static ConfigValueHolder<Integer> SCARF_LENGTH = new ConfigValueHolder<>(MALUM, "client/scarf", (builder ->
2929
builder.comment("How long should the Malignant Stronghold Scarf be? This value will affect all rendered scarves, not just the one worn by the local player.")
3030
.define("scarfLength", 30)));
31-
public static ConfigValueHolder<Float> SCARF_OPACITY = new ConfigValueHolder<>(MALUM, "client/scarf", (builder ->
32-
builder.comment("How visible. should the Malignant Stronghold Scarf be in first person?")
33-
.define("scarfOpacity", 0.4f)));
31+
public static ConfigValueHolder<Double> SCARF_OPACITY = new ConfigValueHolder<>(MALUM, "client/scarf", (builder ->
32+
builder.comment("How visible should the Malignant Stronghold Scarf be in first person?")
33+
.define("scarfOpacity", 0.4d)));
3434

3535
public static ConfigValueHolder<Boolean> PARALLEL_WORLD = new ConfigValueHolder<>(MALUM, "client/renderpass", (builder ->
3636
builder.comment("Enable or disable the parallel world renderer, used for various effects related to a certain feature. Disable if it causes framerate issues.")

0 commit comments

Comments
 (0)