forked from CleverRaven/Cataclysm-DDA
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Follow up #50
Merged
Merged
Follow up #50
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
Made second vet office. Modification of spawns will come out at later date.
Code optimizations reported by static code analysis (2020-02-18)
* Corrected new 22 casing * remove extraneous fields Remove fields relating to ammo. tested changes by disassembling .22 ammo into this casing, then crafting reloaded ammo
Move string consts into single file
One of the changes introduced in PR #37787 dropped the conditional check for character strength being *equal* to exertion required, and instead applied the "It takes some time" movement penalty to everyone (regardless of strength). And, due to the first `if` being changed to `ex >= u.get_str() + 1`, there was no case for `ex == u.get_str()`, allowing characters with a very specific strength, just 1 less than the strength needed, to avoid any movement penalty at all. Fixes #38104 The pain penalty is preserved (with comment), and a narrower condition (potentially matching 2 different strength stat values) gives the "It takes some time" message with the same movement penalty. Any character with sufficient (or more than sufficient) strength to drag the vehicle will see no message and receive no additional movement penalty.
Don't penalize strong characters for dragging vehicles
Move string consts/rename string consts
add lore documentation to the magiclysm mod
- allow build properties override from local.properties; - report values for build properties.
Updated Android build process (2020-02-21)
Added "Trap Back button" option to Android prelaunch menu
Magazine name to object
Skills were in arbitrary order, making it annoying to find and modify skills for debugging. This alphabetizes them to make devs' lives easier.
Magazine name to object
Bionic name to object
Alphabetize the debug skill wishlist
Tweak mapgen for animal pound
…ragment mass. (#37710) * Smaller fragments Change fragment from nailbombs to 0.4g (half a nail) And change to 1g from 2g for fragmenting metal bombs.
* house palette tweak minor changes to make palette more versatile * Update house_general_palette.json
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.
Summary
Purpose of change
Describe the solution
Describe alternatives you've considered
Testing
Additional context