Expanded the engine perk options in perks.ini

artifacts/config_files/Perks.ini

@@ -7,10 +7,39 @@
 ; be used to remove their hardcoded effects, and add new stat/skill effects
 
 [PerksTweak]
+;Changes the Outdoorsman skill bonus for 'Survivalist' perk (ID 16)
+;125 - maximum bonus, 25 - default bonus
+SurvivalistBonus=25
+
 ;Changes the bonus for 'Master Trader' perk (ID 17)
 ;25 - default bonus
 MasterTraderBonus=25
 
+;Changes the Science and Repair skill bonus for 'Mr.Fixit' perk (ID 31)
+;125 - maximum bonus, 10 - default bonus
+MrFixitBonus=10
+
+;Changes the skill bonuses for 'Medic' perk (ID 32)
+;125 - maximum bonus, 10 - default bonus
+MedicFirstAidBonus=10
+MedicDoctorBonus=10
+
+;Changes the Lockpick and Steal skill bonus for 'Master Thief' perk (ID 33)
+;125 - maximum bonus, 15 - default bonus
+MasterThiefBonus=15
+
+;Changes the Speech skill bonus for 'Speaker' perk (ID 34)
+;125 - maximum bonus, 20 - default bonus
+SpeakerBonus=20
+
+;Changes the Sneak skill bonus (when in darkness) for 'Ghost' perk (ID 38)
+;125 - maximum bonus, 20 - default bonus
+GhostBonus=20
+
+;Changes the Outdoorsman skill bonus for 'Ranger' perk (ID 47)
+;125 - maximum bonus, 15 - default bonus
+RangerOutdoorsmanBonus=15
+
 ;Changes the distance bonus for 'Weapon Long Range' weapon perk (ID 58)
 ;2 - no bonus, 4 - default bonus
 WeaponLongRangeBonus=4
@@ -34,9 +63,22 @@ VaultCityInoculationsRadBonus=10
 ;999 - maximum bonus, 10 - default bonus
 DemolitionExpertBonus=10
 
-;Changes the damage bonus for 'Living Anatomy' perk (ID 97)
+;Changes the Gambling skill bonus for 'Gambler' perk (ID 83)
-;125 - maximum bonus, 5 - default bonus
+;125 - maximum bonus, 20 - default bonus
+GamblerBonus=20
+
+;Changes the Steal skill bonus for 'Harmless' perk (ID 91)
+;125 - maximum bonus, 20 - default bonus
+HarmlessBonus=20
+
+;Changes the damage bonus and the Doctor skill bonus for 'Living Anatomy' perk (ID 97)
+;125 - maximum bonus, 5 - default bonus, 10 - default skill bonus
 LivingAnatomyBonus=5
+LivingAnatomyDoctorBonus=10
+
+;Changes the Speech and Barter skill bonus for 'Negotiator' perk (ID 99)
+;125 - maximum bonus, 10 - default bonus
+NegotiatorBonus=10
 
 ;Changes the damage bonus for 'Pyromaniac' perk (ID 101)
 ;125 - maximum bonus, 5 - default bonus
@@ -50,10 +92,23 @@ SalesmanBonus=20
 ;valid range: 0..100, 50 - default percent
 StonewallPercent=50
 
+;Changes the Lockpick, Steal, and Traps skill bonus for 'Thief' perk (ID 105)
+;125 - maximum bonus, 10 - default bonus
+ThiefBonus=10
+
 ;Changes the strength bonus for 'Weapon Handling' perk (ID 106)
 ;0 - no bonus, 10 - maximum bonus, 3 - default bonus
 WeaponHandlingBonus=3
 
+;Changes the skill bonuses for 'Vault City Training' perk (ID 107)
+;125 - maximum bonus, 5 - default bonus
+VaultCityTrainingFirstAidBonus=5
+VaultCityTrainingDoctorBonus=5
+
+;Changes the Speech skill bonus for 'Expert Excrement Expeditor' perk (ID 116)
+;125 - maximum bonus, 5 - default bonus
+ExpertExcrementExpeditorBonus=5
+
 ;##############################################################################
 [Perks]
 ;Set to 1 to enable the modifications for perks

sfall/Modules/SubModules/EnginePerks.cpp

@@ -17,6 +17,7 @@
  */
 
 #include "..\..\main.h"
+#include "..\..\Utils.h"
 #include "..\..\FalloutEngine\Fallout2.h"
 
 #include "EnginePerks.h"
@@ -26,141 +27,39 @@ namespace sfall
 namespace perk
 {
 
-static class EnginePerkBonus {
+static long SalesmanBonus = 20;
-public:
+static long DemolitionExpertBonus = 10;
-	long WeaponScopeRangePenalty;
-	long WeaponScopeRangeBonus;
-	long WeaponLongRangeBonus;
-	long WeaponAccurateBonus;
-	long WeaponHandlingBonus;
 
-	float MasterTraderBonus;
+static bool TryGetModifiedInt(const char* key, int defaultValue, int& outValue, const char* perksFile) {
-	long SalesmanBonus;
+	outValue = IniReader::GetInt("PerksTweak", key, defaultValue, perksFile);
+	return outValue != defaultValue;
+}
 
-	long LivingAnatomyBonus;
+static void TryPatchValue8(const char* key, int defaultValue, int minValue, int maxValue, DWORD addr, const char* perksFile) {
-	long PyromaniacBonus;
+	int value;
-
+	if (TryGetModifiedInt(key, defaultValue, value, perksFile) && value >= minValue) {
-	long StonewallPercent;
+		SafeWrite8(addr, static_cast<BYTE>(min(value, maxValue)));
-
-	long DemolitionExpertBonus;
-
-	long VaultCityInoculationsPoisonBonus;
-	long VaultCityInoculationsRadBonus;
-
-	EnginePerkBonus() {
-		WeaponScopeRangePenalty = 8;
-		WeaponScopeRangeBonus   = 5;
-		WeaponLongRangeBonus    = 4;
-		WeaponAccurateBonus     = 20;
-		WeaponHandlingBonus     = 3;
-
-		MasterTraderBonus       = 25;
-		SalesmanBonus           = 20;
-
-		LivingAnatomyBonus      = 5;
-		PyromaniacBonus         = 5;
-
-		StonewallPercent        = 50;
-
-		DemolitionExpertBonus   = 10;
-
-		VaultCityInoculationsPoisonBonus = 10;
-		VaultCityInoculationsRadBonus    = 10;
 	}
+}
 
-	///////////////////////////////////////////
+static void TryPatchValue32(const char* key, int defaultValue, int minValue, int maxValue, DWORD addr, const char* perksFile) {
-
+	int value;
-	void setWeaponScopeRangePenalty(long value) {
+	if (TryGetModifiedInt(key, defaultValue, value, perksFile) && value >= minValue) {
-		if (value < 0) return;
+		SafeWrite32(addr, min(value, maxValue));
-		WeaponScopeRangePenalty = value;
-		SafeWrite32(0x42448E, value);
 	}
+}
 
-	void setWeaponScopeRangeBonus(long value) {
+static void TryPatchSkillBonus8(const char* key, int defaultValue, DWORD addr, const char* perksFile) {
-		if (value < 2) return;
+	TryPatchValue8(key, defaultValue, 0, 125, addr, perksFile);
-		WeaponScopeRangeBonus = value;
+}
-		SafeWrite32(0x424489, value);
-	}
 
-	void setWeaponLongRangeBonus(long value) {
+static void TryPatchSkillBonus32(const char* key, int defaultValue, DWORD addr, const char* perksFile) {
-		if (value < 2) return;
+	TryPatchValue32(key, defaultValue, 0, 125, addr, perksFile);
-		WeaponLongRangeBonus = value;
+}
-		SafeWrite32(0x424474, value);
-	}
-
-	void setWeaponAccurateBonus(long value) {
-		if (value < 0) return;
-		WeaponAccurateBonus = value;
-		if (WeaponAccurateBonus > 125) WeaponAccurateBonus = 125;
-		SafeWrite8(0x42465D, static_cast<BYTE>(WeaponAccurateBonus));
-	}
-
-	void setWeaponHandlingBonus(long value) {
-		if (value < 0) return;
-		WeaponHandlingBonus = value;
-		if (WeaponHandlingBonus > 10) WeaponHandlingBonus = 10;
-		SafeWrite8(0x424636, static_cast<char>(WeaponHandlingBonus));
-		SafeWrite8(0x4251CE, static_cast<signed char>(-WeaponHandlingBonus));
-	}
-
-	void setMasterTraderBonus(long value) {
-		if (value < 0) return;
-		MasterTraderBonus = static_cast<float>(value);
-		SafeWrite32(0x474BB3, *(DWORD*)&MasterTraderBonus); // write float data
-	}
-
-	void setSalesmanBonus(long value) {
-		if (value < 0) return;
-		SalesmanBonus = value;
-		if (SalesmanBonus > 999) SalesmanBonus = 999;
-	}
-
-	void setLivingAnatomyBonus(long value) {
-		if (value < 0) return;
-		LivingAnatomyBonus = value;
-		if (LivingAnatomyBonus > 125) LivingAnatomyBonus = 125;
-		SafeWrite8(0x424A91, static_cast<BYTE>(LivingAnatomyBonus));
-	}
-
-	void setPyromaniacBonus(long value) {
-		if (value < 0) return;
-		PyromaniacBonus = value;
-		if (PyromaniacBonus > 125) PyromaniacBonus = 125;
-		SafeWrite8(0x424AB6, static_cast<BYTE>(PyromaniacBonus));
-	}
-
-	void setStonewallPercent(long value) {
-		if (value < 0) return;
-		StonewallPercent = value;
-		if (StonewallPercent > 100) StonewallPercent = 100;
-		SafeWrite8(0x424B50, static_cast<BYTE>(StonewallPercent));
-	}
-
-	void setDemolitionExpertBonus(long value) {
-		if (value < 0) return;
-		DemolitionExpertBonus = value;
-		if (DemolitionExpertBonus > 999) DemolitionExpertBonus = 999;
-	}
-
-	void setVaultCityInoculationsPoisonBonus(long value) {
-		if (value < -100) value = -100;
-		if (value > 100) value = 100;
-		VaultCityInoculationsPoisonBonus = value;
-		SafeWrite8(0x4AF26A, static_cast<signed char>(VaultCityInoculationsPoisonBonus));
-	}
-
-	void setVaultCityInoculationsRadBonus(long value) {
-		if (value < -100) value = -100;
-		if (value > 100) value = 100;
-		VaultCityInoculationsRadBonus = value;
-		SafeWrite8(0x4AF287, static_cast<signed char>(VaultCityInoculationsRadBonus));
-	}
-
-} perks;
 
 static void __declspec(naked) perk_adjust_skill_hack_salesman() {
 	__asm {
-		imul eax, [perks.SalesmanBonus];
+		imul eax, [SalesmanBonus];
 		add  ecx, eax; // barter_skill + (perkLevel * SalesmanBonus)
 		mov  eax, ecx
 		retn;
@@ -169,7 +68,7 @@ static void __declspec(naked) perk_adjust_skill_hack_salesman() {
 
 static void __declspec(naked) queue_explode_exit_hack_demolition_expert() {
 	__asm {
-		imul eax, [perks.DemolitionExpertBonus];
+		imul eax, [DemolitionExpertBonus];
 		add  ecx, eax; // maxBaseDmg + (perkLevel * DemolitionExpertBonus)
 		add  ebx, eax  // minBaseDmg + (perkLevel * DemolitionExpertBonus)
 		retn;
@@ -183,42 +82,52 @@ void EnginePerkBonusInit() {
 }
 
 void ReadPerksBonuses(const char* perksFile) {
-	int wScopeRangeMod = IniReader::GetInt("PerksTweak", "WeaponScopeRangePenalty", 8, perksFile);
+	int value;
-	if (wScopeRangeMod != 8) perks.setWeaponScopeRangePenalty(wScopeRangeMod);
+	TryPatchValue32("WeaponScopeRangePenalty", 8, 0, 100, 0x42448E, perksFile);
-	wScopeRangeMod = IniReader::GetInt("PerksTweak", "WeaponScopeRangeBonus", 5, perksFile);
+	TryPatchValue32("WeaponScopeRangeBonus", 5, 2, 100, 0x424489, perksFile);
-	if (wScopeRangeMod != 5) perks.setWeaponScopeRangeBonus(wScopeRangeMod);
+	TryPatchValue32("WeaponLongRangeBonus", 4, 2, 100, 0x424474, perksFile);
+	TryPatchSkillBonus8("WeaponAccurateBonus", 20, 0x42465D, perksFile);
+	if (TryGetModifiedInt("WeaponHandlingBonus", 3, value, perksFile) && value >= 0) {
+		if (value > 10) value = 10;
+		SafeWrite8(0x424636, static_cast<char>(value));
+		SafeWrite8(0x4251CE, static_cast<signed char>(-value));
+	}
+	if (TryGetModifiedInt("MasterTraderBonus", 25, value, perksFile) && value >= 0) {
+		float floatValue = static_cast<float>(value);
+		SafeWrite32(0x474BB3, *(DWORD*)&floatValue); // write float data
+	}
+	if (TryGetModifiedInt("SalesmanBonus", SalesmanBonus, value, perksFile) && value >= 0) {
+		SalesmanBonus = min(value, 999);
+	}
+	TryPatchSkillBonus8("LivingAnatomyBonus", 5, 0x424A91, perksFile);
+	TryPatchSkillBonus8("LivingAnatomyDoctorBonus", 10, 0x496E66, perksFile);
+	TryPatchSkillBonus8("PyromaniacBonus", 5, 0x424AB6, perksFile);
+	TryPatchValue8("StonewallPercent", 50, 0, 100, 0x424B50, perksFile);
+	if (TryGetModifiedInt("DemolitionExpertBonus", DemolitionExpertBonus, value, perksFile) && value >= 0) {
+		DemolitionExpertBonus = min(value, 999);
+	}
+	if (TryGetModifiedInt("VaultCityInoculationsPoisonBonus", 10, value, perksFile)) {
+		SafeWrite8(0x4AF26A, static_cast<signed char>(clamp<long>(value, -100, 100)));
+	}
+	if (TryGetModifiedInt("VaultCityInoculationsRadBonus", 10, value, perksFile)) {
+		SafeWrite8(0x4AF287, static_cast<signed char>(clamp<long>(value, -100, 100)));
+	}
+	TryPatchSkillBonus8("VaultCityTrainingFirstAidBonus", 5, 0x496E35, perksFile);
+	TryPatchSkillBonus8("VaultCityTrainingDoctorBonus", 5, 0x496E7F, perksFile);
+	TryPatchSkillBonus32("MedicFirstAidBonus", 10, 0x496E19, perksFile);
+	TryPatchSkillBonus32("MedicDoctorBonus", 10, 0x496E4E, perksFile);
 
-	int wLongRangeBonus = IniReader::GetInt("PerksTweak", "WeaponLongRangeBonus", 4, perksFile);
+	TryPatchSkillBonus32("GhostBonus", 20, 0x496EA9, perksFile);
-	if (wLongRangeBonus != 4) perks.setWeaponLongRangeBonus(wLongRangeBonus);
+	TryPatchSkillBonus8("ThiefBonus", 10, 0x496EC1, perksFile);
-
+	TryPatchSkillBonus8("MasterThiefBonus", 15, 0x496EE0, perksFile);
-	int wAccurateBonus = IniReader::GetInt("PerksTweak", "WeaponAccurateBonus", 20, perksFile);
+	TryPatchSkillBonus8("HarmlessBonus", 20, 0x496F02, perksFile);
-	if (wAccurateBonus != 20) perks.setWeaponAccurateBonus(wAccurateBonus);
+	TryPatchSkillBonus32("SpeakerBonus", 20, 0x496F1B, perksFile);
-
+	TryPatchSkillBonus8("ExpertExcrementExpeditorBonus", 5, 0x496F33, perksFile);
-	int wHandlingBonus = IniReader::GetInt("PerksTweak", "WeaponHandlingBonus", 3, perksFile);
+	TryPatchSkillBonus8("NegotiatorBonus", 10, 0x496F48, perksFile);
-	if (wHandlingBonus != 3) perks.setWeaponHandlingBonus(wHandlingBonus);
+	TryPatchSkillBonus32("GamblerBonus", 20, 0x496F79, perksFile);
-
+	TryPatchSkillBonus32("RangerOutdoorsmanBonus", 15, 0x496F95, perksFile);
-	int masterTraderBonus = IniReader::GetInt("PerksTweak", "MasterTraderBonus", 25, perksFile);
+	TryPatchSkillBonus8("SurvivalistBonus", 25, 0x496FAB, perksFile);
-	if (masterTraderBonus != 25) perks.setMasterTraderBonus(masterTraderBonus);
+	TryPatchSkillBonus8("MrFixitBonus", 10, 0x496E00, perksFile);
-
-	int salesmanBonus = IniReader::GetInt("PerksTweak", "SalesmanBonus", 20, perksFile);
-	if (salesmanBonus != 20) perks.setSalesmanBonus(salesmanBonus);
-
-	int livingAnatomyBonus = IniReader::GetInt("PerksTweak", "LivingAnatomyBonus", 5, perksFile);
-	if (livingAnatomyBonus != 5) perks.setLivingAnatomyBonus(livingAnatomyBonus);
-
-	int pyromaniacBonus = IniReader::GetInt("PerksTweak", "PyromaniacBonus", 5, perksFile);
-	if (pyromaniacBonus != 5) perks.setPyromaniacBonus(pyromaniacBonus);
-
-	int stonewallPercent = IniReader::GetInt("PerksTweak", "StonewallPercent", 50, perksFile);
-	if (stonewallPercent != 50) perks.setStonewallPercent(stonewallPercent);
-
-	int demolitionExpertBonus = IniReader::GetInt("PerksTweak", "DemolitionExpertBonus", 10, perksFile);
-	if (demolitionExpertBonus != 10) perks.setDemolitionExpertBonus(demolitionExpertBonus);
-
-	int vaultCityInoculationsBonus = IniReader::GetInt("PerksTweak", "VaultCityInoculationsPoisonBonus", 10, perksFile);
-	if (vaultCityInoculationsBonus != 10) perks.setVaultCityInoculationsPoisonBonus(vaultCityInoculationsBonus);
-	vaultCityInoculationsBonus = IniReader::GetInt("PerksTweak", "VaultCityInoculationsRadBonus", 10, perksFile);
-	if (vaultCityInoculationsBonus != 10) perks.setVaultCityInoculationsRadBonus(vaultCityInoculationsBonus);
 }
 
 }

sfall/Utils.h

@@ -21,6 +21,13 @@ void split(const std::string &s, char delim, T result, size_t limit = -1) {
 	}
 }
 
+template <typename T>
+T clamp(T value, T min, T max) {
+	if (value < min) return min;
+	if (value > max) return max;
+	return value;
+}
+
 WORD  ByteSwapW(WORD w);
 DWORD ByteSwapD(DWORD dw);