cw-generator/autoloads/morse_state.gd

extends Node

var states: Array[int] = []
var curr_state := false
var last_change: int = 0
var start_time := 0

func reset() -> void:
	last_change = Time.get_ticks_msec()
	start_time = last_change
	states = []

func set_state(state: bool) -> void:
	if state == curr_state:
		return
	curr_state = state
	
	var now := Time.get_ticks_msec()
	states.push_back(now - last_change)
	last_change = now
Move MorseState into autoload Before this every instance of the MorseBanner had its own state, but as we want to use the state also for other purposes, it doesn't make sense to keep it in each instance. Therefore we now have an autoload that keeps the state as a singleton. The MorseBanner now gets its state from the autoload, so no more communication is needed between the main part and the MorseBanner. 2025-02-02 18:36:32 +01:00			`extends Node`

			`var states: Array[int] = []`
			`var curr_state := false`
			`var last_change: int = 0`
			`var start_time := 0`

			`func reset() -> void:`
			`last_change = Time.get_ticks_msec()`
			`start_time = last_change`
			`states = []`

			`func set_state(state: bool) -> void:`
			`if state == curr_state:`
			`return`
			`curr_state = state`

			`var now := Time.get_ticks_msec()`
Switch direction of MorseState state array Originally it was easier for the MorseBanner to have the states with newest in front, but this makes the code harder to read and complicates things in other places. At some point it also might make sense to not use a range() in MorseBanner, but use a int var + counter instead, as we often might only consume a small number of elements. But as this is game programming, "winging it" and getting something done seems to be the modus operandi. 2025-02-03 01:49:46 +01:00			`states.push_back(now - last_change)`
Move MorseState into autoload Before this every instance of the MorseBanner had its own state, but as we want to use the state also for other purposes, it doesn't make sense to keep it in each instance. Therefore we now have an autoload that keeps the state as a singleton. The MorseBanner now gets its state from the autoload, so no more communication is needed between the main part and the MorseBanner. 2025-02-02 18:36:32 +01:00			`last_change = now`