-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Reorganizing server-side source code: entities, monsters and weapons #197
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
7539557
to
ecf9922
Compare
Closed
SNMetamorph
pushed a commit
that referenced
this pull request
Nov 23, 2023
ab0c0a1
to
06a55cb
Compare
SNMetamorph
pushed a commit
that referenced
this pull request
Nov 23, 2023
06a55cb
to
e264daa
Compare
e73a0f8
to
5d54c73
Compare
SNMetamorph
pushed a commit
that referenced
this pull request
Jan 17, 2024
5d54c73
to
41011f2
Compare
SNMetamorph
pushed a commit
that referenced
this pull request
Feb 1, 2024
f86e60e
to
97cff66
Compare
97cff66
to
915ef2d
Compare
SNMetamorph
pushed a commit
that referenced
this pull request
Feb 10, 2024
915ef2d
to
6cc6504
Compare
SNMetamorph
pushed a commit
that referenced
this pull request
Feb 18, 2024
SNMetamorph
pushed a commit
that referenced
this pull request
Feb 29, 2024
6cc6504
to
2ba3831
Compare
Rest of the files that need to be splitted:
|
77c0561
to
1686786
Compare
7833cb8
to
d50366a
Compare
SNMetamorph
pushed a commit
that referenced
this pull request
Mar 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Originally, in HLSDK bunch of code was clumped in single .cpp files with thousands lines of code in them. That isn't easy to maintain and understand, especially for beginner developers. Also less clumped structure of source code will lead to better compiling time, since different translation units may being compiled simultaneously. In final, every generic entity and weapon will be located in separate pair of source files: modifying existing and adding new objects will be relatively easy.