Attampt to index refCounts with std::string when std::type_index is expected

Open
#373 0 comments 0 reactions 1 assignee View on GitHub

@gpx1000 is already working on this.

Since Jun 1, 2026.

Assessment

This issue has not been assessed yet.

Description

Game Engine

Hi,

In the tutorial on Resource management (specifically here: https://docs.vulkan.org/tutorial/latest/Building_a_Simple_Engine/Engine_Architecture/04_resource_management.html#_resource_manager_storage_architecture_and_type_safety), an associative array is created to stored resources' reference counts based on a specific type. But I have a problem with the following section (https://docs.vulkan.org/tutorial/latest/Building_a_Simple_Engine/Engine_Architecture/04_resource_management.html#_resource_manager_resource_loading_and_caching_logic) in the Load method. In my understanding, when we try to increment/initialize the reference counter for a given resource, it does this :

refCounts[resourceId]++

or this :

refCounts[resourceId] = 1;

but based on the definition of the refCounts associative array, I would have thought it should have done something more like this :

auto& typeReferenceCounters = refCounts[std::type_index(typeid(T))];
typeReferenceCounters[resourceId].refCount++; // or typeReferenceCounters[resourceId].refCount = 1 for the other case

Please tell me if I am wrong, I would be glad to learn from my mistakes.

Best regards,

ajvp

Dominant language
C++
Stars
424
Forks
127
Avg merge
2d 22h
Merged PRs (30d)
7

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from KhronosGroup/Vulkan-Tutorial

All issues in KhronosGroup/Vulkan-Tutorial

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.