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

Refactor `script_game_object` money functions to use u32

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

  • Скачать
  • Почтовые патчи
  • Простое отличие
Открыто xottab-duty запросил слияние github/fork/AMS21/refactor/script_game_object_money в dev Окт 26, 2023
  • Обзор 0
  • Коммиты 3
  • Конвейеры 1
  • Изменения 2

Created by: AMS21

Since CInventoryOwner already stored money as a u32 it only makes since for us to also treat it like one.

Also fixed a logic bug which would have allowed the object to transfer money which it didn't have.

Since both values are unsigned subtracting them from another will never produce a negative value.

This bug was also present before changing the function signature since the money variable was implicitly converted to an u32 resulting in the same problem.

Ответственный
Назначить
Проверяющие
Запросить ревью
Оценка трудозатрат
Исходная ветка: github/fork/AMS21/refactor/script_game_object_money