Make MediaPlayerList iteratable instead of providing the "players" property

1 files changed, 19 insertions, 3 deletions

diff --git a/src/media-player-list.vala b/src/media-player-list.vala
index 73f21ce..893ec64 100644
--- a/src/media-player-list.vala
+++ b/src/media-player-list.vala
@@ -31,10 +31,26 @@ public class MediaPlayerList {
 		this.mpris_watcher.client_disappeared.connect (this.player_disappeared);
 	}
 
-	public List<MediaPlayer> players {
-		owned get {
-			return this._players.get_values ();
+	/* only valid while the list is not changed */
+	public class Iterator {
+		HashTableIter<string, MediaPlayer> iter;
+
+		public Iterator (MediaPlayerList list) {
+			this.iter = HashTableIter<string, MediaPlayer> (list._players);
 		}
+
+		public MediaPlayer? next_value () {
+			MediaPlayer? player;
+
+			if (this.iter.next (null, out player))
+				return player;
+			else
+				return null;
+		}
+	}
+
+	public Iterator iterator () {
+		return new Iterator (this);
 	}
 
 	/**