On 09.08.2022 08:36, Atlantis wrote:
[...]
> Czyżby pozostałość po jakichś wcześniejszych wersjach kodu, gdzie
> faktycznie w parametrze był wskaźnik? A potem autor to przepisał, na
> wersję bezparametrową i zamiast dać void wyczyścił listę parametrów, nie
> poprawiając wcześniejszych wywołań?
Tak, na to to właśnie dla mnie wygląda. Widać to zresztą po niektórych
komentarzach w kodzie.

> MQTTReadPacket w tym miejscu odczytywało tylko dwa pierwsze bajty, co
> zgadzałoby się z wartością zwracaną przez funkcję.
Jak dla mnie to ta funkcja to jakieś totalne, niedorobione gówno.
Prowizorka znaczy się. Tutaj bym szukał błędu. Kody stanów w switch-u
zakodowane na twardo, a nie za pomocą enumów/makr - jak mniemam chodzi o
kody wymienione w enum-ie na początku. Stany zmieniane za pomocą
m_state++ czy m_state=3 zamiast m_state=NAZWA_KOLEJNEGO_STANU - aż się
prosi o kłopoty. Nie chce mi się analizować tego kodu, ale wydaje mi
się, że nie wszystkie stany są obsługiwane.