match leij's backwards compat suggestion

This commit is contained in:
Wagyourtail 2021-10-27 13:45:30 -06:00
parent 084798bd1b
commit 2980bb0320
3 changed files with 14 additions and 13 deletions

View File

@ -190,6 +190,13 @@ public final class Settings {
/** /**
* Blocks that Baritone is not allowed to break * Blocks that Baritone is not allowed to break
*/ */
public final Setting<List<Block>> blocksToDisallowBreaking = new Setting<>(new ArrayList<>(
// Leave Empty by Default
));
/**
* blocks that baritone shouldn't break, but can if it needs to.
*/
public final Setting<List<Block>> blocksToAvoidBreaking = new Setting<>(new ArrayList<>(Arrays.asList( // TODO can this be a HashSet or ImmutableSet? public final Setting<List<Block>> blocksToAvoidBreaking = new Setting<>(new ArrayList<>(Arrays.asList( // TODO can this be a HashSet or ImmutableSet?
Blocks.CRAFTING_TABLE, Blocks.CRAFTING_TABLE,
Blocks.FURNACE, Blocks.FURNACE,
@ -200,16 +207,10 @@ public final class Settings {
Blocks.WALL_SIGN Blocks.WALL_SIGN
))); )));
/**
* blocks that baritone shouldn't break, but can if it needs to.
*/
public final Setting<List<Block>> blocksToDiscourageBreaking = new Setting<>(new ArrayList<>(Arrays.asList(
)));
/** /**
* this multiplies the break speed, if set above 1 it's "encourage breaking" instead * this multiplies the break speed, if set above 1 it's "encourage breaking" instead
*/ */
public final Setting<Double> discourageBreakingMultiplier = new Setting<>(.1); public final Setting<Double> avoidBreakingMultiplier = new Setting<>(.1);
/** /**
* A list of blocks to be treated as if they're air. * A list of blocks to be treated as if they're air.

View File

@ -50,7 +50,7 @@ public interface MovementHelper extends ActionCosts, Helper {
static boolean avoidBreaking(BlockStateInterface bsi, int x, int y, int z, IBlockState state) { static boolean avoidBreaking(BlockStateInterface bsi, int x, int y, int z, IBlockState state) {
Block b = state.getBlock(); Block b = state.getBlock();
return Baritone.settings().blocksToAvoidBreaking.value.contains(b) return Baritone.settings().blocksToDisallowBreaking.value.contains(b)
|| b == Blocks.ICE // ice becomes water, and water can mess up the path || b == Blocks.ICE // ice becomes water, and water can mess up the path
|| b instanceof BlockSilverfish // obvious reasons || b instanceof BlockSilverfish // obvious reasons
// call context.get directly with x,y,z. no need to make 5 new BlockPos for no reason // call context.get directly with x,y,z. no need to make 5 new BlockPos for no reason

View File

@ -165,7 +165,7 @@ public class ToolSet {
} }
private double avoidanceMultiplier(Block b) { private double avoidanceMultiplier(Block b) {
return Baritone.settings().blocksToDiscourageBreaking.value.contains(b) ? Baritone.settings().discourageBreakingMultiplier.value : 1; return Baritone.settings().blocksToAvoidBreaking.value.contains(b) ? Baritone.settings().avoidBreakingMultiplier.value : 1;
} }
/** /**