Open AliceLR opened 3 years ago
Based on what some other projects do, I'd suggest:
#define
: reduced initialization (e.g. hard-coded null device), reduced / eliminated terminal setup, etc.xmp_load_module_from_file
), possibly slightly less bare-bones variant of the libfuzzer-based front-end I posted at https://github.com/libxmp/libxmp/issues/302 - maybe basic module information could still be displayed upon successfully loading the module.Anyone has a patch for this?
Not yet. I think the first one wouldn't be all that much clutter to xmp, and ideally it'd just suppress the player display portion of the terminal output (which is presumably the reason echo/etc is changed). The driver portion could just be handled with the driver flag, if needed (WAV output or playback might be desirable in some cases). It'd also be good to have a way to target any arbitrary one of the four xmp_load_module* functions.
edit: -q
suppresses the player portion but it also suppresses the other output (which might not matter much if libxmp is built with -DDEBUG
).
When xmp is terminated by AddressSanitizer, MemorySanitizer, etc. encountering an error and exiting, terminal echo is still turned off until manually fixed with
stty echo
,tput reset
, or similar. I don't know what the correct solution is for this, but I tried adding anatexit
hook to callreset_tty
and it didn't help. This has been mildly annoying when diagnosing issues like https://github.com/libxmp/libxmp/issues/319 with input files that cause a lot of ASan/MSan exits.