AxioDL
/
metaforce
mirror of https://github.com/AxioDL/metaforce.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

CEntityInfo: std::move vector in constructor 

Allows calling code to potentially avoid copies altogether by moving
into the constructor.
This commit is contained in:
Lioncash 2019-09-15 16:04:13 -04:00
parent cc236dcaba
commit 0281029015
2 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
Runtime/CStateManager.cpp
View File

@ -1292,21 +1292,21 @@ void CStateManager::LoadScriptObjects(TAreaId aid, CInputStream& in, std::vector
std::pair<TEditorId, TUniqueId> CStateManager::LoadScriptObject(TAreaId aid, EScriptObjectType type, u32 length, std::pair<TEditorId, TUniqueId> CStateManager::LoadScriptObject(TAreaId aid, EScriptObjectType type, u32 length,
CInputStream& in) { CInputStream& in) {
TEditorId id = in.readUint32Big(); const TEditorId id = in.readUint32Big();
u32 connCount = in.readUint32Big(); const u32 connCount = in.readUint32Big();
length -= 8; length -= 8;
std::vector<SConnection> conns; std::vector<SConnection> conns;
conns.reserve(connCount); conns.reserve(connCount);
for (int i = 0; i < connCount; ++i) { for (u32 i = 0; i < connCount; ++i) {
EScriptObjectState state = EScriptObjectState(in.readUint32Big()); const auto state = EScriptObjectState(in.readUint32Big());
EScriptObjectMessage msg = EScriptObjectMessage(in.readUint32Big()); const auto msg = EScriptObjectMessage(in.readUint32Big());
TEditorId target = in.readUint32Big(); const TEditorId target = in.readUint32Big();
length -= 12; length -= 12;
conns.push_back(SConnection{state, msg, target}); conns.push_back(SConnection{state, msg, target});
} }
u32 propCount = in.readUint32Big(); const u32 propCount = in.readUint32Big();
length -= 4; length -= 4;
auto startPos = in.position(); const auto startPos = in.position();
bool error = false; bool error = false;
FScriptLoader loader = {}; FScriptLoader loader = {};
@ -1315,7 +1315,7 @@ std::pair<TEditorId, TUniqueId> CStateManager::LoadScriptObject(TAreaId aid, ESc
CEntity* ent = nullptr; CEntity* ent = nullptr;
if (loader) { if (loader) {
CEntityInfo info(aid, conns, id); const CEntityInfo info(aid, std::move(conns), id);
ent = loader(*this, in, propCount, info); ent = loader(*this, in, propCount, info);
} else { } else {
error = true; error = true;

4
Runtime/World/CEntityInfo.hpp
View File

@ -18,8 +18,8 @@ class CEntityInfo {
TEditorId x14_editorId; TEditorId x14_editorId;
public: public:
CEntityInfo(TAreaId aid, const std::vector<SConnection>& conns, TEditorId eid = kInvalidEditorId) CEntityInfo(TAreaId aid, std::vector<SConnection> conns, TEditorId eid = kInvalidEditorId)
: x0_areaId(aid), x4_conns(conns), x14_editorId(eid) {} : x0_areaId(aid), x4_conns(std::move(conns)), x14_editorId(eid) {}
TAreaId GetAreaId() const { return x0_areaId; } TAreaId GetAreaId() const { return x0_areaId; }
std::vector<SConnection> GetConnectionList() const { return x4_conns; } std::vector<SConnection> GetConnectionList() const { return x4_conns; }
TEditorId GetEditorId() const { return x14_editorId; } TEditorId GetEditorId() const { return x14_editorId; }