minor refactoring and compilation fixes

DynObj.cpp

@@ -1622,8 +1622,7 @@ TDynamicObject::Init(std::string Name, // nazwa pojazdu, np. "EU07-424"
                      double fDist, // dystans względem punktu 1
                      std::string DriverType, // typ obsady
                      double fVel, // prędkość początkowa
-                     std::string TrainName, // nazwa składu, np. "PE2307" albo Vmax, jeśli pliku
-                     // nie ma a są cyfry
+                     std::string TrainName, // nazwa składu, np. "PE2307" albo Vmax, jeśli pliku nie ma a są cyfry
                      float Load, // ilość ładunku
                      std::string LoadType, // nazwa ładunku
                      bool Reversed, // true, jeśli ma stać odwrotnie w składzie
@@ -1639,18 +1638,18 @@ TDynamicObject::Init(std::string Name, // nazwa pojazdu, np. "EU07-424"
         DriverType = "1"; // sterujący kabiną +1
     else if (DriverType == "reardriver")
         DriverType = "2"; // sterujący kabiną -1
-    // else if (DriverType=="connected") DriverType="c"; //tego trzeba się pozbyć
-    // na rzecz
-    // ukrotnienia
     else if (DriverType == "passenger")
         DriverType = "p"; // to do przemyślenia
     else if (DriverType == "nobody")
         DriverType = ""; // nikt nie siedzi
+
     int Cab = 0; // numer kabiny z obsadą (nie można zająć obu)
     if (DriverType == "1") // od przodu składu
         Cab = 1; // iDirection?1:-1; //iDirection=1 gdy normalnie, =0 odwrotnie
     else if (DriverType == "2") // od tyłu składu
         Cab = -1; // iDirection?-1:1;
+/*
+    // NOTE: leave passenger in the middle section, this is most likely to be 'passenger' section in MU trains 
     else if (DriverType == "p")
     {
         if (Random(6) < 3)
@@ -1658,20 +1657,10 @@ TDynamicObject::Init(std::string Name, // nazwa pojazdu, np. "EU07-424"
         else
             Cab = -1; // losowy przydział kabiny
     }
-    /* to nie ma uzasadnienia
-     else
-     {//obsada nie rozpoznana
-      Cab=0;  //McZapkie-010303: w przyszlosci dac tez pomocnika, palacza,
-     konduktora itp.
-      Error("Unknown DriverType description: "+DriverType);
-      DriverType="nobody";
-     }
-    */
+*/
     // utworzenie parametrów fizyki
-    MoverParameters =
-        new TMoverParameters(iDirection ? fVel : -fVel, Type_Name, asName, Load, LoadType, Cab);
+    MoverParameters = new TMoverParameters(iDirection ? fVel : -fVel, Type_Name, asName, Load, LoadType, Cab);
     iLights = MoverParameters->iLights; // wskaźnik na stan własnych świateł
-    // (zmienimy dla rozrządczych EZT)
     // McZapkie: TypeName musi byc nazwą CHK/MMD pojazdu
     if (!MoverParameters->LoadFIZ(asBaseDir))
     { // jak wczytanie CHK się nie uda, to błąd
@@ -1865,43 +1854,9 @@ TDynamicObject::Init(std::string Name, // nazwa pojazdu, np. "EU07-424"
                 // wygenerować
                 fDist = -fDist; // to traktujemy, jakby przesunięcie było w drugą stronę
     }
-    // w wagonie tez niech jedzie
-    // if (MoverParameters->MainCtrlPosNo>0 &&
-    // if (MoverParameters->CabNo!=0)
-    if (DriverType != "")
-    { // McZapkie-040602: jeśli coś siedzi w pojeździe
-        if (Name == Global::asHumanCtrlVehicle) // jeśli pojazd wybrany do prowadzenia
-        {
-            if ( MoverParameters->EngineType != Dumb)
-                Controller = Humandriver; // wsadzamy tam sterującego
-            else // w przeciwnym razie trzeba włączyć pokazywanie kabiny
-                bDisplayCab = true;
-        }
-        // McZapkie-151102: rozkład jazdy czytany z pliku *.txt z katalogu w którym
-        // jest sceneria
-        if (DriverType == "1" || DriverType == "2")
-        { // McZapkie-110303: mechanik i rozklad tylko gdy jest obsada
-            // MoverParameters->ActiveCab=MoverParameters->CabNo; //ustalenie aktywnej
-            // kabiny
-            // (rozrząd)
-            Mechanik = new TController(Controller, this, Aggressive);
-            if (TrainName.empty()) // jeśli nie w składzie
-            {
-                Mechanik->DirectionInitial(); // załączenie rozrządu (wirtualne kabiny) itd.
-                Mechanik->PutCommand(
-                    "Timetable:", iDirection ? -fVel : fVel, 0,
-                    NULL); // tryb pociągowy z ustaloną prędkością (względem sprzęgów)
-            }
-            // if (TrainName!="none")
-            // Mechanik->PutCommand("Timetable:"+TrainName,fVel,0,NULL);
-        }
-        else if (DriverType == "p")
-        { // obserwator w charakterze pasażera
-            // Ra: to jest niebezpieczne, bo w razie co będzie pomagał hamulcem
-            // bezpieczeństwa
-            Mechanik = new TController(Controller, this, Easyman, false);
-        }
-    }
+
+    create_controller( DriverType, !TrainName.empty() );
+
     // McZapkie-250202
     iAxles = (MaxAxles < MoverParameters->NAxles) ? MaxAxles : MoverParameters->NAxles; // ilość osi
     // wczytywanie z pliku nazwatypu.mmd, w tym model
@@ -2073,9 +2028,6 @@ TDynamicObject::Init(std::string Name, // nazwa pojazdu, np. "EU07-424"
     loc.Y = vPosition.z;
     loc.Z = vPosition.y;
     MoverParameters->Loc = loc; // normalnie przesuwa ComputeMovement() w Update()
-    // pOldPos4=Axle1.pPosition; //Ra: nie używane
-    // pOldPos1=Axle0.pPosition;
-    // ActualTrack= GetTrack(); //McZapkie-030303
     // ABuWozki 060504
     if (mdModel) // jeśli ma w czym szukać
     {
@@ -2090,17 +2042,54 @@ TDynamicObject::Init(std::string Name, // nazwa pojazdu, np. "EU07-424"
         if (smBogie[1])
             smBogie[1]->WillBeAnimated();
     }
-    // ABu: zainicjowanie zmiennej, zeby nic sie nie ruszylo
-    // w pierwszej klatce, potem juz liczona prawidlowa wartosc masy
+    // ABu: zainicjowanie zmiennej, zeby nic sie nie ruszylo w pierwszej klatce,
+    // potem juz liczona prawidlowa wartosc masy
     MoverParameters->ComputeConstans();
-    /*Ra: to nie działa - Event0 musi być wykonywany ciągle
-    if (fVel==0.0) //jeśli stoi
-     if (MoverParameters->CabNo!=0) //i ma kogoś w kabinie
-      if (Track->Event0) //a jest w tym torze event od stania
-       RaAxleEvent(Track->Event0); //dodanie eventu stania do kolejki
-    */
-    vFloor = vector3(0, 0, MoverParameters->Floor); // wektor podłogi dla wagonów, przesuwa ładunek
-    return MoverParameters->Dim.L; // długość większa od zera oznacza OK; 2mm docisku?
+    // wektor podłogi dla wagonów, przesuwa ładunek
+    vFloor = vector3(0, 0, MoverParameters->Floor);
+    // długość większa od zera oznacza OK; 2mm docisku?
+    return MoverParameters->Dim.L;
+}
+
+void
+TDynamicObject::create_controller( std::string const Type, bool const Trainset ) {
+
+    if( Type == "" ) { return; }
+
+    if( asName == Global::asHumanCtrlVehicle ) {
+        // jeśli pojazd wybrany do prowadzenia
+        if( MoverParameters->EngineType != Dumb ) {
+            // wsadzamy tam sterującego
+            Controller = Humandriver;
+        }
+        else {
+            // w przeciwnym razie trzeba włączyć pokazywanie kabiny
+            bDisplayCab = true;
+        }
+    }
+    // McZapkie-151102: rozkład jazdy czytany z pliku *.txt z katalogu w którym jest sceneria
+    if( ( Type == "1" )
+     || ( Type == "2" ) ) {
+        // McZapkie-110303: mechanik i rozklad tylko gdy jest obsada
+        Mechanik = new TController( Controller, this, Aggressive );
+
+        if( false == Trainset ) {
+            // jeśli nie w składzie
+            // załączenie rozrządu (wirtualne kabiny) itd.
+            Mechanik->DirectionInitial();
+            // tryb pociągowy z ustaloną prędkością (względem sprzęgów)
+            Mechanik->PutCommand(
+                "Timetable:",
+                MoverParameters->V * 3.6 * ( iDirection ? -1.0 : 1.0 ),
+                0,
+                nullptr );
+        }
+    }
+    else if( Type == "p" ) {
+        // obserwator w charakterze pasażera
+        // Ra: to jest niebezpieczne, bo w razie co będzie pomagał hamulcem bezpieczeństwa
+        Mechanik = new TController( Controller, this, Easyman, false );
+    }
 }
 
 void TDynamicObject::FastMove(double fDistance)

DynObj.h

@@ -402,17 +402,18 @@ private:
     int iCabs; // maski bitowe modeli kabin
     TTrack *MyTrack; // McZapkie-030303: tor na ktorym stoi, ABu
     std::string asBaseDir;
-    int iOverheadMask; // maska przydzielana przez AI pojazdom posiadającym pantograf, aby wymuszały
-    // jazdę bezprądową
+    int iOverheadMask; // maska przydzielana przez AI pojazdom posiadającym pantograf, aby wymuszały jazdę bezprądową
     TTractionParam tmpTraction;
-    double fAdjustment; // korekcja - docelowo przenieść do TrkFoll.cpp wraz z odległością od
-    // poprzedniego
+    double fAdjustment; // korekcja - docelowo przenieść do TrkFoll.cpp wraz z odległością od poprzedniego
+
     TDynamicObject();
     ~TDynamicObject();
-    double TDynamicObject::Init( // zwraca długość pojazdu albo 0, jeśli błąd
+    // zwraca długość pojazdu albo 0, jeśli błąd
+    double TDynamicObject::Init(
         std::string Name, std::string BaseDir, std::string asReplacableSkin, std::string Type_Name,
         TTrack *Track, double fDist, std::string DriverType, double fVel, std::string TrainName,
         float Load, std::string LoadType, bool Reversed, std::string);
+    void create_controller( std::string const Type, bool const Trainset );
     void AttachPrev(TDynamicObject *Object, int iType = 1);
     bool UpdateForce(double dt, double dt1, bool FullVer);
     void LoadUpdate();

scenenode.cpp

@@ -405,7 +405,7 @@ shape_node::merge( shape_node &Shape ) {
     m_data.area.center =
         interpolate(
             m_data.area.center, Shape.m_data.area.center,
-            static_cast<float>( Shape.m_data.vertices.size() ) / ( Shape.m_data.vertices.size() + m_data.vertices.size() ) );
+            static_cast<double>( Shape.m_data.vertices.size() ) / ( Shape.m_data.vertices.size() + m_data.vertices.size() ) );
     m_data.vertices.insert(
         std::end( m_data.vertices ),
         std::begin( Shape.m_data.vertices ), std::end( Shape.m_data.vertices ) );
@@ -625,7 +625,7 @@ lines_node::merge( lines_node &Lines ) {
     m_data.area.center =
         interpolate(
             m_data.area.center, Lines.m_data.area.center,
-            static_cast<float>( Lines.m_data.vertices.size() ) / ( Lines.m_data.vertices.size() + m_data.vertices.size() ) );
+            static_cast<double>( Lines.m_data.vertices.size() ) / ( Lines.m_data.vertices.size() + m_data.vertices.size() ) );
     m_data.vertices.insert(
         std::end( m_data.vertices ),
         std::begin( Lines.m_data.vertices ), std::end( Lines.m_data.vertices ) );

simulation.cpp

@@ -806,7 +806,7 @@ state_manager::deserialize_dynamic( cParser &Input, scene::scratch_data &Scratch
         Scratchpad.trainset.offset -= length;
         // automatically establish permanent connections for couplers which specify them in their definitions
         if( ( coupling != 0 )
-         && ( vehicle->MoverParameters->Couplers[ ( offset == -1.0 ? 0 : 1 ) ].AllowedFlag & coupling::permanent ) ) {
+         && ( vehicle->MoverParameters->Couplers[ ( offset == -1.0 ? side::front : side::rear ) ].AllowedFlag & coupling::permanent ) ) {
             coupling |= coupling::permanent;
         }
         if( true == Scratchpad.trainset.is_open ) {