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

[raymath.h] Small code refactor #3753

Merged

raysan5 merged 2 commits into raysan5:master from idircarlos:rmath-small-refactor

Jan 22, 2024

Contributor

idircarlos commented Jan 22, 2024

The pull request is divided into 2 small commits, so you may don't want to merge both.

I identified two functions that have duplicated code, so I refactored to avoid it. You might don't want to merge this change because you might prefer the readability, but I just wanted to give it a try in case you agree with it :)
A ternary operator didn't have proper left space as the rest. Some divisions have both spaces also.

idircarlos added 2 commits

January 22, 2024 18:43


          [raymath.h] Small refactor to avoid duplicated code

0f7c460


          Fixing some blank spaces to match same syle convention

be5e8d6

raysan5 merged commit c133fee into raysan5:master

14 checks passed

Owner

raysan5 commented Jan 22, 2024

@idircarlos Nice! Thanks for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet