Minor clean-up

Author: oschwald

geoip2/database.py

@@ -100,7 +100,7 @@ def _model_for(self, model_class, ip_address):
         record = self.db_reader.get(ip_address)
         if record is None:
             raise geoip2.errors.AddressNotFoundError(
-                "The address %s is not in the database." % (ip_address))
+                "The address %s is not in the database." % ip_address)
         record.setdefault('traits', {})['ip_address'] = ip_address
         return model_class(record, locales=self.locales)
 

geoip2/records.py

@@ -433,6 +433,9 @@ def __new__(cls, locales, *subdivisions):
         obj._locales = locales
         return obj
 
+    def __init__(self, locales, *subdivisions):
+        super(Subdivisions, self).__init__()
+
     @property
     def most_specific(self):
         """The most specific (smallest) subdivision available.

geoip2/webservice.py

@@ -93,7 +93,7 @@ def __init__(self, user_id, license_key, host='geoip.maxmind.com',
         self.locales = locales
         self.user_id = user_id
         self.license_key = license_key
-        self._base_uri = 'https://%s/geoip/v2.0' % (host)
+        self._base_uri = 'https://%s/geoip/v2.0' % host
 
     def city(self, ip_address='me'):
         """This method calls the GeoIP2 Precision City endpoint.
@@ -152,7 +152,7 @@ def _response_for(self, path, model_class, ip_address):
         response = requests.get(uri, auth=(self.user_id, self.license_key),
                                 headers={'Accept': 'application/json',
                                          'User-Agent': self._user_agent()})
-        if (response.status_code == 200):  # pylint:disable=E1103
+        if response.status_code == 200:  # pylint:disable=E1103
             body = self._handle_success(response, uri)
             return model_class(body, locales=self.locales)
         else:
@@ -174,9 +174,9 @@ def _handle_success(self, response, uri):
     def _handle_error(self, response, uri):
         status = response.status_code
 
-        if status >= 400 and status < 499:
+        if 400 <= status < 499:
             self._handle_4xx_status(response, status, uri)
-        elif status >= 500 and status < 599:
+        elif 500 <= status < 599:
             self._handle_5xx_status(status, uri)
         else:
             self._handle_non_200_status(status, uri)