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.

Sign up for GitHub

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

Jump to bottom

static_cast vs C-style casts #52

Open
jonathonbrady opened this issue Sep 9, 2022 · 2 comments
Open

static_cast vs C-style casts #52

jonathonbrady opened this issue Sep 9, 2022 · 2 comments
Assignees
@EpochFlame

Comments

@jonathonbrady
Copy link
Collaborator

We are using both all over the place. Should prefer C++ casts.
@EpochFlame EpochFlame self-assigned this Nov 25, 2022
@intns
Copy link
Collaborator

intns commented Sep 14, 2023

Need to have a look through the code-base and convert the current usages:

@HeartPiece44 @SodiumChlorideLogic @EpochFlame 👀

@intns
Copy link
Collaborator

intns commented May 12, 2024

Bump too. Now we're consolidating files I'll be going through them and seeing what can be statically casted. IIRC they affect code generation, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@EpochFlame EpochFlame

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jonathonbrady @EpochFlame @intns