Eliminate while True: by querying the feed count if max_results is None (caller is requesting all data).

Author: lcmcninch

Adafruit_IO/client.py

@@ -251,14 +251,17 @@ def data(self, feed, data_id=None, max_results=API_PAGE_LIMIT):
         :param int max_results: The maximum number of results to return. To
             return all data, set to None.
         """
+        if max_results is None:
+            res = self._get(f'feeds/{feed}/details')
+            max_results = res['details']['data']['count']
         if data_id:
             path = "feeds/{0}/data/{1}".format(feed, data_id)
             return Data.from_dict(self._get(path))
 
         params = {'limit': max_results} if max_results else None
         data = []
         path = "feeds/{0}/data".format(feed)
-        while True:
+        while len(data) < max_results:
             data.extend(list(map(Data.from_dict, self._get(path,
                                                            params=params))))
             nlink = self.get_next_link()
@@ -267,8 +270,6 @@ def data(self, feed, data_id=None, max_results=API_PAGE_LIMIT):
             # Parse the link for the query parameters
             params = parse_qs(urlparse(nlink).query)
             if max_results:
-                if len(data) >= max_results:
-                    break
                 params['limit'] = max_results - len(data)
         return data