Skip to content

[rcore] Removing #pragma message and adding TRACELOG inside the function#5596

Merged
raysan5 merged 2 commits into
raysan5:masterfrom
maiconpintoabreu:fix-rcore-warnings
Feb 26, 2026
Merged

[rcore] Removing #pragma message and adding TRACELOG inside the function#5596
raysan5 merged 2 commits into
raysan5:masterfrom
maiconpintoabreu:fix-rcore-warnings

Conversation

@maiconpintoabreu

Copy link
Copy Markdown
Contributor

Making Warning message consistent and avoiding Warning every build by default.
Extra: Cosmetic refactor in raymath

@maiconpintoabreu

Copy link
Copy Markdown
Contributor Author

Left it as Draft as I will test again on Windows.

@dreais

dreais commented Feb 26, 2026

Copy link
Copy Markdown

Just pointing out the irony after this post: https://x.com/raysan5/status/2026929451938808072

Good luck!

@raysan5 raysan5 marked this pull request as ready for review February 26, 2026 22:45
@raysan5 raysan5 merged commit 05a34b0 into raysan5:master Feb 26, 2026
15 checks passed
@raysan5

raysan5 commented Feb 26, 2026

Copy link
Copy Markdown
Owner

@maiconpintoabreu thanks for the review! Maybe I'm further reviewing it to simplify it a bit... too many flags involved...

@maiconpintoabreu

Copy link
Copy Markdown
Contributor Author

@raysan5 maybe we could combine the warnings later and just say what it needs to run instead of a validation by "field".

@raysan5

raysan5 commented Feb 26, 2026

Copy link
Copy Markdown
Owner

@raysan5 maybe we could combine the warnings later and just say what it needs to run instead of a validation by "field".

Yeah, that was my thought, it will be easier to read with same result, do you want to send another PR?

@maiconpintoabreu

Copy link
Copy Markdown
Contributor Author

@raysan5 sure I will do it.

@maiconpintoabreu maiconpintoabreu deleted the fix-rcore-warnings branch March 10, 2026 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants