confusing logging in audioconvert/audioadapter.c

In audioadapter.c we have the following code:

   482		spa_log_debug(this->log, "%p: configure format:", this);
   483		if (format && spa_log_level_enabled(this->log, SPA_LOG_LEVEL_DEBUG))
   484			spa_debug_format(0, NULL, format);

The line "configure format:" is printed via the logger and will go to stderr. The following format however is printed via spa_debug() which ends up in stdout.

Due to output buffering this means that there is no guarantee that the two statements are logged next to each other. In my case this debug log was the only thing printed to stdout and ended up as the last entry in the logfile, after all the other logs. Also there was no indication where this log came from.

It would probably be better to log this via a logger and/or make spa_debug() use stderr by default like the loggers.