monome / aleph

open source sound computer
Other
80 stars 39 forks source link

op_grid: changing 'focus' before grid attached causes crash #206

Closed trentgill closed 10 years ago

trentgill commented 10 years ago

with no grid attached: -start with a blank scene -add op_grid -use hotkey to jump to inputs page -> freeze

else: -start with a blank scene -add op_grid -scroll to inputs page -select focus -turn ENC1/3 -> freeze

this issue does not occur with op_step or op_life

this is also apparent when a grid is attached at startup but not recognised in the boot process (tested with mext & series devices).

catfact commented 10 years ago

it is surely the naive call to net_monome_grid_clear() , which was recently added. it should check that something is actually connected

bees/src/net_monome.c : net_monome_grid_clear()

probably need to re-add the connection flag boolean in the bees handler for monomeConnect / monomeDisconnect events.

catfact commented 10 years ago

possibly fixed with commit e4bae4268391dd8d2aa37f9a16e866d924c0a36c . will test soon unless someone beats me to it.

tehn commented 10 years ago

fixed