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 change_side() #36103

Merged
merged 2 commits into from
Dec 22, 2019
Merged

refactor change_side() #36103

merged 2 commits into from
Dec 22, 2019

Conversation

KorGgenT
Copy link
Member

@KorGgenT KorGgenT commented Dec 14, 2019

Summary

SUMMARY: Infrastructure "refactor change_side()"

Purpose of change

moves change_side() from player to Character, removes superfluous game::change_side, and changes parameter from int pos to item_location loc

Describe the solution

the solution is the same as the purpose

Testing

load up a game and swap a wrist watch

Additional notes

requires #36016 as this pr is based on those commits

@KorGgenT KorGgenT added [C++] Changes (can be) made in C++. Previously named `Code` Code: Infrastructure / Style / Static Analysis Code internal infrastructure and style 0.E Feature Freeze labels Dec 14, 2019
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` Code: Infrastructure / Style / Static Analysis Code internal infrastructure and style
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants