Replace DEFINE_* macro with inplace type alias (Part 2)

Thank to Im-Dex.
From commit:
Im-dex/xray-162@9107297

Author: Xottab-DUTY

src/xrAICore/Navigation/game_graph_space.h

@@ -148,5 +148,5 @@ struct STerrainPlace
     svector<_LOCATION_ID, LOCATION_TYPE_COUNT> tMask;
 };
 
-DEFINE_VECTOR(STerrainPlace, TERRAIN_VECTOR, TERRAIN_IT);
+using TERRAIN_VECTOR = xr_vector<STerrainPlace>;
 }

src/xrEngine/Stats.cpp

@@ -217,8 +217,8 @@ void CStats::OnRender()
     {
         CSound_stats_ext snd_stat_ext;
         ::Sound->statistic(0, &snd_stat_ext);
-        CSound_stats_ext::item_vec_it _I = snd_stat_ext.items.begin();
-        CSound_stats_ext::item_vec_it _E = snd_stat_ext.items.end();
+        auto _I = snd_stat_ext.items.begin();
+        auto _E = snd_stat_ext.items.end();
         for (; _I != _E; _I++)
         {
             const CSound_stats_ext::SItem& item = *_I;

src/xrEngine/editor_environment_manager.cpp

@@ -111,8 +111,8 @@ void manager::load_weathers()
     m_weathers->load();
 
     // sorting weather envs
-    EnvsMapIt _I = WeatherCycles.begin();
-    EnvsMapIt _E = WeatherCycles.end();
+    auto _I = WeatherCycles.begin();
+    auto _E = WeatherCycles.end();
     for (; _I != _E; _I++)
     {
         R_ASSERT3(_I->second.size() > 1, "Environment in weather must >=2", *_I->first);

src/xrEngine/xr_collide_form.cpp

@@ -47,7 +47,7 @@ void CCF_Skeleton::SElement::center(Fvector& center) const
 bool pred_find_elem(const CCF_Skeleton::SElement& E, u16 elem) { return E.elem_id < elem; }
 bool CCF_Skeleton::_ElementCenter(u16 elem_id, Fvector& e_center)
 {
-    ElementVecIt it = std::lower_bound(elements.begin(), elements.end(), elem_id, pred_find_elem);
+    auto it = std::lower_bound(elements.begin(), elements.end(), elem_id, pred_find_elem);
     if (it->elem_id == elem_id)
     {
         it->center(e_center);
@@ -132,7 +132,7 @@ void CCF_Skeleton::BuildState()
         }
     }
 
-    for (ElementVecIt I = elements.begin(); I != elements.end(); I++)
+    for (auto I = elements.begin(); I != elements.end(); I++)
     {
         if (!I->valid())
             continue;
@@ -241,7 +241,7 @@ BOOL CCF_Skeleton::_RayQuery(const collide::ray_defs& Q, collide::rq_results& R)
     }
 
     BOOL bHIT = FALSE;
-    for (ElementVecIt I = elements.begin(); I != elements.end(); I++)
+    for (auto I = elements.begin(); I != elements.end(); I++)
     {
         if (!I->valid())
             continue;

src/xrEngine/xr_collide_form.h

@@ -90,7 +90,6 @@ class ENGINE_API ICollisionForm
     IGameObject* owner; // владелец
     u32 dwQueryID;
 
-protected:
     Fbox bv_box; // (Local) BBox объекта
     Fsphere bv_sphere; // (Local) Sphere
 private:
@@ -134,13 +133,12 @@ class ENGINE_API CCF_Skeleton : public ICollisionForm
         u16 type;
         u16 elem_id;
 
-    public:
         SElement() : elem_id(u16(-1)), type(0) {}
         SElement(u16 id, u16 t) : elem_id(id), type(t) {}
         BOOL valid() const { return (elem_id != (u16(-1))) && (type != 0); }
         void center(Fvector& center) const;
     };
-    DEFINE_VECTOR(SElement, ElementVec, ElementVecIt);
+    using ElementVec = xr_vector<SElement>;
 
 private:
     u64 vis_mask;
@@ -169,7 +167,6 @@ class ENGINE_API CCF_Skeleton : public ICollisionForm
 
 class ENGINE_API CCF_EventBox : public ICollisionForm
 {
-private:
     Fplane Planes[6];
 
 public:
@@ -200,7 +197,6 @@ class ENGINE_API CCF_Shape : public ICollisionForm
     };
     xr_vector<shape_def> shapes;
 
-public:
     CCF_Shape(IGameObject* _owner);
 
     virtual BOOL _RayQuery(const collide::ray_defs& Q, collide::rq_results& R);

src/xrEngine/xr_efflensflare.cpp

@@ -120,12 +120,12 @@ void CLensFlareDescriptor::OnDeviceCreate()
     // shaders
     m_Gradient.m_pRender->CreateShader(*m_Gradient.shader, *m_Gradient.texture);
     m_Source.m_pRender->CreateShader(*m_Source.shader, *m_Source.texture);
-    for (FlareIt it = m_Flares.begin(); it != m_Flares.end(); it++)
+    for (auto it = m_Flares.begin(); it != m_Flares.end(); it++)
         it->m_pRender->CreateShader(*it->shader, *it->texture);
     /*
     m_Gradient.hShader = CreateShader (*m_Gradient.texture,*m_Gradient.shader);
     m_Source.hShader = CreateShader (*m_Source.texture,*m_Source.shader);
-    for (FlareIt it=m_Flares.begin(); it!=m_Flares.end(); it++) it->hShader = CreateShader(*it->texture,*it->shader);
+    for (auto it=m_Flares.begin(); it!=m_Flares.end(); it++) it->hShader = CreateShader(*it->texture,*it->shader);
     */
 }
 
@@ -134,12 +134,12 @@ void CLensFlareDescriptor::OnDeviceDestroy()
     // shaders
     m_Gradient.m_pRender->DestroyShader();
     m_Source.m_pRender->DestroyShader();
-    for (FlareIt it = m_Flares.begin(); it != m_Flares.end(); it++)
+    for (auto it = m_Flares.begin(); it != m_Flares.end(); it++)
         it->m_pRender->DestroyShader();
     /*
     m_Gradient.hShader.destroy ();
     m_Source.hShader.destroy ();
-    for (FlareIt it=m_Flares.begin(); it!=m_Flares.end(); it++) it->hShader.destroy();
+    for (auto it=m_Flares.begin(); it!=m_Flares.end(); it++) it->hShader.destroy();
     */
 }
 
@@ -552,7 +552,7 @@ void CLensFlare::Render(BOOL bSun, BOOL bFlares, BOOL bGradient)
     vecDx.normalize (vecAxis);
     vecDy.crossproduct (vecDx, vecDir);
     if (m_Current->m_Flags.is(CLensFlareDescriptor::flFlare)){
-    for (CLensFlareDescriptor::FlareIt it=m_Current->m_Flares.begin(); it!=m_Current->m_Flares.end(); it++){
+    for (CLensFlareDescriptor::auto it=m_Current->m_Flares.begin(); it!=m_Current->m_Flares.end(); it++){
     CLensFlareDescriptor::SFlare& F = *it;
     vec.mul (vecAxis, F.fPosition);
     vec.add (vecCenter);
@@ -608,7 +608,7 @@ shared_str CLensFlare::AppendDef(CEnvironment& environment, CInifile* pIni, LPCS
 {
     if (!sect || (0 == sect[0]))
         return "";
-    for (LensFlareDescIt it = m_Palette.begin(); it != m_Palette.end(); it++)
+    for (auto it = m_Palette.begin(); it != m_Palette.end(); it++)
         if (0 == xr_strcmp(*(*it)->section, sect))
             return sect;
 
@@ -623,14 +623,14 @@ void CLensFlare::OnDeviceCreate()
     m_pRender->OnDeviceCreate();
 
     // palette
-    for (LensFlareDescIt it = m_Palette.begin(); it != m_Palette.end(); it++)
+    for (auto it = m_Palette.begin(); it != m_Palette.end(); it++)
         (*it)->OnDeviceCreate();
 }
 
 void CLensFlare::OnDeviceDestroy()
 {
     // palette
-    for (LensFlareDescIt it = m_Palette.begin(); it != m_Palette.end(); it++)
+    for (auto it = m_Palette.begin(); it != m_Palette.end(); it++)
         (*it)->OnDeviceDestroy();
 
     // VS

src/xrEngine/xr_efflensflare.h

@@ -23,11 +23,13 @@ class ENGINE_API CLensFlareDescriptor
         // ref_shader hShader;
         SFlare() { fOpacity = fRadius = fPosition = 0; }
     };
+
     struct SSource : public SFlare
     {
         BOOL ignore_color;
     };
-    DEFINE_VECTOR(SFlare, FlareVec, FlareIt);
+    using FlareVec = xr_vector<SFlare>;
+
     FlareVec m_Flares;
 
     enum

src/xrParticles/particle_actions.h

@@ -21,7 +21,9 @@ struct PARTICLES_API ParticleAction
     virtual void Load(IReader& F) = 0;
     virtual void Save(IWriter& F) = 0;
 };
-DEFINE_VECTOR(ParticleAction*, PAVec, PAVecIt);
+using PAVec = xr_vector<ParticleAction*>;
+using PAVecIt = PAVec::iterator;
+
 class ParticleActions
 {
     PAVec actions;
@@ -34,23 +36,27 @@ class ParticleActions
         m_bLocked = false;
     }
     ~ParticleActions() { clear(); }
-    IC void clear()
+
+    void clear()
     {
         R_ASSERT(!m_bLocked);
         for (PAVecIt it = actions.begin(); it != actions.end(); it++)
             xr_delete(*it);
         actions.clear();
     }
-    IC void append(ParticleAction* pa)
+
+    void append(ParticleAction* pa)
     {
         R_ASSERT(!m_bLocked);
         actions.push_back(pa);
     }
-    IC bool empty() { return actions.empty(); }
-    IC PAVecIt begin() { return actions.begin(); }
-    IC PAVecIt end() { return actions.end(); }
-    IC int size() { return actions.size(); }
-    IC void resize(int cnt)
+
+    bool empty() { return actions.empty(); }
+    PAVecIt begin() { return actions.begin(); }
+    PAVecIt end() { return actions.end(); }
+    int size() { return actions.size(); }
+
+    void resize(int cnt)
     {
         R_ASSERT(!m_bLocked);
         actions.resize(cnt);

src/xrParticles/particle_manager.h

@@ -10,8 +10,8 @@ class CParticleManager : public IParticleManager
 {
     // These are static because all threads access the same effects.
     // All accesses to these should be locked.
-    DEFINE_VECTOR(ParticleEffect*, ParticleEffectVec, ParticleEffectVecIt);
-    DEFINE_VECTOR(ParticleActions*, ParticleActionsVec, ParticleActionsVecIt);
+    using ParticleEffectVec = xr_vector<ParticleEffect*>;
+    using ParticleActionsVec = xr_vector<ParticleActions*>;
     ParticleEffectVec effect_vec;
     ParticleActionsVec m_alist_vec;
 

src/xrPhysics/PHActorCharacter.h

@@ -74,7 +74,9 @@ struct TPHCharacterRestrictor : public SPHCharacterRestrictor
         }
     }
 };
-DEFINE_VECTOR(SPHCharacterRestrictor*, RESRICTORS_V, RESTRICTOR_I);
+using RESRICTORS_V = xr_vector<SPHCharacterRestrictor*>;
+using RESTRICTOR_I = RESRICTORS_V::iterator;
+
 // typedef SPHCharacterRestrictor*		RESRICTORS_V[2];
 // typedef SPHCharacterRestrictor**	RESTRICTOR_I;
 IC RESTRICTOR_I begin(RESRICTORS_V& v)