• X xray-16
  • Информация о проекте
    • Информация о проекте
    • Активность
    • Метки
    • Участники
  • Репозиторий
    • Репозиторий
    • Файлы
    • Коммиты
    • Ветки
    • Теги
    • Участники
    • Диаграмма
    • Сравнение
  • Задачи 216
    • Задачи 216
    • Список
    • Доски
    • Спринты
  • Запросы на слияние 31
    • Запросы на слияние 31
  • CI/CD
    • CI/CD
    • Конвейеры
    • Задания
    • Расписания
  • Развертывания
    • Развертывания
    • Окружения
    • Релизы
  • Пакеты и реестры
    • Пакеты и реестры
    • Реестр пакетов
    • Реестр контейнеров
  • Мониторинг
    • Мониторинг
    • Инциденты
  • Аналитика
    • Аналитика
    • Поток ценности
    • CI/CD
    • Репозиторий
  • Wiki
    • Wiki
  • Сниппеты
    • Сниппеты
  • Активность
  • Диаграмма
  • Создать новую задачу
  • Задания
  • Коммиты
  • Доски с задачами
Свернуть панель
  • OpenXRay
  • xray-16
  • Запросы на слияние
  • !1361

[Vanilla Bugfix]: corrected create_hud_item logic for duplicate hud section (closes #1359)

  • Ревью изменений

  • Скачать
  • Почтовые патчи
  • Простое отличие
Закрыто xottab-duty запросил слияние github/fork/yohjimane/fix/1359 в dev Май 31, 2023
  • Обзор 0
  • Коммиты 2
  • Конвейеры 0
  • Изменения 4

Created by: yohjimane

Fixes issue 1359

The root issue is this function:

attachable_hud_item* player_hud::create_hud_item(const shared_str& sect)
{
    auto& item = m_pool[sect]; // each key in m_pool is set to hud_sect. These are not unique.

    if (!item)  // If equipped item 1 hud_sect == equipped item 2 hud_sect, the line below is not called, causing the bug.
        item = xr_new<attachable_hud_item>(this, sect, m_model);

    return item;
}

The fix:

  1. Add instance id m_inst_id to CHudItem, which gets incremented any time a new item gets loaded.
  2. Add m_unique_hud_sect to CHudItem, which = hud_sect + m_inst_id
  3. Refactor create_hud_item to take CHudItem as parameter & key m_pool by m_unique_hud_sect to avoid duplicate item bug.
Ответственный
Назначить
Проверяющие
Запросить ревью
Оценка трудозатрат
Исходная ветка: github/fork/yohjimane/fix/1359