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

Decouple websocketpp from the rest of the program #44

Closed
ctrlaltf2 opened this issue Jun 2, 2020 · 2 comments
Closed

Decouple websocketpp from the rest of the program #44

ctrlaltf2 opened this issue Jun 2, 2020 · 2 comments
Assignees
@ctrlaltf2
Labels
1.0-issues enhancement medium effort refactor server Work on the server is needed to close this issue.
Projects
First Release

Comments

@ctrlaltf2
Copy link
Owner

This is a step towards (#35).
@ctrlaltf2 ctrlaltf2 added server Work on the server is needed to close this issue. medium effort enhancement labels Jun 2, 2020
@ctrlaltf2 ctrlaltf2 self-assigned this Jun 2, 2020
@ctrlaltf2 ctrlaltf2 added this to In progress in First Release Jun 2, 2020
@ctrlaltf2 ctrlaltf2 mentioned this issue Jun 2, 2020
Closed
@ctrlaltf2
Copy link
Owner Author

See https://github.com/ctrlaltf2/lets-play/tree/abstract-network

@modeco80
Copy link
Contributor

modeco80 commented Feb 9, 2021 •
edited

Call me stupid but I'd use CRTP for this since there's not really any scenario where you have to switch at runtime unless for cases you could write into the transport (also just since the call is elided very quickly thanks to the fact that this causes the compiler to be able to do more aggressive optimization)

something ala
https://godbolt.org/z/cKP659

(also yes, I know it's 3AM and I shouldn't be up.)

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

@ctrlaltf2 ctrlaltf2

Labels
1.0-issues enhancement medium effort refactor server Work on the server is needed to close this issue.
Projects
No open projects
First Release
  
In progress
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ctrlaltf2 @modeco80