Skip to content

Comments

fix: all memory leaks and resource management #103

Open
Varnill wants to merge 24 commits intomainfrom
fix
Open

fix: all memory leaks and resource management #103
Varnill wants to merge 24 commits intomainfrom
fix

Conversation

@Varnill
Copy link
Collaborator

@Varnill Varnill commented Sep 28, 2025

Исправления и улучшения

Критические исправления

  • Исправлены множественные утечки памяти в различных компонентах движка
  • Устранено двойное освобождение ресурсов
  • Исправлена инициализация значений (Valgrind warnings)
  • Исправлен device для transparentPipeline
  • Корректное уничтожение ImGui ресурсов

Архитектурные улучшения

  • Внедрение RAII для автоматического управления ресурсами ImGui
  • Рефакторинг ключевых компонентов:
    • Выделение Device из VulkanEngine
    • Рефакторинг Allocator, Sampler, Instance
    • Создание отдельного класса VulkanInit
  • Переработана система очистки ресурсов:
    • Перенос деструкторов в отдельный файл (dtor.cpp)
    • Очистка ресурсов в деструкторах
    • Удаление бесполезного createInstance

Теперь valgrind и санитайзеры не показывают ни одной утечки памяти. Благодаря этому удалось избавиться от ошибок при запуске демки.

Другое

  • Замена nullptr на VK_NULL_HANDLE для ясности
  • Упорядочивание кодовой базы и улучшение структуры проекта

@Varnill Varnill changed the title fix build fix: all memory leaks and resource management Sep 28, 2025
@Varnill Varnill marked this pull request as ready for review September 28, 2025 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants