Memory allocation problem about simplefsm HOT 3 CLOSED

By the way, I ran some tests using sizeOf. A state should occupy 32 bytes, while a transition occupies 40. Given this, at the moment my states occupy 2400 bytes, while my transitions occupy 8040 bytes, for a total of 10440. My chip should have ~90KiB of memory for instructions, so I should be well within that limit.
Moreover, I checked https://github.com/jonblack/arduino-fsm/ to which this library is inspired: In https://github.com/jonblack/arduino-fsm/blob/master/Fsm.cpp the way to add transitions is incremental (it doesn't recurse over an array, but adds transitions one by one at user request). In particular, immediately before adding a transition, memory is reallocated by giving:

 m_transitions = (Transition*) realloc(m_transitions, (m_num_transitions + 1) * sizeof(Transition));

I tried to do the same in the function I defined in the comment above, adding the line:

transitions = (Transition*) realloc(transitions, (num_standard + size) * sizeof(Transition));

just before the for cycle. This has the only effect of putting my chip in a cycle of crashes.

from simplefsm.

Ok, solved it. Created a pull request in case someone is interested: #7

from simplefsm.

Hey Fabrizio,
thank you for your issue report and for your pull request!
Will review your code and merge it.
🙇

from simplefsm.