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.

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

Add FileData and FileText #300

Merged
merged 2 commits into from
Mar 2, 2024
Merged

Conversation

furudbat
Copy link
Contributor

@furudbat furudbat commented Mar 2, 2024

Add Wrapper for Files management functions (LoadFileData, LoadFileText, ...)

@RobLoach
Copy link
Owner

RobLoach commented Mar 2, 2024

Great idea! 😉

@RobLoach
Copy link
Owner

RobLoach commented Mar 2, 2024

With these additions, we'll be able to overload the other file methods too. Mind if we get a couple tests in place for them? Just something simple.

@furudbat
Copy link
Contributor Author

furudbat commented Mar 2, 2024

Should be working I moved them from my other Project :^) ... I also add some tests.

I would recommend using a test framework like Catch2 or google-test, like I did in my fork.

@RobLoach RobLoach merged commit 9917777 into RobLoach:master Mar 2, 2024
8 checks passed
@RobLoach
Copy link
Owner

RobLoach commented Mar 2, 2024

Was considering catch2 a while ago, but wanted to keep the syntax similar to raylib formatting. And just needed some simple assertions anyway 😉

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.

2 participants