LabSound
LabSound copied to clipboard
LOG_PRINT should replace ASN_PRINT, printf, and std::iostream
This is probably from debugging: https://github.com/LabSound/LabSound/blob/v1.2.0/src/core/AudioNode.cpp#L20
A similar log is currently in main
:
https://github.com/LabSound/LabSound/blob/main/src/core/AudioNode.cpp#L19
Also, why not use LOG_TRACE
or some of those?
It's historic. ASN_PRINT is very old, and should have been moved when LOG_TRACE was introduced. I hadn't thought about that point.
Well that's currently named LOG_PLAYBACK_STATE_TRANSITION
. Switching to LOG_TRACE
makes sense.
Also when using loadHrtfDatabase
, I get a ton of logs from https://github.com/LabSound/LabSound/blob/main/src/extended/AudioFileReader.cpp#L85 - because printf
is used directly.
You've found enough inconsistencies to make this a specific task to clean up all logging and tracing ~ I renamed the issue to be more general.