Skip to content

box2d physics library #63

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

box2d physics library #63

wants to merge 8 commits into from

Conversation

HMart81
Copy link
Contributor

@HMart81 HMart81 commented Mar 28, 2025

Box2D is a popular 2D Physics engine for 2d games, docs and more on https://box2d.org/

add ID values for GuiIconName to make it easy to embed icons id's in strings
changed loadDirectoryFiles to raylib 55 to follow new function signture and unloadDirectoryFiles missed the necessary argument
Box2D is a 2D Physics engine, see more on https://box2d.org/
Tried to follow c3 vendor guide lines "conform to C3 stdlib layout (tabs, Allman braces)" didn't do this first because was unaware of them, don't known if the tabs stuff is fully correct, thou I changed the leading ones to real tab, hope is enough.
@lerno
Copy link
Contributor

lerno commented Mar 30, 2025

This looks like this is also making changes to the raylib bindings? Also, it's 0.7.0 and 0.6.8, not 7.0.0 and 6.8.0 :D

@HMart81
Copy link
Contributor Author

HMart81 commented Mar 30, 2025

Oh sorry about that! I'm still a noob at this github technical stuff (yeh shows I'm not a professional programmer, i'm just a self taught one), if possible ignore the raygui and raylib stuff.
And I most have been on drugs about the 6.8.0 and 7.0.0 versioning numbers, I don't know how the hell I thought that.

@HMart81
Copy link
Contributor Author

HMart81 commented Mar 30, 2025

Thou looking at the two raylib changes I did (I did them some time ago, so I forgot about it...) they do make sense, as those were wrong function definitions/bindings in the original, unless those functions were already corrected in the mean time, I recommend that you also merge my raylib changes.

@lerno
Copy link
Contributor

lerno commented Mar 31, 2025

I've removed the 0.6.8 versions and I'm doing 0.7.0 now exclusively. Could you update the bindings and changes for it?

@HMart81
Copy link
Contributor Author

HMart81 commented Mar 31, 2025

Yes I can. :)

@lerno
Copy link
Contributor

lerno commented Apr 1, 2025

Could you do these changes?

@HMart81
Copy link
Contributor Author

HMart81 commented Apr 2, 2025

I already did the updates to 0.7.0, if that is what you are asking?

@lerno
Copy link
Contributor

lerno commented Apr 2, 2025

No, there are some new changes requested, see in the boxes above!

@HMart81
Copy link
Contributor Author

HMart81 commented Apr 4, 2025

Sorry for my ignorance but what boxes above? Can you point me to what you are talking about. Or can you write what you want me to change in a comment here?
I'm used to tortoise svn for most of my programing, so I still don't know all ins and outs of this website.

HMart81 added 2 commits April 5, 2025 15:23
changed the code to follow more closely @lerno suggestions (still not tested at this point...)
now fully compiles
@HMart81 HMart81 requested a review from lerno April 6, 2025 15:26
@lerno
Copy link
Contributor

lerno commented Apr 6, 2025

I'm sorry I haven't checked this yet. I'll try to have time soon. 🙏

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