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

Fixed Bug Not Loading/Saving in adventure.dsk #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

prasys
Copy link

@prasys prasys commented May 22, 2021

Hi,

There is a bug in the original implementation of adventure whereby the original author made the assumption that all of the variables are stored consecutively but however this is not the case for FLEX. Therefore, in our implementation - we had to change the save and load functions so that it writes individual values into disk and then loads them back in.

Also Flex likes everything to be in Upper case. So I've fixed them as well

Two files were modified ADVENT.C and SAVEADV.C

The full source code can be seen here (with the modifications)

https://github.com/prasys/ADVENT-Flex

I am submitting this because I thought it would be helpful for people who want to play colossal cave adventure

Let me know should you require more details

Fixed the bug in saving and loading
Implementation is in

https://github.com/prasys/ADVENT-Flex
@prasys prasys changed the title Update adventure.dsk Fixed Bug Not Loading/Saving in adventure.dsk May 22, 2021
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.

1 participant