2.6.x: playdate.sound.sequence:allNotesOff() triggers PDSynthInstrument error in simulator

SDK Bug Reports
, ,
1

On Windows and Linux, with SDK versions 2.6.1 and 2.6.2, calling the Lua function playdate.sound.sequence:allNotesOff() causes the simulator to pause and print the following error, twice:

SynthInstrument_allNotesOff: non-NULL value required for argument 'inst'

which I assume comes from the C instrument layer. It's then possible to unpause the simulator and continue (unclear if this causes other issues).

Reproducing this is as trivial as:

local seq = playdate.sound.sequence.new("test.mid")
seq:allNotesOff()

This isn't observed on device, I'm guessing since warnings are disabled and not being a full-on crash this gets bypassed.

2

@dave since you asked me to @ you :sweat_smile: this may be a less serious one but still looks unintended.

3

Thanks! I put a fix in, forgot to note it here. :slight_smile:

1 Like
4

This still seems present in 2.7.4 :frowning:

5

Ugh, sorry! Looks like I forgot to push the MR for the fix before it got buried under all the 2.7 (and then 3.0) work. It’s in the system for 3.0.1, finally. :person_bowing: Huge thanks for the reminder!

1 Like
6

Confirmed fixed on 3.0.1! Marking resolved. Thanks!

1 Like
7

Thanks for reporting it and, as always, thank you for your patience!

1 Like