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 generic #93

Merged
merged 22 commits into from Apr 6, 2019

Conversation

Projects
None yet
3 participants
@gitMarky
Copy link
Contributor

commented Apr 1, 2019

Lots of code style changes, some simplifications. Wanted to do C4Object.cpp first, but that is not really viable - I prefer smaller files for that, also found lots of things that I wanted to change in that file, but splitting it up, similar to C4Movement.cpp should be done first.

Show resolved Hide resolved src/object/C4Id.h Outdated
Show resolved Hide resolved src/object/C4ObjectScript.cpp Outdated
Show resolved Hide resolved src/object/C4ObjectScript.cpp Outdated
Show resolved Hide resolved src/object/C4Id.h
Show resolved Hide resolved src/object/C4ObjectInfoList.cpp Outdated
Show resolved Hide resolved src/object/C4ObjectList.cpp Outdated
Show resolved Hide resolved src/object/C4ObjectList.cpp Outdated
Show resolved Hide resolved src/object/C4ObjectList.cpp Outdated
@isilkor

isilkor approved these changes Apr 2, 2019

@gitMarky gitMarky merged commit 3497a9a into openclonk:master Apr 6, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@gitMarky gitMarky deleted the gitMarky:refactor_generic branch Apr 6, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.