city41 / rotary-bobble

Puzzle Bobble with rotary controls
https://rotary-bobble.mattgreer.dev/
GNU General Public License v3.0
2 stars 0 forks source link

Input routine: use existing flow as much as possible #5

Closed city41 closed 7 months ago

city41 commented 7 months ago

The current hack hijacks the "update angle" routine. This is not good. It would be better to alter the entire flow as needed. For example the routine that determines shooter delta is where rotary input should be considered, then that delta should get passed along to existing routines. Existing routines should be altered as minimally as possible.

This has the advantage of input not needing to be changed (especially for A), and the dino will crank faster the larger the delta, which is a nice touch

city41 commented 7 months ago

This is going well. This approach has introduced one unexpected bug but also fixed several bad bugs. I think this is the way to go.

city41 commented 7 months ago

Doh, the bug was just me using a memory address the game was also using. I think single player rotary controls is ... done?

patch is singlePlayerRotaryControls.json in fc1d874bf9caca2cea774a14e059492e8cb4c706