cdemoulins/pamixer

--get-volume and --get-volume-human return non-zero

mfiano opened this issue · 4 comments

The --get-volume and --get-volume-human switches return 0 only if volume is not 0 and not muted. If they are it exits with 1 (while still returning the string to stdout).

Sorry but what is the issue with this ?

If you are using this in a script to retrieve volume values, I'd expect it to return a non-error exit status when it displays the expected value you requested, unless there was an actual error in the process.

I agree with @mfiano, I also find this behaviour unexpected and undesirable.

Ok, it's something i don't really care and added a long time ago (696aa78).

I will revert to the previous behavior : return 0