Skip to content

Play library: implement repetition and halt commands#1091

Open
oskostenko wants to merge 1 commit into
boriel-basic:mainfrom
oskostenko:play-lib-update
Open

Play library: implement repetition and halt commands#1091
oskostenko wants to merge 1 commit into
boriel-basic:mainfrom
oskostenko:play-lib-update

Conversation

@oskostenko

Copy link
Copy Markdown
Contributor

This adds support for repetition () and halt H commands to the Play routine.

@wilco2009, you might probably want to incorporate these changes to your zx81sd port.

@wilco2009

Copy link
Copy Markdown
Contributor

This adds support for repetition () and halt H commands to the Play routine.

@wilco2009, you might probably want to incorporate these changes to your zx81sd port.

Perfect, thank you. I will implement your new features in the next version.

@oskostenko

Copy link
Copy Markdown
Contributor Author

@wilco2009, would it be possible to maintain a single version of the library, with conditional macros?

Something like:

#ifdef ARCH_ZX81SD
  ' zx81sd-specific const values
#elif
  ' default const values
#endif

That way it would be much less pain to maintain multiple architectures.

@wilco2009

Copy link
Copy Markdown
Contributor

I agree with you, that's the correct way to do it.
If the example is adapted that way, I could remove it from the architecture subfolder since it would simply work without needing to port anything.
I'll do it that way in my next version.

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