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

[NOSQUASH] TouchScreenGUI refactor #13640

Merged
merged 2 commits into from
Jul 14, 2023
Merged

[NOSQUASH] TouchScreenGUI refactor #13640

merged 2 commits into from
Jul 14, 2023

Commits on Jul 14, 2023

  1. Fix some memory leaks and code style issues 

    Maximum line length is 95 characters.
Some members' name are changed.
Struct initialisations use brace syntax; eliminating the usage of the memset function.
Iterations use for-each-loop instead of while-loop+iterator.
char * -> std::string
button_info * -> std::shared_ptr<button_info>
    @srifqi
    srifqi committed Jul 14, 2023
    Configuration menu
    Copy the full SHA
    6121851 View commit details
    Browse the repository at this point in the history

  2. Simplifies code by using Irrlicht's operator overloads 

    New variables are added to replace in-place calculations.
    @srifqi
    srifqi committed Jul 14, 2023
    Configuration menu
    Copy the full SHA
    fa53d32 View commit details
    Browse the repository at this point in the history