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

modernization cleanups #35

Merged
merged 3 commits into from Nov 27, 2018

Conversation

Projects
None yet
3 participants
@t4im
Contributor

t4im commented Nov 26, 2018

Uses default constructors, using instead of typedef and makes the Entity message enum typesafe to guard against clashes with MSG_* macros (e.g. socket.h already uses that). Replaces c-style casts with static_casts for compile-time over runtime checking.

@t4im t4im added the T-Cleanup label Nov 26, 2018

@slipher

This comment has been minimized.

slipher commented Nov 26, 2018

LGTM

Show resolved Hide resolved templates/Backend.cpp Outdated
@Kangz

Kangz approved these changes Nov 27, 2018

@t4im t4im force-pushed the t4im:master branch from 89b0ef9 to 1d6874b Nov 27, 2018

@t4im t4im merged commit b5b85b9 into DaemonEngine:master Nov 27, 2018

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