sid
/
NazaraEngine
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #206 from alexandre-janniaux/warnings/misc/fix-shadow-variable/1 

Fix some shadowing variable warnings
This commit is contained in:
Jérôme Leclercq 2019-07-08 12:54:10 +02:00 committed by GitHub
parent 7da8945c72 e29f2f7a92
commit c2dd032ed2
3 changed files with 43 additions and 43 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
SDK/src/NDK/BaseWidget.cpp
View File

@ -197,9 +197,9 @@ namespace Ndk
newEntity->Enable(m_visible); newEntity->Enable(m_visible);
m_entities.emplace_back(); m_entities.emplace_back();
WidgetEntity& widgetEntity = m_entities.back(); WidgetEntity& newWidgetEntity = m_entities.back();
widgetEntity.handle = newEntity; newWidgetEntity.handle = newEntity;
widgetEntity.onDisabledSlot.Connect(newEntity->OnEntityDisabled, [this](Entity* entity) newWidgetEntity.onDisabledSlot.Connect(newEntity->OnEntityDisabled, [this](Entity* entity)
{ {
auto it = std::find_if(m_entities.begin(), m_entities.end(), [&](const WidgetEntity& widgetEntity) { return widgetEntity.handle == entity; }); auto it = std::find_if(m_entities.begin(), m_entities.end(), [&](const WidgetEntity& widgetEntity) { return widgetEntity.handle == entity; });
NazaraAssert(it != m_entities.end(), "Entity does not belong to this widget"); NazaraAssert(it != m_entities.end(), "Entity does not belong to this widget");
@ -207,7 +207,7 @@ namespace Ndk
it->isEnabled = false; it->isEnabled = false;
}); });
widgetEntity.onEnabledSlot.Connect(newEntity->OnEntityEnabled, [this](Entity* entity) newWidgetEntity.onEnabledSlot.Connect(newEntity->OnEntityEnabled, [this](Entity* entity)
{ {
auto it = std::find_if(m_entities.begin(), m_entities.end(), [&](const WidgetEntity& widgetEntity) { return widgetEntity.handle == entity; }); auto it = std::find_if(m_entities.begin(), m_entities.end(), [&](const WidgetEntity& widgetEntity) { return widgetEntity.handle == entity; });
NazaraAssert(it != m_entities.end(), "Entity does not belong to this widget"); NazaraAssert(it != m_entities.end(), "Entity does not belong to this widget");

6
SDK/src/NDK/Widgets/TextAreaWidget.cpp
View File

@ -582,9 +582,9 @@ namespace Ndk
}; };
// Move text so that cursor is always visible // Move text so that cursor is always visible
const auto* glyph = GetGlyph(m_cursorPositionEnd, nullptr); const auto* lastGlyph = GetGlyph(m_cursorPositionEnd, nullptr);
float glyphPos = (glyph) ? glyph->bounds.x : 0.f; float glyphPos = (lastGlyph) ? lastGlyph->bounds.x : 0.f;
float glyphWidth = (glyph) ? glyph->bounds.width : 0.f; float glyphWidth = (lastGlyph) ? lastGlyph->bounds.width : 0.f;
auto& node = m_textEntity->GetComponent<Ndk::NodeComponent>(); auto& node = m_textEntity->GetComponent<Ndk::NodeComponent>();
float textPosition = node.GetPosition(Nz::CoordSys_Local).x - paddingWidth; float textPosition = node.GetPosition(Nz::CoordSys_Local).x - paddingWidth;

72
plugins/Assimp/Plugin.cpp
View File

@ -255,14 +255,14 @@ MeshRef LoadMesh(Stream& stream, const MeshParams& parameters)
bool animatedMesh = false; bool animatedMesh = false;
if (parameters.animated) if (parameters.animated)
{ {
for (unsigned int i = 0; i < scene->mNumMeshes; ++i) for (unsigned int meshIdx = 0; meshIdx < scene->mNumMeshes; ++meshIdx)
{ {
aiMesh* currentMesh = scene->mMeshes[i]; aiMesh* currentMesh = scene->mMeshes[meshIdx];
if (currentMesh->HasBones()) // Inline functions can be safely called if (currentMesh->HasBones()) // Inline functions can be safely called
{ {
animatedMesh = true; animatedMesh = true;
for (unsigned int j = 0; j < currentMesh->mNumBones; ++j) for (unsigned int boneIdx = 0; boneIdx < currentMesh->mNumBones; ++boneIdx)
joints.insert(currentMesh->mBones[j]->mName.C_Str()); joints.insert(currentMesh->mBones[boneIdx]->mName.C_Str());
} }
} }
} }
@ -284,9 +284,9 @@ MeshRef LoadMesh(Stream& stream, const MeshParams& parameters)
// aiMaterial index in scene => Material index and data in Mesh // aiMaterial index in scene => Material index and data in Mesh
std::unordered_map<unsigned int, std::pair<UInt32, ParameterList>> materials; std::unordered_map<unsigned int, std::pair<UInt32, ParameterList>> materials;
for (unsigned int i = 0; i < scene->mNumMeshes; ++i) for (unsigned int meshIdx = 0; meshIdx < scene->mNumMeshes; ++meshIdx)
{ {
aiMesh* iMesh = scene->mMeshes[i]; aiMesh* iMesh = scene->mMeshes[meshIdx];
if (iMesh->HasBones()) if (iMesh->HasBones())
{ {
// For now, process only skeletal meshes // For now, process only skeletal meshes
@ -303,9 +303,9 @@ MeshRef LoadMesh(Stream& stream, const MeshParams& parameters)
IndexMapper indexMapper(indexBuffer, BufferAccess_DiscardAndWrite); IndexMapper indexMapper(indexBuffer, BufferAccess_DiscardAndWrite);
IndexIterator index = indexMapper.begin(); IndexIterator index = indexMapper.begin();
for (unsigned int j = 0; j < iMesh->mNumFaces; ++j) for (unsigned int faceIdx = 0; faceIdx < iMesh->mNumFaces; ++faceIdx)
{ {
aiFace& face = iMesh->mFaces[j]; aiFace& face = iMesh->mFaces[faceIdx];
if (face.mNumIndices != 3) if (face.mNumIndices != 3)
NazaraWarning("Assimp plugin: This face is not a triangle!"); NazaraWarning("Assimp plugin: This face is not a triangle!");
@ -324,30 +324,30 @@ MeshRef LoadMesh(Stream& stream, const MeshParams& parameters)
BufferMapper<VertexBuffer> vertexMapper(vertexBuffer, BufferAccess_ReadWrite); BufferMapper<VertexBuffer> vertexMapper(vertexBuffer, BufferAccess_ReadWrite);
SkeletalMeshVertex* vertices = static_cast<SkeletalMeshVertex*>(vertexMapper.GetPointer()); SkeletalMeshVertex* vertices = static_cast<SkeletalMeshVertex*>(vertexMapper.GetPointer());
for (std::size_t i = 0; i < vertexCount; ++i) for (std::size_t vertexIdx = 0; vertexIdx < vertexCount; ++vertexIdx)
{ {
aiVector3D normal = iMesh->mNormals[i]; aiVector3D normal = iMesh->mNormals[vertexIdx];
aiVector3D position = iMesh->mVertices[i]; aiVector3D position = iMesh->mVertices[vertexIdx];
aiVector3D tangent = iMesh->mTangents[i]; aiVector3D tangent = iMesh->mTangents[vertexIdx];
aiVector3D uv = iMesh->mTextureCoords[0][i]; aiVector3D uv = iMesh->mTextureCoords[0][vertexIdx];
vertices[i].weightCount = 0; vertices[vertexIdx].weightCount = 0;
vertices[i].normal = normalTangentMatrix.Transform({ normal.x, normal.y, normal.z }, 0.f); vertices[vertexIdx].normal = normalTangentMatrix.Transform({ normal.x, normal.y, normal.z }, 0.f);
vertices[i].position = parameters.matrix * Vector3f(position.x, position.y, position.z); vertices[vertexIdx].position = parameters.matrix * Vector3f(position.x, position.y, position.z);
vertices[i].tangent = normalTangentMatrix.Transform({ tangent.x, tangent.y, tangent.z }, 0.f); vertices[vertexIdx].tangent = normalTangentMatrix.Transform({ tangent.x, tangent.y, tangent.z }, 0.f);
vertices[i].uv = parameters.texCoordOffset + Vector2f(uv.x, uv.y) * parameters.texCoordScale; vertices[vertexIdx].uv = parameters.texCoordOffset + Vector2f(uv.x, uv.y) * parameters.texCoordScale;
} }
for (unsigned int i = 0; i < iMesh->mNumBones; ++i) for (unsigned int boneIdx = 0; boneIdx < iMesh->mNumBones; ++boneIdx)
{ {
aiBone* bone = iMesh->mBones[i]; aiBone* bone = iMesh->mBones[boneIdx];
for (unsigned int j = 0; j < bone->mNumWeights; ++j) for (unsigned int weightIdx = 0; weightIdx < bone->mNumWeights; ++weightIdx)
{ {
aiVertexWeight& vertexWeight = bone->mWeights[j]; aiVertexWeight& vertexWeight = bone->mWeights[weightIdx];
SkeletalMeshVertex& vertex = vertices[vertexWeight.mVertexId]; SkeletalMeshVertex& vertex = vertices[vertexWeight.mVertexId];
std::size_t weightIndex = vertex.weightCount++; std::size_t weightIndex = vertex.weightCount++;
vertex.jointIndexes[weightIndex] = i; vertex.jointIndexes[weightIndex] = boneIdx;
vertex.weights[weightIndex] = vertexWeight.mWeight; vertex.weights[weightIndex] = vertexWeight.mWeight;
} }
} }
@ -445,9 +445,9 @@ MeshRef LoadMesh(Stream& stream, const MeshParams& parameters)
// aiMaterial index in scene => Material index and data in Mesh // aiMaterial index in scene => Material index and data in Mesh
std::unordered_map<unsigned int, std::pair<UInt32, ParameterList>> materials; std::unordered_map<unsigned int, std::pair<UInt32, ParameterList>> materials;
for (unsigned int i = 0; i < scene->mNumMeshes; ++i) for (unsigned int meshIdx = 0; meshIdx < scene->mNumMeshes; ++meshIdx)
{ {
aiMesh* iMesh = scene->mMeshes[i]; aiMesh* iMesh = scene->mMeshes[meshIdx];
if (!iMesh->HasBones()) // Don't process skeletal meshs if (!iMesh->HasBones()) // Don't process skeletal meshs
{ {
unsigned int indexCount = iMesh->mNumFaces * 3; unsigned int indexCount = iMesh->mNumFaces * 3;
@ -461,9 +461,9 @@ MeshRef LoadMesh(Stream& stream, const MeshParams& parameters)
IndexMapper indexMapper(indexBuffer, BufferAccess_DiscardAndWrite); IndexMapper indexMapper(indexBuffer, BufferAccess_DiscardAndWrite);
IndexIterator index = indexMapper.begin(); IndexIterator index = indexMapper.begin();
for (unsigned int j = 0; j < iMesh->mNumFaces; ++j) for (unsigned int faceIdx = 0; faceIdx < iMesh->mNumFaces; ++faceIdx)
{ {
aiFace& face = iMesh->mFaces[j]; aiFace& face = iMesh->mFaces[faceIdx];
if (face.mNumIndices != 3) if (face.mNumIndices != 3)
NazaraWarning("Assimp plugin: This face is not a triangle!"); NazaraWarning("Assimp plugin: This face is not a triangle!");
@ -485,17 +485,17 @@ MeshRef LoadMesh(Stream& stream, const MeshParams& parameters)
VertexMapper vertexMapper(vertexBuffer, BufferAccess_DiscardAndWrite); VertexMapper vertexMapper(vertexBuffer, BufferAccess_DiscardAndWrite);
auto posPtr = vertexMapper.GetComponentPtr<Vector3f>(VertexComponent_Position); auto posPtr = vertexMapper.GetComponentPtr<Vector3f>(VertexComponent_Position);
for (unsigned int j = 0; j < vertexCount; ++j) for (unsigned int vertexIdx = 0; vertexIdx < vertexCount; ++vertexIdx)
{ {
aiVector3D position = iMesh->mVertices[j]; aiVector3D position = iMesh->mVertices[vertexIdx];
*posPtr++ = parameters.matrix * Vector3f(position.x, position.y, position.z); *posPtr++ = parameters.matrix * Vector3f(position.x, position.y, position.z);
} }
if (auto normalPtr = vertexMapper.GetComponentPtr<Vector3f>(VertexComponent_Normal)) if (auto normalPtr = vertexMapper.GetComponentPtr<Vector3f>(VertexComponent_Normal))
{ {
for (unsigned int j = 0; j < vertexCount; ++j) for (unsigned int vertexIdx = 0; vertexIdx < vertexCount; ++vertexIdx)
{ {
aiVector3D normal = iMesh->mNormals[j]; aiVector3D normal = iMesh->mNormals[vertexIdx];
*normalPtr++ = normalTangentMatrix.Transform({normal.x, normal.y, normal.z}, 0.f); *normalPtr++ = normalTangentMatrix.Transform({normal.x, normal.y, normal.z}, 0.f);
} }
} }
@ -505,9 +505,9 @@ MeshRef LoadMesh(Stream& stream, const MeshParams& parameters)
{ {
if (iMesh->HasTangentsAndBitangents()) if (iMesh->HasTangentsAndBitangents())
{ {
for (unsigned int j = 0; j < vertexCount; ++j) for (unsigned int vertexIdx = 0; vertexIdx < vertexCount; ++vertexIdx)
{ {
aiVector3D tangent = iMesh->mTangents[j]; aiVector3D tangent = iMesh->mTangents[vertexIdx];
*tangentPtr++ = normalTangentMatrix.Transform({tangent.x, tangent.y, tangent.z}, 0.f); *tangentPtr++ = normalTangentMatrix.Transform({tangent.x, tangent.y, tangent.z}, 0.f);
} }
} }
@ -519,15 +519,15 @@ MeshRef LoadMesh(Stream& stream, const MeshParams& parameters)
{ {
if (iMesh->HasTextureCoords(0)) if (iMesh->HasTextureCoords(0))
{ {
for (unsigned int j = 0; j < vertexCount; ++j) for (unsigned int vertexIdx = 0; vertexIdx < vertexCount; ++vertexIdx)
{ {
aiVector3D uv = iMesh->mTextureCoords[0][j]; aiVector3D uv = iMesh->mTextureCoords[0][vertexIdx];
*uvPtr++ = parameters.texCoordOffset + Vector2f(uv.x, uv.y) * parameters.texCoordScale; *uvPtr++ = parameters.texCoordOffset + Vector2f(uv.x, uv.y) * parameters.texCoordScale;
} }
} }
else else
{ {
for (unsigned int j = 0; j < vertexCount; ++j) for (unsigned int vertexIdx = 0; vertexIdx < vertexCount; ++vertexIdx)
*uvPtr++ = Vector2f::Zero(); *uvPtr++ = Vector2f::Zero();
} }
} }