Skip to content

Optimize Villager food inventory checks #185

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Smorki wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Optimize Villager food inventory checks #185

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1f94cfc
Optimize Villager food inventory checks
Smorki Jul 26, 2026
faea98d
Address review: remove inventory hoist, keep stream change
Smorki Aug 3, 2026
e193355
Address review: single pass inventory scan, short circuit canBreed, f…
Smorki Aug 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 35 additions & 0 deletions ...ver/minecraft-patches/sources/net/minecraft/world/entity/npc/villager/Villager.java.patch
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
--- a/net/minecraft/world/entity/npc/villager/Villager.java
+++ b/net/minecraft/world/entity/npc/villager/Villager.java
@@ -688,7 +_,9 @@

@Override
public boolean canBreed() {
- return this.foodLevel + this.countFoodPointsInInventory() >= 12 && !this.isSleeping() && this.getAge() == 0;
+ // Gale start - Local code optimization - skip inventory scan if food level sufficient
+ return (this.foodLevel >= 12 || this.foodLevel + this.countFoodPointsInInventory() >= 12) && !this.isSleeping() && this.getAge() == 0;
+ // Gale end - Local code optimization - skip inventory scan if food level sufficient
}

private boolean hungry() {
@@ -843,8 +_,20 @@
}

private int countFoodPointsInInventory() {
+ // Gale start - Local code optimization - single pass food point count
SimpleContainer inventory = this.getInventory();
- return FOOD_POINTS.entrySet().stream().mapToInt(entry -> inventory.countItem(entry.getKey()) * entry.getValue()).sum();
+ int total = 0;
+ for (int slot = 0; slot < inventory.getContainerSize(); slot++) {
+ ItemStack itemStack = inventory.getItem(slot);
+ if (!itemStack.isEmpty()) {
+ Integer points = FOOD_POINTS.get(itemStack.getItem());
+ if (points != null) {
+ total += points * itemStack.getCount();
+ }
+ }
+ }
+ return total;
+ // Gale end - Local code optimization - single pass food point count
}

public boolean hasFarmSeeds() {
Loading