Remove recursive callback in ATParser::vrecv

ARMmbed / ATParser

Parser for AT commands and similar protocols.

36 stars 27 forks source link

Remove recursive callback in ATParser::vrecv #5

Closed kegilbert closed 7 years ago

kegilbert commented 7 years ago

Recursive call in vrecv was causing the main stack to overflow in the WiFi example. Main stack usage went from {4264, 4096} to {2128,4096} with this change. Tested on GCC_ARM with K64F - ESP8266.

geky commented 7 years ago

Looks good, another user made a comment suggesting this fix, but I don't remember where that comment lives. Hopefully they'll be happy this landed anyways. 👍