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
kill CNetworkServices global #13785
kill CNetworkServices global #13785
Conversation
consider incorporating some ala 56514f1 so you kill the ifdef spaghetti. |
03a51ad
to
d67a526
Compare
@notspiff I'll add it. thanks. |
d67a526
to
2e8f7c3
Compare
@@ -160,6 +167,8 @@ class CNetwork | |||
#include "platform/win32/network/NetworkWin32.h" | |||
#elif defined(HAS_WIN10_NETWORK) | |||
#include "platform/win10/network/NetworkWin10.h" | |||
#else | |||
using CNetwork = CNetworkBase; |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
a112bd9
to
5644b22
Compare
and add typedefs to CNetwork for specific platforms. in preparation for rtti'ing the service
5644b22
to
594d81c
Compare
jenkins build this please |
@garbear is this good to go? |
4 days, no comment. i'm merging. |
#ifdef HAS_WEB_SERVER | ||
: | ||
m_settings(settings), |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
see title