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

Refactor fields (step 7) #32488

Merged
merged 14 commits into from
Jul 24, 2019

Conversation

ZhilkinSerg
Copy link
Contributor

Summary

SUMMARY: Infrastructure "Refactor fields (step 7)"

Purpose of change

Unhardcode data structures to allow field type customization/addition via json.

Describe the solution

Current pull request does the following:

  • code refactoring for field immunity (and some minor refactoring elsewhere);
  • unhardcodes several field type attributes (convection temperature modifier, gas absorption factor and field splattering).

Additional context:

Includes changes from #32481.

@ZhilkinSerg ZhilkinSerg added [JSON] Changes (can be) made in JSON [C++] Changes (can be) made in C++. Previously named `Code` Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. labels Jul 19, 2019
src/iuse_actor.cpp Outdated Show resolved Hide resolved
src/vehicle_use.cpp Outdated Show resolved Hide resolved
src/mondeath.cpp Outdated Show resolved Hide resolved
src/mapgen.cpp Outdated Show resolved Hide resolved
src/map.cpp Outdated Show resolved Hide resolved
src/map.cpp Outdated Show resolved Hide resolved
src/map.cpp Outdated Show resolved Hide resolved
@kevingranade kevingranade merged commit 824ddc5 into CleverRaven:master Jul 24, 2019
@ZhilkinSerg ZhilkinSerg deleted the refactor-fields-step-7 branch July 24, 2019 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. [JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants