From dca2661ccbecebc9d123ea7090a73547275def40 Mon Sep 17 00:00:00 2001
From: Frank van der Heijden <22407829+FrankHeijden@users.noreply.github.com>
Date: Mon, 17 Aug 2020 23:21:09 +0200
Subject: [PATCH] Don't set double drop if result is already a full stack
 (#4263)

Thanks, merging because I can't think of anything bad that would result from this change.
---
 .../java/com/gmail/nossr50/skills/smelting/SmeltingManager.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/com/gmail/nossr50/skills/smelting/SmeltingManager.java b/src/main/java/com/gmail/nossr50/skills/smelting/SmeltingManager.java
index 9d622ea62..518718765 100644
--- a/src/main/java/com/gmail/nossr50/skills/smelting/SmeltingManager.java
+++ b/src/main/java/com/gmail/nossr50/skills/smelting/SmeltingManager.java
@@ -113,7 +113,7 @@ public class SmeltingManager extends SkillManager {
         applyXpGain(Smelting.getResourceXp(smelting), XPGainReason.PVE, XPGainSource.PASSIVE);
 
         if (Config.getInstance().getDoubleDropsEnabled(PrimarySkillType.SMELTING, result.getType())
-                && isSecondSmeltSuccessful()) {
+                && isSecondSmeltSuccessful() && result.getAmount() < 64) {
             ItemStack newResult = result.clone();
 
             newResult.setAmount(result.getAmount() + 1);