libPhoneNumber-iOS icon indicating copy to clipboard operation
libPhoneNumber-iOS copied to clipboard

Xcode 14.3 Analyzer Warnings Related To Assumed Nullability

Open LowAmmo opened this issue 2 years ago • 1 comments

When running Xcode analyzer on our project (and directly on the libPhoneNumber-iOS project), seeing some warnings about passing nil to methods that are expecting it, or returning nil from methods that don't support returning nil.

Working under an assumption that the current code is doing what is desired, I believe the majority of the "issues" are just due to assumed Nullability with methods.

I'm working on a PR that adds nullability specifiers primarily in NBPhoneNumberUtil, but also a few other places that gave warnings.

Everything seems to be benign, so not sure there is actually a problem - but, my OCD appreciates when code analysis doesn't produce any warnings at all. 😄

LowAmmo avatar May 12 '23 17:05 LowAmmo

Created Pull Request: https://github.com/iziz/libPhoneNumber-iOS/pull/392

  • Analyzer returns no issues with any targets (base, Geocoding, or ShortNumber)
  • All tests targets pass successfully

Everything was tested/developed with Xcode 14.3

LowAmmo avatar May 12 '23 18:05 LowAmmo