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

Towards flexibility #15

Open
wants to merge 49 commits into
base: master
Choose a base branch
from
Open

Towards flexibility #15

wants to merge 49 commits into from

Conversation

cmorty
Copy link

@cmorty cmorty commented Dec 7, 2014

Here are a lot of changes I made to use this for a wedding. During the end I got bored making clean commits, but I still think the code is in better shape than at the beginning. And maybe someone can pull some ideas out of it. Changes include:

  • Board looks more like the original
  • Board can have an arbitrary number of fields
  • Points can be arbitrary
  • Unlimited number of players
  • Lots of QLists instead of static arrays
  • Code design was clean a bit.
    Regressions:
  • Saving does not work (the format was so crude I was turned of fixing it)
  • Although I closed some memory leaks I might have introduced new ones
  • Never crashed in the final version, but well.... Might need some more testing
  • Only tested with Windows. Not sure how the full screen stuff will behave on linux.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants