mirror of
https://gitea.wildfiregames.com/0ad/0ad
synced 2026-06-17 05:44:08 -07:00
bae258f9a1 implemented CCmpUnitMotionManager, which handles motion for
CCmpUnitMotion (as distinct from ICmpUnitMotion, the interface).
The tight coupling between these two components was awkward at the
interface level, leaking underlying implementation details.
This diff makes CmpUnitMotionManager explicitly manage CmpUnitMotion,
instead of any implementation of ICmpUnitMotion, and moves files around
as requir
ed.
This:
- Makes it impossible to accidentally try to have the wrong
IID_UnitMotion managed by the UnitMotionManager.
- Allows devirtualising the calls from the manager to UnitMotion itself
(and inlining, as they are compiled in the same TU).
- Cleans up the manager interface - MotionState is now part of
CCmpUnitMotionManager.
- Cleans up ICmpUnitMotion interface - no need to provide a private
interface for the manager.
Differential Revision: https://code.wildfiregames.com/D3732
This was SVN commit r25125.
51 lines
1.6 KiB
C++
51 lines
1.6 KiB
C++
/* Copyright (C) 2021 Wildfire Games.
|
|
* This file is part of 0 A.D.
|
|
*
|
|
* 0 A.D. is free software: you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License as published by
|
|
* the Free Software Foundation, either version 2 of the License, or
|
|
* (at your option) any later version.
|
|
*
|
|
* 0 A.D. is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with 0 A.D. If not, see <http://www.gnu.org/licenses/>.
|
|
*/
|
|
|
|
#ifndef INCLUDED_ICMPUNITMOTIONMANAGER
|
|
#define INCLUDED_ICMPUNITMOTIONMANAGER
|
|
|
|
#include "simulation2/system/Interface.h"
|
|
|
|
class CCmpUnitMotion;
|
|
|
|
/**
|
|
* UnitMotionManager - handles motion for CCmpUnitMotion.
|
|
* This allows units to push past each other instead of requiring pathfinder computations,
|
|
* making movement much smoother overall.
|
|
*/
|
|
class ICmpUnitMotionManager : public IComponent
|
|
{
|
|
public:
|
|
DECLARE_INTERFACE_TYPE(UnitMotionManager)
|
|
|
|
private:
|
|
/**
|
|
* This class makes no sense outside of CCmpUnitMotion. This enforces that tight coupling.
|
|
*/
|
|
friend class CCmpUnitMotion;
|
|
|
|
virtual void Register(CCmpUnitMotion* component, entity_id_t ent, bool formationController) = 0;
|
|
virtual void Unregister(entity_id_t ent) = 0;
|
|
|
|
/**
|
|
* True if entities are currently in the "Move" phase.
|
|
*/
|
|
virtual bool ComputingMotion() const = 0;
|
|
|
|
};
|
|
|
|
#endif // INCLUDED_ICMPUNITMOTIONMANAGER
|