From a0d9be18b735ec95b159c0c096d7daac0e40af9b Mon Sep 17 00:00:00 2001
From: sla-ppy <sla-ppy-pm@protonmail.com>
Date: Wed, 14 Feb 2024 13:51:37 +0100
Subject: [PATCH] documentate unicycle dirty fix

---
 src/Client.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/Client.cpp b/src/Client.cpp
index 742c982..af40243 100644
--- a/src/Client.cpp
+++ b/src/Client.cpp
@@ -120,13 +120,13 @@ void TClient::SetCarData(int Ident, const std::string& Data) {
     DeleteCar(Ident);
 }
 
-// TODO: count unicycles and vehicles different #246    sla-ppy
-// PROPOSAL: separate unicycles completely from vehicles???
 int TClient::GetCarCount() const {
+    // mVechileData holds both unicycle and cars which both count towards the maximum car count
+    // spawning a unicycle meant reaching the max, hence being unable to spawn car. this dirty fixes the problem for now.
     std::unique_lock lock(mVehicleDataMutex);
     for (auto& v : mVehicleData) {
         if (v.ID() == mUnicycleID) {
-            return int(mVehicleData.size() - 1); // dont keep track of unicycles
+            return int(mVehicleData.size() - 1);
         }
     }
     return int(mVehicleData.size());