From 52f809b8bb26d4c4fd3464169fbd521f31bcb38f Mon Sep 17 00:00:00 2001
From: krzys-h <krzys_h@interia.pl>
Date: Fri, 10 Jul 2015 13:40:29 +0200
Subject: [PATCH] Removed cmdline limit

---
 src/object/old_object.cpp         | 35 ++++++++++++++++---------------
 src/object/old_object.h           |  7 +++----
 src/object/old_object_interface.h |  4 ++--
 src/object/robotmain.cpp          |  2 +-
 4 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/src/object/old_object.cpp b/src/object/old_object.cpp
index 49df96fd..0b1abf3d 100644
--- a/src/object/old_object.cpp
+++ b/src/object/old_object.cpp
@@ -280,10 +280,7 @@ COldObject::COldObject(int id)
         m_partiSel[i] = -1;
     }
 
-    for (int i=0 ; i<OBJECTMAXCMDLINE ; i++ )
-    {
-        m_cmdLine[i] = NAN;
-    }
+    m_cmdLine.clear();
 
     DeleteAllCrashSpheres();
     m_globalSpherePos = Math::Vector(0.0f, 0.0f, 0.0f);
@@ -867,8 +864,6 @@ int COldObject::GetOption()
 void COldObject::Write(CLevelParserLine* line)
 {
     Math::Vector    pos;
-    float       value;
-    int         i;
 
     line->AddParam("camera", CLevelParserParamUPtr{new CLevelParserParam(GetCameraType())});
 
@@ -936,11 +931,8 @@ void COldObject::Write(CLevelParserLine* line)
 
     // Sets the parameters of the command line.
     CLevelParserParamVec cmdline;
-    for ( i=0 ; i<OBJECTMAXCMDLINE ; i++ )
+    for(float value : m_cmdLine)
     {
-        value = GetCmdLine(i);
-        if ( std::isnan(value) )  break;
-
         cmdline.push_back(CLevelParserParamUPtr{new CLevelParserParam(value)});
     }
     if (cmdline.size() > 0)
@@ -1010,7 +1002,6 @@ void COldObject::Read(CLevelParserLine* line)
         int i = 0;
         for (auto& p : line->GetParam("cmdline")->AsArray())
         {
-            if (i >= OBJECTMAXCMDLINE) break;
             SetCmdLine(i, p->AsFloat());
             i++;
         }
@@ -1652,16 +1643,26 @@ float COldObject::GetInfoReturn()
     return m_infoReturn;
 }
 
-bool COldObject::SetCmdLine(int rank, float value)
+void COldObject::SetCmdLine(unsigned int rank, float value)
 {
-    if ( rank < 0 || rank >= OBJECTMAXCMDLINE )  return false;
-    m_cmdLine[rank] = value;
-    return true;
+    if (rank == m_cmdLine.size())
+    {
+        m_cmdLine.push_back(value);
+    }
+    else if (rank < m_cmdLine.size())
+    {
+        m_cmdLine[rank] = value;
+    }
+    else
+    {
+        // should never happen
+        assert(false);
+    }
 }
 
-float COldObject::GetCmdLine(int rank)
+float COldObject::GetCmdLine(unsigned int rank)
 {
-    if ( rank < 0 || rank >= OBJECTMAXCMDLINE )  return 0.0f;
+    if ( rank >= m_cmdLine.size() )  return 0.0f;
     return m_cmdLine[rank];
 }
 
diff --git a/src/object/old_object.h b/src/object/old_object.h
index 19d6639b..1b0b29f0 100644
--- a/src/object/old_object.h
+++ b/src/object/old_object.h
@@ -30,7 +30,6 @@
 // The father of all parts must always be the part number zero!
 const int OBJECTMAXPART         = 40;
 const int OBJECTMAXDESELLIST    = 10;
-const int OBJECTMAXCMDLINE      = 20;
 
 struct ObjectPart
 {
@@ -168,8 +167,8 @@ public:
     CObject*    GetTransporter() override;
     void        SetTransporterPart(int part) override;
 
-    bool        SetCmdLine(int rank, float value) override;
-    float       GetCmdLine(int rank) override;
+    void        SetCmdLine(unsigned int rank, float value) override;
+    float       GetCmdLine(unsigned int rank) override;
 
     Math::Matrix*   GetRotateMatrix(int part) override;
     Math::Matrix*   GetWorldMatrix(int part) override;
@@ -407,5 +406,5 @@ protected:
 
     float m_infoReturn;
 
-    float       m_cmdLine[OBJECTMAXCMDLINE];
+    std::vector<float> m_cmdLine;
 };
diff --git a/src/object/old_object_interface.h b/src/object/old_object_interface.h
index add90d9d..9cec3418 100644
--- a/src/object/old_object_interface.h
+++ b/src/object/old_object_interface.h
@@ -165,8 +165,8 @@ public:
     virtual CObject*    GetTransporter() = 0;
     virtual void        SetTransporterPart(int part) = 0;
 
-    virtual bool        SetCmdLine(int rank, float value) = 0;
-    virtual float       GetCmdLine(int rank) = 0;
+    virtual void        SetCmdLine(unsigned int rank, float value) = 0;
+    virtual float       GetCmdLine(unsigned int rank) = 0;
 
     virtual Math::Matrix*   GetRotateMatrix(int part) = 0;
     virtual Math::Matrix*   GetWorldMatrix(int part) = 0;
diff --git a/src/object/robotmain.cpp b/src/object/robotmain.cpp
index 4f47cfd8..01971670 100644
--- a/src/object/robotmain.cpp
+++ b/src/object/robotmain.cpp
@@ -3558,7 +3558,7 @@ void CRobotMain::CreateScene(bool soluce, bool fixScene, bool resetObject)
                     if (line->GetParam("cmdline")->IsDefined())
                     {
                         const auto& cmdline = line->GetParam("cmdline")->AsArray();
-                        for (unsigned int i = 0; i < OBJECTMAXCMDLINE && i < cmdline.size(); i++) //TODO: get rid of the limit
+                        for (unsigned int i = 0; i < cmdline.size(); i++)
                         {
                             obj->SetCmdLine(i, cmdline[i]->AsFloat());
                         }