mpclient: change tagtypes from list to a set

it's more logical data structure for it and it supports some additional
useful stuff, like intersection.

1 files changed, 5 insertions, 4 deletions

diff --git a/nephilim/mpclient.py b/nephilim/mpclient.py
index 4973409..afd0ebb 100644
--- a/nephilim/mpclient.py
+++ b/nephilim/mpclient.py
@@ -106,7 +106,7 @@ class MPClient(QtCore.QObject):
     # these don't change while we are connected
     """A list of AudioOutputs available."""
     outputs     = None
-    """A list of supported tags (valid indices for Song)."""
+    """A set of supported tags (valid indices for Song)."""
     tagtypes    = None
     """A list of supported URL handlers."""
     urlhandlers = None
@@ -160,7 +160,7 @@ class MPClient(QtCore.QObject):
 
         self.outputs     = []
         self.urlhandlers = []
-        self.tagtypes    = []
+        self.tagtypes    = set()
     def __str__(self):
         return self._logger.name
 
@@ -440,7 +440,8 @@ class MPClient(QtCore.QObject):
         Update a list of tag types.
         """
         self._logger.debug('Receiving tag types.')
-        self.tagtypes = list(self._parse_list(data)) + ['file']
+        self.tagtypes = set(self._parse_list(data))
+        self.tagtypes.add('file')
     def _parse_urlhandlers(self, data):
         """
         Update a list of URL handlers and finish connection.
@@ -470,7 +471,7 @@ class MPClient(QtCore.QObject):
         self._logger.info('Disconnected from MPD.')
         self._commands   = []
         self.outputs     = {}
-        self.tagtypes    = []
+        self.tagtypes    = set()
         self.urlhandlers = []
 
         self._mpd_changed()