mirror/OpenMW
1
0
Fork 0
mirror of https://gitlab.com/OpenMW/openmw.git synced 2025-02-22 12:39:59 +00:00
Code Issues Projects Releases Wiki Activity

Simplify iteration over Bone::mChildren

Browse Source
This commit is contained in:
elsid 2022-07-17 13:52:24 +02:00
parent e4f599575e
commit 0e12d331f9
No known key found for this signature in database
GPG Key ID: 4DE04C198CBA7625
1 changed files with 12 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
components/sceneutil/skeleton.cpp
View File

@ -5,6 +5,8 @@
#include <components/debug/debuglog.hpp>
#include <components/misc/stringops.hpp>
#include <algorithm>
namespace SceneUtil
{
@ -75,24 +77,16 @@ Bone* Skeleton::getBone(const std::string &name)
Bone* bone = mRootBone.get();
for (osg::MatrixTransform* matrixTransform : found->second)
{
Bone* child = nullptr;
for (unsigned int i=0; i<bone->mChildren.size(); ++i)
{
if (bone->mChildren[i]->mNode == matrixTransform)
{
child = bone->mChildren[i].get();
break;
}
}
const auto it = std::find_if(bone->mChildren.begin(), bone->mChildren.end(),
[&] (const auto& v) { return v->mNode == matrixTransform; });
if (child == nullptr)
if (it == bone->mChildren.end())
{
auto childBone = std::make_unique<Bone>();
child = childBone.get();
bone->mChildren.push_back(std::move(childBone));
bone = bone->mChildren.emplace_back(std::make_unique<Bone>()).get();
mNeedToUpdateBoneMatrices = true;
}
bone = child;
else
bone = it->get();
bone->mNode = matrixTransform;
}
@ -111,8 +105,8 @@ void Skeleton::updateBoneMatrices(unsigned int traversalNumber)
{
if (mRootBone.get())
{
for (unsigned int i=0; i<mRootBone->mChildren.size(); ++i)
mRootBone->mChildren[i]->update(nullptr);
for (const auto& child : mRootBone->mChildren)
child->update(nullptr);
}
mNeedToUpdateBoneMatrices = false;
@ -178,10 +172,8 @@ void Bone::update(const osg::Matrixf* parentMatrixInSkeletonSpace)
else
mMatrixInSkeletonSpace = mNode->getMatrix();
for (unsigned int i=0; i<mChildren.size(); ++i)
{
mChildren[i]->update(&mMatrixInSkeletonSpace);
}
for (const auto& child : mChildren)
child->update(&mMatrixInSkeletonSpace);
}
}