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

CMake refactoring #1535

Merged
merged 4 commits into from
Dec 1, 2023
Merged

CMake refactoring #1535

merged 4 commits into from
Dec 1, 2023

Conversation

Masterkatze
Copy link
Contributor

No description provided.

@Masterkatze Masterkatze self-assigned this Nov 19, 2023
@github-actions github-actions bot added Renderer AI Artificial Intelligence Sound UI Physics External (3rd party) This issue is related to external component used by our project. labels Nov 19, 2023
@Masterkatze Masterkatze force-pushed the cmake branch 6 times, most recently from 222fac4 to bf43070 Compare November 27, 2023 01:02
@Masterkatze Masterkatze marked this pull request as ready for review November 27, 2023 17:30
Copy link
Member

@Xottab-DUTY Xottab-DUTY left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Masterkatze, please, fix compilation with smart_cast enabled (without PURE_DYNAMIC_CAST define). This is really crucial for performance.

@Masterkatze Masterkatze dismissed Xottab-DUTY’s stale review December 1, 2023 21:29

PURE_DYNAMIC_CAST was enabled, disabling it will be done in another PR

@Masterkatze Masterkatze merged commit 64778f1 into OpenXRay:dev Dec 1, 2023
21 checks passed
@Masterkatze Masterkatze deleted the cmake branch December 1, 2023 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI Artificial Intelligence Code Quality External (3rd party) This issue is related to external component used by our project. Physics Renderer Sound UI
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants