CPU utilization improvements, minor enhancements, and other bug fixes - Githubissues

laamaa / m8c

Cross-platform M8 tracker headless client

Other

419 stars 87 forks source link

CPU utilization improvements, minor enhancements, and other bug fixes #63

Closed bloopclick closed 2 years ago

bloopclick commented 2 years ago

Enhance idle performance:

Modify oscilloscope draw command to suppress empty redraws

Improve general CPU usage:

Change to nonblocking calls to serial read
Moves main thread delay to an SDL_Delay() call
Adds an INI parameter to control SDL_Delay() argument / thread sleep time

(Moving to SDL_Delay may or may not work well on a wide range of platforms)

Fix bugs:

Fix buffer overflow in INI write (oops!)
Add assert to guard similar errors
Fix bug that failed to write last line of INI file (existing bug)
Fix segfault on reading empty INI file (existing bug)

Enhancements:

Add dirty flag to only render after draw commands (alternate solution to palette fx bug)
Add ability to make a debug build from the command line
Slightly optimize byte copies to SLIP parser

laamaa commented 2 years ago

Seems to work and the delay didn't seem to cause significant input lag either at least on Linux.. I'd say lets merge it see if anything comes up.