From 4552b17691fa7c4f1fdb4ca7652ca48278a68fec Mon Sep 17 00:00:00 2001
From: Quipyowert2 <38995150+Quipyowert2@users.noreply.github.com>
Date: Sun, 16 Feb 2020 17:57:22 -0800
Subject: Codechange: Avoid copying function parameters by using const
 references

---
 src/ai/ai_gui.cpp   | 2 +-
 src/company_cmd.cpp | 6 +++---
 src/company_func.h  | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/ai/ai_gui.cpp b/src/ai/ai_gui.cpp
index a0552628c..d61cfea0d 100644
--- a/src/ai/ai_gui.cpp
+++ b/src/ai/ai_gui.cpp
@@ -589,7 +589,7 @@ struct AISettingsWindow : public Window {
 	}
 
 private:
-	bool IsEditableItem(const ScriptConfigItem config_item) const
+	bool IsEditableItem(const ScriptConfigItem &config_item) const
 	{
 		return _game_mode == GM_MENU || ((this->slot != OWNER_DEITY) && !Company::IsValidID(this->slot)) || (config_item.flags & SCRIPTCONFIG_INGAME) != 0;
 	}
diff --git a/src/company_cmd.cpp b/src/company_cmd.cpp
index 6308c1447..835a7c4a8 100644
--- a/src/company_cmd.cpp
+++ b/src/company_cmd.cpp
@@ -208,7 +208,7 @@ bool CheckCompanyHasMoney(CommandCost &cost)
  * @param c Company to pay the bill.
  * @param cost Money to pay.
  */
-static void SubtractMoneyFromAnyCompany(Company *c, CommandCost cost)
+static void SubtractMoneyFromAnyCompany(Company *c, const CommandCost &cost)
 {
 	if (cost.GetCost() == 0) return;
 	assert(cost.GetExpensesType() != INVALID_EXPENSES);
@@ -237,7 +237,7 @@ static void SubtractMoneyFromAnyCompany(Company *c, CommandCost cost)
  * Subtract money from the #_current_company, if the company is valid.
  * @param cost Money to pay.
  */
-void SubtractMoneyFromCompany(CommandCost cost)
+void SubtractMoneyFromCompany(const CommandCost &cost)
 {
 	Company *c = Company::GetIfValid(_current_company);
 	if (c != nullptr) SubtractMoneyFromAnyCompany(c, cost);
@@ -248,7 +248,7 @@ void SubtractMoneyFromCompany(CommandCost cost)
  * @param company Company paying the bill.
  * @param cst     Cost of a command.
  */
-void SubtractMoneyFromCompanyFract(CompanyID company, CommandCost cst)
+void SubtractMoneyFromCompanyFract(CompanyID company, const CommandCost &cst)
 {
 	Company *c = Company::Get(company);
 	byte m = c->money_fraction;
diff --git a/src/company_func.h b/src/company_func.h
index cec611004..01f585910 100644
--- a/src/company_func.h
+++ b/src/company_func.h
@@ -25,8 +25,8 @@ void CompanyAdminBankrupt(CompanyID company_id);
 void UpdateLandscapingLimits();
 
 bool CheckCompanyHasMoney(CommandCost &cost);
-void SubtractMoneyFromCompany(CommandCost cost);
-void SubtractMoneyFromCompanyFract(CompanyID company, CommandCost cost);
+void SubtractMoneyFromCompany(const CommandCost& cost);
+void SubtractMoneyFromCompanyFract(CompanyID company, const CommandCost& cost);
 CommandCost CheckOwnership(Owner owner, TileIndex tile = 0);
 CommandCost CheckTileOwnership(TileIndex tile);
 
-- 
cgit v1.2.3-54-g00ecf