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

Coin additions #895

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

Coin additions #895

wants to merge 2 commits into from

Conversation

NyakoFox
Copy link
Contributor

@NyakoFox NyakoFox commented Oct 31, 2022

This pull request adds a few features, which don't¹ affect gameplay. It does the following:

  • Reimplements the coin display from the VVVVVV SS1 prototype
  • Makes coins use their own ID array in save files, instead of sharing the trinket ID array
  • Adds coin editor entities² to allow external tools to use coins, and to help the game figure out when it should render the coin display
  • Unlimits the coin amount and stores it differently than other collectibles

[1] Technically, making coins use their own ID array no longer allows level makers to use iftrinkets(x) to check if you've gotten coins, however the interaction between coins and trinkets made this hard to use anyway... I don't think any levels did this.
[2] This PR adds coin editor entities, however it does not add a new editor tool. I personally just don't feel like I'm the one who should make such a big decision like adding a new tool.

Legal Stuff:

By submitting this pull request, I confirm that...

  • My changes may be used in a future commercial release of VVVVVV
  • I will be credited in a CONTRIBUTORS file and the "GitHub Friends"
    section of the credits for all of said releases, but will NOT be compensated
    for these changes

@InfoTeddy InfoTeddy self-assigned this Oct 31, 2022
@InfoTeddy InfoTeddy changed the title [2.5] Coin additions Coin additions Oct 31, 2022
@NyakoFox NyakoFox marked this pull request as ready for review October 31, 2022 03:12
- Make coins use their own ID array in save files, instead of sharing the trinket ID array

- Add coin editor entities to allow external tools to use coins, and help the game figure out when it should render the coin display

- Unlimits the coin amount and stores it differently than other collectibles

- Add `ifcoins` logic command to match trinkets

- Add editor tool
Copy link
Contributor

@InfoTeddy InfoTeddy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style pass.

desktop_version/src/Render.cpp Outdated Show resolved Hide resolved
desktop_version/src/Render.cpp Outdated Show resolved Hide resolved
desktop_version/src/Render.cpp Outdated Show resolved Hide resolved
desktop_version/src/Render.cpp Outdated Show resolved Hide resolved
desktop_version/src/Game.cpp Outdated Show resolved Hide resolved
desktop_version/src/Game.cpp Outdated Show resolved Hide resolved
Co-authored-by: Misa Elizabeth Kai <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants