Closed folkertvanheusden closed 3 years ago
Dug a bit into it and:
In src/AppleMIDI.hpp there's (line 27):
template <class UdpClass, class Settings, class Platform>
void AppleMIDISession<UdpClass, Settings, Platform>::parseControlPackets()
{
while (controlBuffer.size() > 0)
{
auto retVal = _appleMIDIParser.parse(controlBuffer, amPortType::Control);
if (retVal == parserReturn::UnexpectedData)
But 'parse' can also return parserReturn::NotSureGiveMeMoreData (and others) causing the loop to loop forever which then triggers the wdt. So I suggest parserReturn::Processed to be checked against there.
It doesn't explain why it fails to communicate with https://github.com/davidmoreno/rtpmidid though.
additional test to avoid infinite loop (added in master branch, tested on ESP32)
@folkertvanheusden does this resolve this issue?
@folkertvanheusden thank you reporting and proactive digging into the issue
The one from the branch I responded last on worked fine. Haven't tested the merged version yet.
On Sat, Aug 21, 2021 at 9:03 PM lathoub @.***> wrote:
additional test to avoid infinite loop (added in master branch, tested on ESP32)
@folkertvanheusden https://github.com/folkertvanheusden does this resolve this issue?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/lathoub/Arduino-AppleMIDI-Library/issues/127#issuecomment-903161960, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUN5IW6BIFCFXDJWFFV2BATT57Z6DANCNFSM5BOGLDCQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .
The one from the branch I responded last on worked fine. Haven't tested the merged version yet.
That did not have the fix yet :-)
The master? Or the branch? The branch works fine with long- and short names.
On Sat, Aug 21, 2021 at 9:09 PM lathoub @.***> wrote:
The one from the branch I responded last on worked fine. Haven't tested the merged version yet.
That did not have the fix yet :-)
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.
Only 1 branch now: master Master now contains a fix for the above issue (session names already solved and folded in master earlier today)
I did another fix: the session name of the data invitation was incorrect: it send back the remote session name, rather than the local name.
Maybe that addresses the below issue:
It doesn't explain why it fails to communicate with https://github.com/davidmoreno/rtpmidid though.
Hi,
I believe it may need a few calls to yield() here and there. I took the basic ESP8266_NoteOnOffEverySec.ino, added mDNS and as soon as something connects to it, the watchdog gets triggered:
esp8266_test.zip