From 050da542b38b67b1c10043d01050d5766185d75d Mon Sep 17 00:00:00 2001
From: Matt Martz <matt@sivel.net>
Date: Mon, 16 May 2016 11:34:38 -0500
Subject: [PATCH] Remove unneeded var, clear servers on get_servers and break
 if we were successful collecting servers

---
 speedtest.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/speedtest.py b/speedtest.py
index 2fe595f..761c049 100755
--- a/speedtest.py
+++ b/speedtest.py
@@ -755,6 +755,7 @@ class Speedtest(object):
         """Retrieve a the list of speedtest.net servers, optionally filtered
         to servers matching those specified in the ``servers`` argument
         """
+        self.servers.clear()
 
         for i, s in enumerate(servers):
             try:
@@ -834,6 +835,8 @@ class Speedtest(object):
                 del serversxml
                 del elements
 
+                break
+
             except ServersRetrievalError:
                 continue
 
@@ -1239,14 +1242,12 @@ def shell():
             print_('Cannot retrieve speedtest server list')
             sys.exit(1)
 
-        server_list = []
         for _, servers in sorted(speedtest.servers.items()):
             for server in servers:
                 line = ('%(id)5s) %(sponsor)s (%(name)s, %(country)s) '
                         '[%(d)0.2f km]' % server)
                 try:
                     print_(line)
-                    server_list.append(line)
                 except IOError:
                     e = sys.exc_info()[1]
                     if e.errno != errno.EPIPE: