Remove a useless variable and fix doc

Phobos-developers · Feb 24, 2025 · 13cdc0c · 13cdc0c
1 parent 00752e4
commit 13cdc0c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/docs/Fixed-or-Improved-Logics.md b/docs/Fixed-or-Improved-Logics.md
@@ -1350,8 +1350,6 @@ NoTurret.TrackTarget=false   ; boolean
 
 [SOMEUNIT]                   ; UnitType
 NoTurret.TrackTarget=        ; boolean, defaults to [General]->NoTurret.TrackTarget
-```{warning}
-Due to technical constraints, units that use `Convert.Deploy` from [Ares’ Type Conversion](https://ares-developers.github.io/Ares-docs/new/typeconversion.html) to change type with `Ammo.AddOnDeploy` will add or substract ammo despite of convertion success. This will also happen when unit exits tank bunker.
 ```
 
 ## Veinholes & Weeds

diff --git a/src/Ext/Techno/Hooks.ReceiveDamage.cpp b/src/Ext/Techno/Hooks.ReceiveDamage.cpp
@@ -25,7 +25,6 @@ DEFINE_HOOK(0x701900, TechnoClass_ReceiveDamage_Shield, 0x6)
 	const auto pType = pTypeExt->OwnerObject();
 	const auto pWHExt = WarheadTypeExt::ExtMap.Find(args->WH);
 
-	int nDamageLeft = *args->Damage;
 	const auto pSourceHouse = args->SourceHouse;
 	const auto pTargetHouse = pThis->Owner;
 
@@ -50,7 +49,7 @@ DEFINE_HOOK(0x701900, TechnoClass_ReceiveDamage_Shield, 0x6)
 	}
 
 	// Raise Combat Alert
-	if (pRules->CombatAlert && nDamageLeft > 1)
+	if (pRules->CombatAlert && *args->Damage > 1)
 	{
 		auto raiseCombatAlert = [&]()
 		{
@@ -108,7 +107,7 @@ DEFINE_HOOK(0x701900, TechnoClass_ReceiveDamage_Shield, 0x6)
 			if (!pShieldData->IsActive())
 				return 0;
 
-			nDamageLeft = pShieldData->ReceiveDamage(args);
+			int nDamageLeft = pShieldData->ReceiveDamage(args);
 
 			if (nDamageLeft >= 0)
 			{