Removal of all mentions of 50AE #20
Open
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
begin removing 50AE
remove mentions and credits for 50AE
Remove spawn options for 50AE
remove tag definition for 50AE
remove resetcvar and defaults for 50AE
remove cvars for 50AE
remove 50AE from spawn handler
|
Hmm, due to the way Ace built the bit-shifted flags for each ammo type, does this break things in existing INI files, since it's the first one in the list? I was personally just going to wait until a new ammo type was proposed to be added, such as PG's NDM rounds, and simply replace .50 AE with that instead. I do agree these should be removed, though. |
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.
I'm fairly certain that I've removed all mentions of 50AE. Decided to do this after seeing folks getting confused between this and 50AM, hence its removal.