Skip to content

Commit 7602cfa

Browse files
committed
move important members of option builder subinterfaces more useful
1 parent 7e3d50f commit 7602cfa

File tree

3 files changed

+24
-24
lines changed

3 files changed

+24
-24
lines changed

common/src/api/java/net/caffeinemc/mods/sodium/api/config/structure/EnumOptionBuilder.java

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,16 @@
1313
import java.util.function.Supplier;
1414

1515
public interface EnumOptionBuilder<E extends Enum<E>> extends OptionBuilder<E> {
16+
static <E extends Enum<E>> Function<E, Component> nameProviderFrom(Component... names) {
17+
return e -> names[e.ordinal()];
18+
}
19+
20+
EnumOptionBuilder<E> setAllowedValues(Set<E> allowedValues);
21+
22+
EnumOptionBuilder<E> setAllowedValuesProvider(Function<ConfigState, Set<E>> provider, ResourceLocation... dependencies);
23+
24+
EnumOptionBuilder<E> setElementNameProvider(Function<E, Component> provider);
25+
1626
@Override
1727
EnumOptionBuilder<E> setName(Component name);
1828

@@ -48,14 +58,4 @@ public interface EnumOptionBuilder<E extends Enum<E>> extends OptionBuilder<E> {
4858

4959
@Override
5060
EnumOptionBuilder<E> setBinding(OptionBinding<E> binding);
51-
52-
EnumOptionBuilder<E> setAllowedValues(Set<E> allowedValues);
53-
54-
EnumOptionBuilder<E> setAllowedValuesProvider(Function<ConfigState, Set<E>> provider, ResourceLocation... dependencies);
55-
56-
EnumOptionBuilder<E> setElementNameProvider(Function<E, Component> provider);
57-
58-
static <E extends Enum<E>> Function<E, Component> nameProviderFrom(Component... names) {
59-
return e -> names[e.ordinal()];
60-
}
6161
}

common/src/api/java/net/caffeinemc/mods/sodium/api/config/structure/IntegerOptionBuilder.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,14 @@
1010
import java.util.function.Supplier;
1111

1212
public interface IntegerOptionBuilder extends OptionBuilder<Integer> {
13+
IntegerOptionBuilder setRange(int min, int max, int step);
14+
15+
IntegerOptionBuilder setRange(Range range);
16+
17+
IntegerOptionBuilder setRangeProvider(Function<ConfigState, Range> provider, ResourceLocation... dependencies);
18+
19+
IntegerOptionBuilder setValueFormatter(ControlValueFormatter formatter);
20+
1321
@Override
1422
IntegerOptionBuilder setName(Component name);
1523

@@ -45,12 +53,4 @@ public interface IntegerOptionBuilder extends OptionBuilder<Integer> {
4553

4654
@Override
4755
IntegerOptionBuilder setBinding(OptionBinding<Integer> binding);
48-
49-
IntegerOptionBuilder setRange(int min, int max, int step);
50-
51-
IntegerOptionBuilder setRange(Range range);
52-
53-
IntegerOptionBuilder setRangeProvider(Function<ConfigState, Range> provider, ResourceLocation... dependencies);
54-
55-
IntegerOptionBuilder setValueFormatter(ControlValueFormatter formatter);
5656
}

common/src/main/java/net/caffeinemc/mods/sodium/client/config/structure/IntegerOptionBuilderImpl.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,12 @@ public IntegerOptionBuilder setRangeProvider(Function<ConfigState, Range> provid
5757
return this;
5858
}
5959

60+
@Override
61+
public IntegerOptionBuilder setValueFormatter(ControlValueFormatter formatter) {
62+
this.valueFormatter = formatter;
63+
return this;
64+
}
65+
6066
@Override
6167
public IntegerOptionBuilder setName(Component name) {
6268
super.setName(name);
@@ -128,10 +134,4 @@ public IntegerOptionBuilder setBinding(OptionBinding<Integer> binding) {
128134
super.setBinding(binding);
129135
return this;
130136
}
131-
132-
@Override
133-
public IntegerOptionBuilder setValueFormatter(ControlValueFormatter formatter) {
134-
this.valueFormatter = formatter;
135-
return this;
136-
}
137137
}

0 commit comments

Comments
 (0)