diff options
author | Jason Wray <jason@kulturny.com> | 2015-05-20 12:14:50 -0700 |
---|---|---|
committer | John Firebaugh <john.firebaugh@gmail.com> | 2015-06-25 16:02:55 -0700 |
commit | f5c1e2630e553a727a91f0de623bf3ec1bdcc559 (patch) | |
tree | c9b09d02de64dc0a673a71278bbce3acd47e7446 | |
parent | 64c05d02e294be4c2ab6ef239964630325adf177 (diff) | |
download | qtlocation-mapboxgl-f5c1e2630e553a727a91f0de623bf3ec1bdcc559.tar.gz |
Remove redundant didChangeAuthorizationStatus: in favor of didFailWithError:
Listening to both `locationManager:didChangeAuthorizationStatus:` and `locationManager:didFailWithError:` delegate methods is unnecessary. Setting `showsUserLocation = NO` in the first means that the second never gets called and our `mapView:didFailToLocateUserWithError:` delegate method never gets called.
-rw-r--r-- | platform/ios/MGLMapView.mm | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/platform/ios/MGLMapView.mm b/platform/ios/MGLMapView.mm index cf2274e2b0..4c014d12cf 100644 --- a/platform/ios/MGLMapView.mm +++ b/platform/ios/MGLMapView.mm @@ -2254,15 +2254,6 @@ CLLocationCoordinate2D MGLLocationCoordinate2DFromLatLng(mbgl::LatLng latLng) } } -- (void)locationManager:(__unused CLLocationManager *)manager didChangeAuthorizationStatus:(CLAuthorizationStatus)status -{ - if (status == kCLAuthorizationStatusDenied || status == kCLAuthorizationStatusRestricted) - { - self.userTrackingMode = MGLUserTrackingModeNone; - self.showsUserLocation = NO; - } -} - - (void)locationManager:(__unused CLLocationManager *)manager didFailWithError:(NSError *)error { if ([error code] == kCLErrorDenied) |