alsa-project / alsa-utils

The Advanced Linux Sound Architecture (ALSA) - utilities
GNU General Public License v2.0
197 stars 136 forks source link

aplay: Print '=== PAUSE ===' only if it is supported #271

Closed ujfalusi closed 2 months ago

ujfalusi commented 2 months ago

Instead of printing the '=== PAUSE ===' unconditionally before calling do_pause(), move it to the function and only print it if the stream can be paused.

If the stream cannot be paused that the '=== PAUSE ===' will be replaced by PAUSE command ignored (no hw support) immediately, which is not observable by users but automation scripts will catch the '=== PAUSE ===' and might think that the stream is indeed got paused.

Move the print into do_pause() function after the snd_pcm_pause() have returned without error to make sure it is only printed if the stream is paused and we are waiting for the pause release from user to proceed.