Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactor script_game_object money functions to use u32 #1479

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Refactor script_game_object money functions to use u32 #1479

wants to merge 3 commits into from

Commits on May 14, 2024

  1. xrGame: Refactor script_game_object money functions to use u32 

    Since `CInventoryOwner` also uses `u32` for its `money` member
    @AMS21
    AMS21 committed May 14, 2024
    Configuration menu
    Copy the full SHA
    377199e View commit details
    Browse the repository at this point in the history

  2. xrGame: Fix logic error in TransferMoney 

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

This bug was also present before the previous commit since the `money`
variable was implicitly converted to an `u32` resulting in the same bug.
    @AMS21
    AMS21 committed May 14, 2024
    Configuration menu
    Copy the full SHA
    a203c65 View commit details
    Browse the repository at this point in the history

  3. Misc: Fix typo enought -> enough

    @AMS21
    AMS21 committed May 14, 2024
    Configuration menu
    Copy the full SHA
    9238c63 View commit details
    Browse the repository at this point in the history