Add some useful comments and commented out immutability checks.

Author: achalpandeyy

include/nbl/asset/ICPUDescriptorSet.h

@@ -70,6 +70,14 @@ class ICPUDescriptorSet final : public IDescriptorSet<ICPUDescriptorSetLayout>,
 
 	inline core::SRange<SDescriptorInfo> getDescriptorInfoStorage(const IDescriptor::E_TYPE type) const
 	{
+		// TODO: @Hazardu
+		// Cannot do the mutability check here because it requires the function to be non-const, but the function cannot be non-const because it's called
+		// from const functions in the asset converter.
+		// Relevant comments/conversations:
+		// https://github.com/Devsh-Graphics-Programming/Nabla/pull/345#discussion_r1054258384
+		// https://github.com/Devsh-Graphics-Programming/Nabla/pull/345#discussion_r1056289599
+		// 
+		// assert(!isImmutable_debug());
 		if (!m_descriptorInfos[static_cast<uint32_t>(type)])
 			return { nullptr, nullptr };
 		else

include/nbl/video/IDescriptorPool.h

@@ -72,7 +72,7 @@ class IDescriptorPool : public core::IReferenceCounted, public IBackendObject
             for (auto i = 0; i < static_cast<uint32_t>(asset::IDescriptor::E_TYPE::ET_COUNT); ++i)
                 m_descriptorAllocators[i] = std::make_unique<allocator_state_t>(m_creationParameters.maxDescriptorCount[i], m_creationParameters.flags.hasFlags(ECF_FREE_DESCRIPTOR_SET_BIT));
 
-            // For (possibly) mutable samplers.
+            // For mutable samplers. We don't know if there will be mutable samplers in sets allocated by this pool when we create the pool.
             m_descriptorAllocators[static_cast<uint32_t>(asset::IDescriptor::E_TYPE::ET_COUNT)] = std::make_unique<allocator_state_t>(m_creationParameters.maxDescriptorCount[static_cast<uint32_t>(asset::IDescriptor::E_TYPE::ET_COMBINED_IMAGE_SAMPLER)], m_creationParameters.flags.hasFlags(ECF_FREE_DESCRIPTOR_SET_BIT));
 
             // Initialize the storages.

src/nbl/video/IDescriptorPool.cpp

@@ -27,6 +27,7 @@ bool IDescriptorPool::reset()
         if (!m_descriptorAllocators[t])
             continue;
 
+        // TODO(achal): I should call the dtors on descriptors only after calling vkResetDescriptorPool (in reset_impl)
         const uint32_t allocatedCount = m_descriptorAllocators[t]->getAllocatedDescriptorCount(allowsFreeing);
         std::destroy_n(getDescriptorStorage(static_cast<asset::IDescriptor::E_TYPE>(t)), allocatedCount);