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

Remove box2d-optimized #1913

Merged
merged 4 commits into from
May 22, 2024
Merged

Remove box2d-optimized #1913

merged 4 commits into from
May 22, 2024

Conversation

aismann
Copy link
Contributor

@aismann aismann commented May 22, 2024

Removed Box2D-optimized
This project is now obsolete;
Erin Catto has developed Box2D v3 with many performance improvements, so please use that instead

Describe your changes

Issue ticket number and link

Checklist before requesting a review

For each PR

  • Add Copyright if it missed:
    - "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

  • I have performed a self-review of my code.

    Optional:

    • I have checked readme and add important infos to this PR.
    • I have added/adapted some tests too.

For core/new feature PR

  • I have checked readme and add important infos to this PR.
  • I have added thorough tests.

Removed Box2D-optimized
This project is now obsolete; Erin Catto has developed Box2D v3 with many performance improvements, so please use that instead
@halx99
Copy link
Collaborator

halx99 commented May 22, 2024

shoud we also need remove box2d-optimized from 3rdparty?

@halx99
Copy link
Collaborator

halx99 commented May 22, 2024

@halx99 halx99 changed the title Update README.md Remove box2d-optimized May 22, 2024
@halx99 halx99 merged commit e1b3023 into axmolengine:dev May 22, 2024
@halx99 halx99 added this to the 2.1.3 milestone May 22, 2024
@halx99 halx99 added the enhancement New feature or request label May 22, 2024
@aismann
Copy link
Contributor Author

aismann commented May 22, 2024

@halx99
Thanks for removing on CMakeLists.txt.

@aismann aismann deleted the patch-4 branch May 23, 2024 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants