diff --git a/mopidy/mpd/protocol/current_playlist.py b/mopidy/mpd/protocol/current_playlist.py index e50c1bff..45858f86 100644 --- a/mopidy/mpd/protocol/current_playlist.py +++ b/mopidy/mpd/protocol/current_playlist.py @@ -1,5 +1,7 @@ from __future__ import unicode_literals +import warnings + from mopidy.mpd import exceptions, protocol, translator @@ -160,7 +162,8 @@ def playlist(context): Do not use this, instead use ``playlistinfo``. """ - # TODO: warn about this being deprecated + warnings.warn( + 'Do not use this, instead use playlistinfo', DeprecationWarning) return playlistinfo(context) diff --git a/mopidy/mpd/protocol/playback.py b/mopidy/mpd/protocol/playback.py index d18aff50..c550a33a 100644 --- a/mopidy/mpd/protocol/playback.py +++ b/mopidy/mpd/protocol/playback.py @@ -1,5 +1,7 @@ from __future__ import unicode_literals +import warnings + from mopidy.core import PlaybackState from mopidy.mpd import exceptions, protocol @@ -102,7 +104,10 @@ def pause(context, state=None): - Calls ``pause`` without any arguments to toogle pause. """ if state is None: - # TODO: emit warning about this being deprecated + warnings.warn( + 'The use of pause command w/o the PAUSE argument is deprecated.', + DeprecationWarning) + if (context.core.playback.state.get() == PlaybackState.PLAYING): context.core.playback.pause() elif (context.core.playback.state.get() == PlaybackState.PAUSED): @@ -339,10 +344,10 @@ def seekcur(context, time): """ if time.startswith(('+', '-')): position = context.core.playback.time_position.get() - position += int(time) * 1000 + position += protocol.INT(time) * 1000 context.core.playback.seek(position).get() else: - position = int(time) * 1000 + position = protocol.UINT(time) * 1000 context.core.playback.seek(position).get()