Super-Square-Bros icon indicating copy to clipboard operation
Super-Square-Bros copied to clipboard

Patch gadgetoid

Open Gadgetoid opened this issue 4 years ago • 3 comments

Adds PicoSystem support in the least invasive way I could muster. PicoAudio is on standby for when PicoSystem can do something vaguely audio-like for bleeps and bloops.

  • Basic setup for PicoSystem build and 120x120 screen resolution
  • Stub out Audio for Pico using PicoAudio.cpp/hpp
  • Exit some collision detection early for a significant speedup
  • Remove remaining references to backgound_image and clear() to the background colour only once
  • Crude speedhack for Pico - only update onscreen enemies - makes busier levels playable at the cost of consistency with other ports
  • Remove "Colour" and use aligned "Pen" instead

This is a sometimes-different, sometimes-the-same take on @Daft-Freak's diff here - https://github.com/ThePythonator/Super-Square-Bros/compare/main...Daft-Freak:pico-maybe

I also have a patch to change all iterators to the form for(auto &item : items) if you want it.

Gadgetoid avatar Aug 17 '21 22:08 Gadgetoid

Another speed patch: https://github.com/ThePythonator/Super-Square-Bros/commit/a9237384e408a97a77663bd106152e00ba3e066a (fixed-point to the rescue again). Got to the start of level 7 with that (on my branch, so no speedhack), testing further would require me to not die.

Daft-Freak avatar Aug 18 '21 11:08 Daft-Freak

With all the changes I've made, it's got a bit out of sync. The Colour -> Pen and support for PicoSystem are the main commits which I still need, is it possible for you to rebase and just PR those? I'm regretting everything being in one file because of all these merge conflicts 🤦‍♂️ (edit: just need Colour->Pen now)

ThePythonator avatar Aug 18 '21 18:08 ThePythonator

Most of the PicoSystem stuff is done, but audio stuff messes everything up a bit. Using one shared Audio.hpp header is probably the best.

ThePythonator avatar Aug 18 '21 18:08 ThePythonator