laamaa / m8c

Cross-platform M8 tracker headless client
Other
380 stars 81 forks source link

Read all available serial data each loop. #61

Closed jasonporritt closed 2 years ago

jasonporritt commented 2 years ago

Reads all available serial data at once instead of one read per run loop execution.

With these changes, m8c runs flawlessly on an rg351mp handheld device and my i9 Mac.

laamaa commented 2 years ago

Thanks for the PR. This seems to break the fx palette rendering on my pc so I won't merge just yet, need to think about a way to fix that.

jasonporritt commented 2 years ago

Interesting. I'll have to dig out one of my old PCs and check it out.

laamaa commented 2 years ago

After some more testing, it didn't occur so much that it would bother me. Probably better this way than having poor performance on a mac and some handhelds. No idea how this will affect pi though, but I guess someone will tell :D