Skip to content
Snippets Groups Projects
Verified Commit 58623b14 authored by Tobias Frisch's avatar Tobias Frisch
Browse files

[#104] Fixed some warnings in asset_loader

parent 27bae72a
No related branches found
No related tags found
1 merge request!87Resolve "Pass Debug Profiling"
Pipeline #26939 passed
...@@ -107,7 +107,7 @@ namespace vkcv::asset { ...@@ -107,7 +107,7 @@ namespace vkcv::asset {
const std::vector<fx::gltf::BufferView> &bufferViews, const std::vector<fx::gltf::BufferView> &bufferViews,
std::vector<VertexAttribute> &dst) { std::vector<VertexAttribute> &dst) {
for (const auto &attrib : src) { for (const auto &attrib : src) {
VertexAttribute att; VertexAttribute att {};
if (attrib.first == "POSITION") { if (attrib.first == "POSITION") {
att.type = PrimitiveType::POSITION; att.type = PrimitiveType::POSITION;
...@@ -282,7 +282,7 @@ namespace vkcv::asset { ...@@ -282,7 +282,7 @@ namespace vkcv::asset {
* modes, default is repeat, the other modes aren't available. * modes, default is repeat, the other modes aren't available.
*/ */
static vkcv::asset::Sampler loadSampler(const fx::gltf::Sampler &src) { static vkcv::asset::Sampler loadSampler(const fx::gltf::Sampler &src) {
Sampler dst; Sampler dst {};
dst.minLOD = 0; dst.minLOD = 0;
dst.maxLOD = VK_LOD_CLAMP_NONE; dst.maxLOD = VK_LOD_CLAMP_NONE;
...@@ -414,13 +414,13 @@ namespace vkcv::asset { ...@@ -414,13 +414,13 @@ namespace vkcv::asset {
} }
if (posAccessor.bufferView >= sceneObjects.bufferViews.size()) { if (posAccessor.bufferView >= sceneObjects.bufferViews.size()) {
vkcv_log(LogLevel::ERROR, "Access to bufferView out of bounds: %lu", vkcv_log(LogLevel::ERROR, "Access to bufferView out of bounds: %d",
posAccessor.bufferView); posAccessor.bufferView);
return ASSET_ERROR; return ASSET_ERROR;
} }
const fx::gltf::BufferView& vertexBufferView = sceneObjects.bufferViews[posAccessor.bufferView]; const fx::gltf::BufferView& vertexBufferView = sceneObjects.bufferViews[posAccessor.bufferView];
if (vertexBufferView.buffer >= sceneObjects.buffers.size()) { if (vertexBufferView.buffer >= sceneObjects.buffers.size()) {
vkcv_log(LogLevel::ERROR, "Access to buffer out of bounds: %lu", vkcv_log(LogLevel::ERROR, "Access to buffer out of bounds: %d",
vertexBufferView.buffer); vertexBufferView.buffer);
return ASSET_ERROR; return ASSET_ERROR;
} }
...@@ -571,7 +571,7 @@ namespace vkcv::asset { ...@@ -571,7 +571,7 @@ namespace vkcv::asset {
texture.sampler = -1; texture.sampler = -1;
} else } else
if (static_cast<size_t>(textureObject.sampler) >= scene.samplers.size()) { if (static_cast<size_t>(textureObject.sampler) >= scene.samplers.size()) {
vkcv_log(LogLevel::ERROR, "Sampler of texture '%s' missing (%s) %d", vkcv_log(LogLevel::ERROR, "Sampler of texture '%s' missing (%s)",
textureObject.name.c_str(), path.c_str()); textureObject.name.c_str(), path.c_str());
return ASSET_ERROR; return ASSET_ERROR;
} else { } else {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment