Skip to content

Update box2d to v2.4.2#275

Merged
daid merged 2 commits intodaid:masterfrom
oznogon:box-242
Oct 31, 2025
Merged

Update box2d to v2.4.2#275
daid merged 2 commits intodaid:masterfrom
oznogon:box-242

Conversation

@oznogon
Copy link
Contributor

@oznogon oznogon commented Oct 30, 2025

Another go at #67 and #222, bumped to the final Box2D v2 release.

v2.4.1 introduced a breaking change to user data pointers, which this also attempts to incorporate.

As of box2d 2.4.1, "default user data is now
uintptr_t instead of void*". getUserData() returns
a struct containing a pointer instead.

See erincatto/box2d/commit/f0f9d50a328a709cc3a287a61b864e7d0e3ef35f.
@daid
Copy link
Owner

daid commented Oct 30, 2025

What are the benefits of updating? And the risks?

@oznogon
Copy link
Contributor Author

oznogon commented Oct 31, 2025

Relevant to SP that I can spot:

The only breaking change affecting SP that I could find was the change in user data using a struct member instead of a raw pointer. The other significant changes all appear to be related to net-new additions like linear shape casting, or fixes or changes in behavior for ropes, joints, and concave polygons, none of which I believe are used in EE/SP.

Or as someone smarter than me said about 2.4.1 in #67:

Daid: Not yet, might merge that now, not sure if it changes much. The changelog is petty small and most changes are in features not used by EE

I'm more concerned about potential issues with building on platforms that I don't build on, like MSVC.

@daid
Copy link
Owner

daid commented Oct 31, 2025

Well, mscv isn't an issue. And with all the ecs changes we can just pile this up as well I guess

@daid daid merged commit b9ad837 into daid:master Oct 31, 2025
6 checks passed
@oznogon oznogon deleted the box-242 branch October 31, 2025 16:49
@oznogon oznogon mentioned this pull request Oct 31, 2025
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