-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat/new/validations #7
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…idering minimap position in screen. Also creating player position structure and implementing it to t_cub3d
…ows and rotate the player direction minimap. Also inserting mlx struct and image pointer into cub struct
… then finish minimap
…ication; including drawing background function
…or was causing not print de wall in that ray. Fixed using a condition in draw_rays function to check if perpwalldist is equal to 0, and if this is true, then set that ray to print from the beggining of the screen to the end.
…ill not working 100% yet.
… loaded by mlx and use this to draw the texture. - Still not working yet and will need a lot of refactor. -Need to convert textures from dir textures to png in fact.
…ure at s_cub3d struct to load it at the beggining of exec(need improvements); also implemented function to get color to return the address of the array position of texture->pixels already converting backwards with bitwise XOR operation to use it in putpixel function later
…serting improvements like N,S,W and E textures. Still having problems with correct insertion of texturesd
…ture t_cub3d; Redifing MACROS; Changing functions prototype
…t into more files to be ok!
…refactor mlx_handler.c file into more functions and fix texture implementation
…pecific jobs. Inserting all this functions into a new file called player_movements. This let the code much better and readable
…n. ) Inserting 42 header.
… to understand why and fix it
Introduced a `test` target to the Makefile to enable unit testing using Criterion. This includes test-specific variables, compilation rules, and automated cleanup of the test binary. Streamlines the testing process for the project.
Introduces functions for file validation, scene loading, and getter utilities. Implements structure updates and validation for `.cub` and `.png` files, textures, and RGB color parsing to enhance configuration handling.
Moved scene initialization and cleanup logic into dedicated functions (`run_scene` and `clean_scene`) to improve code modularity and readability. Replaced `ssize_t` with `int` for file descriptors for better type consistency. Added a new error message for MLX initialization failure.
…d into feat/new/validations
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.