Use ARRAY_LEN instead of sizeof to detect end of fixed tone pattern The two are equivalent for uint8_t, but break if the type changes to something larger. Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
1 files changed, 1 insertions(+), 1 deletions(-) M main.c
M main.c +1 -1
@@ 179,7 179,7 @@ static inline uint8_t first_fixed_tone(v static inline uint8_t next_fixed_tone(uint8_t tone) { tone++; - if (tone == sizeof(tones_fixed)) + if (tone == ARRAY_LEN(tones_fixed)) tone = 0; return tone;