mpd: Fix review comments
This commit is contained in:
parent
14444fe24f
commit
b09e667127
@ -41,6 +41,7 @@ Bug fix release.
|
|||||||
|
|
||||||
- MPD: Add ``volume`` command that was reintroduced, though still as a
|
- MPD: Add ``volume`` command that was reintroduced, though still as a
|
||||||
deprecated command, in MPD 0.18 and is in use by some clients like mpc.
|
deprecated command, in MPD 0.18 and is in use by some clients like mpc.
|
||||||
|
(Fixes: :issue:`1393`, PR: :issue:`1397`)
|
||||||
|
|
||||||
- Proxy: Handle case where :confval:`proxy/port` is either missing from config
|
- Proxy: Handle case where :confval:`proxy/port` is either missing from config
|
||||||
or set to an empty string. (PR: :issue:`1371`)
|
or set to an empty string. (PR: :issue:`1371`)
|
||||||
|
|||||||
@ -440,7 +440,7 @@ def volume(context, change):
|
|||||||
Note: ``volume`` is deprecated, use ``setvol`` instead.
|
Note: ``volume`` is deprecated, use ``setvol`` instead.
|
||||||
"""
|
"""
|
||||||
if change < -100 or change > 100:
|
if change < -100 or change > 100:
|
||||||
raise exceptions.MpdArgError('Invalid volume value', command='volume')
|
raise exceptions.MpdArgError('Invalid volume value')
|
||||||
|
|
||||||
old_volume = context.core.mixer.get_volume().get()
|
old_volume = context.core.mixer.get_volume().get()
|
||||||
if old_volume is None:
|
if old_volume is None:
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user