Framebuffer refactorings

[MaxGraey]

No description provided.

[netlify]

✔️ Deploy Preview for wasm4 ready!

🔨 Explore the source changes: ac6caa8

🔍 Inspect the deploy log: https://app.netlify.com/sites/wasm4/deploys/6140666546ba030007ad9649

😎 Browse the preview: https://deploy-preview-79--wasm4.netlify.app

[eXponenta]

Fixing a #11 will be nice addition for this PR

[aduros]

Looking good to me, thanks for the needed cleanup!

I wonder about the inlining of drawPoint, is it possible to do it in a way that doesn't sacrifice readability too much?

[MaxGraey]

is it possible to do it in a way that doesn't sacrifice readability too much?

Basic idea is manually hoist all invariants out of loop, not just inline drawPoint. Basically browser should do this by itself but it's not always happening. I could just revert this inlines back if readability is more preffered

[aduros]

Ok, let's revert the inlining for readability for now. I'd like to do the C port, fix bugs (#11), and add missing functionality (#13) before we go too far with optimization.

[MaxGraey]

Done!

[aduros]

Thanks!