Fix #29: Overflow on Starting with Minutes and Hours

[cornelius-koepp]

Noch UNGETESTET
Fix für #29

[philthepunk85]

Danke fürs schnelle Aufgreifen — die Ursache passt exakt zu den Messungen aus #29.

Zur Bestätigung der Mechanik: Der Überlauf entsteht durch das Compound Assignment. value *= 60 rechnet zwar in int (integer promotion), konvertiert das Ergebnis aber wieder nach uint8_t zurück ⇒ effektive Laufzeit = (Wert × 60) mod 256 Sekunden. Das deckt sich 1:1 mit allen Messpunkten aus dem Issue, auch mit denen, die dort zunächst unerklärlich wirkten.

Zwei Ergänzungen, die für die Release Notes relevant sein könnten:

1. Bei „Einheit Stunden" war die Funktion vollständig unbrauchbar, nicht nur oberhalb einer Grenze.
Wegen 3600 mod 256 = 16 ergaben sich konstant 16 Sekunden je Stunde: 1 h → 16 s, 2 h → 32 s, 3 h → 48 s. Anders als bei „Einheit Minuten" (dort waren Werte ≤ 4 korrekt) gab es im Stundenmodus keinen einzigen gültigen Wert.

2. Der Fix wirkt vollständig.
Ich habe den zweiten zeitverarbeitenden Pfad, case FCB_KoCHTimeOffset, gegengeprüft: Dort wird uint16_t mit festen Konstanten bis 3600 verwendet, ohne Multiplikation — kein Überlauf möglich. Weitere betroffene Stellen sehe ich nicht.