CBasicModel: Return CAABox by const reference

Allows avoiding unnecessary 24 byte copies, plus unnecessary function
calls for extremely trivial members.
This commit is contained in:
Lioncache
2025-12-14 01:26:27 -05:00
parent 71303fffd3
commit 2825960079
2 changed files with 14 additions and 33 deletions

View File

@@ -14,37 +14,17 @@ CBasicModel::~CBasicModel()
delete surface;
}
size_t CBasicModel::GetVertexCount() const
const CAABox& CBasicModel::GetSurfaceAABox(size_t Surface) const
{
return mVertexCount;
}
size_t CBasicModel::GetTriangleCount() const
{
return mTriangleCount;
}
CAABox CBasicModel::AABox() const
{
return mAABox;
}
bool CBasicModel::IsBuffered() const
{
return mBuffered;
}
size_t CBasicModel::GetSurfaceCount() const
{
return mSurfaces.size();
}
CAABox CBasicModel::GetSurfaceAABox(size_t Surface) const
{
return mSurfaces[Surface]->AABox;
return GetSurface(Surface)->AABox;
}
SSurface* CBasicModel::GetSurface(size_t Surface)
{
return mSurfaces[Surface];
}
const SSurface* CBasicModel::GetSurface(size_t Surface) const
{
return mSurfaces[Surface];
}

View File

@@ -24,13 +24,14 @@ public:
explicit CBasicModel(CResourceEntry *pEntry = nullptr);
~CBasicModel() override;
size_t GetVertexCount() const;
size_t GetTriangleCount() const;
CAABox AABox() const;
bool IsBuffered() const;
size_t GetSurfaceCount() const;
CAABox GetSurfaceAABox(size_t Surface) const;
size_t GetVertexCount() const { return mVertexCount; }
size_t GetTriangleCount() const { return mTriangleCount; }
const CAABox& AABox() const { return mAABox; }
bool IsBuffered() const { return mBuffered; }
size_t GetSurfaceCount() const { return mSurfaces.size(); }
const CAABox& GetSurfaceAABox(size_t Surface) const;
SSurface* GetSurface(size_t Surface);
const SSurface* GetSurface(size_t Surface) const;
virtual void ClearGLBuffer() = 0;
};