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

Add get/set pixel routines #476

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

MaxGraey
Copy link
Contributor

Currently, only for AssemblyScript. But I can also add for everyone else.
@aduros wdyt?

@aduros
Copy link
Owner

aduros commented Jul 23, 2022

I'm feeling like we should keep default API stubs very minimal. Things like pixel drawing, polygon drawing, etc. would be better to go in 3rd party libraries that can be maintained by other users as they see fit.

@MaxGraey
Copy link
Contributor Author

set / get pixel is quite basic and useful operation which is pretty tricky in wasm4 due to packing. And users may easily do it in suboptimal way. See example: f128495#diff-222620e907510b8af2a9e5b17a92c840a445fefa24f760b26babd640fa7eb6f8R745

@haviital
Copy link

I'm feeling like we should keep default API stubs very minimal. Things like pixel drawing, polygon drawing, etc. would be better to go in 3rd party libraries that can be maintained by other users as they see fit.

Would that 3rd party library eat the fixed 64 kb memory area reserved for the game etc.?

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.

3 participants