Skip to content

Restore CLRegion.contains handling for non-circular regions on macOS/watchOS#62

Merged
SoundBlaster merged 1 commit intomainfrom
claude/restore-clregion-contains-01DFzFd11buq2gs1n35ha8DG
Nov 15, 2025
Merged

Restore CLRegion.contains handling for non-circular regions on macOS/watchOS#62
SoundBlaster merged 1 commit intomainfrom
claude/restore-clregion-contains-01DFzFd11buq2gs1n35ha8DG

Conversation

@SoundBlaster
Copy link
Owner

The previous change removed CLRegion.contains(_:) usage entirely, causing
non-circular regions (beacon regions, custom regions) to always return false.
While this method is deprecated, it remains functional on macOS and watchOS.

This fix:

  • Restores CLRegion.contains(_:) for non-circular regions on macOS/watchOS
  • Keeps manual distance calculation for circular regions (avoids deprecation)
  • Maintains iOS behavior where non-circular regions return false
  • Documents the platform-specific handling clearly

Fixes regression where location specifications with beacon and custom regions
stopped working on macOS and watchOS platforms.

…watchOS

The previous change removed CLRegion.contains(_:) usage entirely, causing
non-circular regions (beacon regions, custom regions) to always return false.
While this method is deprecated, it remains functional on macOS and watchOS.

This fix:
- Restores CLRegion.contains(_:) for non-circular regions on macOS/watchOS
- Keeps manual distance calculation for circular regions (avoids deprecation)
- Maintains iOS behavior where non-circular regions return false
- Documents the platform-specific handling clearly

Fixes regression where location specifications with beacon and custom regions
stopped working on macOS and watchOS platforms.
@SoundBlaster SoundBlaster merged commit b8e966e into main Nov 15, 2025
3 checks passed
@SoundBlaster SoundBlaster deleted the claude/restore-clregion-contains-01DFzFd11buq2gs1n35ha8DG branch November 15, 2025 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants