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

Delete and move around stuff... #200

Merged
merged 4 commits into from Apr 17, 2019

Conversation

Projects
None yet
3 participants
@slipher
Copy link
Contributor

slipher commented Apr 7, 2019

No description provided.

@illwieckz illwieckz added the cleanup label Apr 8, 2019

@illwieckz

This comment has been minimized.

Copy link
Member

illwieckz commented Apr 10, 2019

  • Remove unused startup code: LGTM
  • Fix 2 unused variable warnings: LGTM
  • Move some download functions to a new file: LGTM
  • Remove unused "binary message" code: what the code was for?
@slipher

This comment has been minimized.

Copy link
Contributor Author

slipher commented Apr 10, 2019

I don't know. Apparently it is some hack added in ET.

@slipher

This comment has been minimized.

Copy link
Contributor Author

slipher commented Apr 10, 2019

I can say roughly what it does which is some out of band way that the sgame can send data to the cgame.

@slipher slipher merged commit 2f1cbf9 into DaemonEngine:master Apr 17, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.