Warning that in Swift 6 will be an error

[faabaliamacias]

Warning that in Swift 6 will be an error

[gongzhang]

Swift 5.7 does allow Any value to store a wildcard nil. Therefore the case nil is already useless in Swift 5.7. I think wrap the case in #if swift(<5.7) is a solution.

[HangyuYe]

With same warning

[smixik]

Same here

[gongzhang]

Same here. I think we can just delete this case, right?

[santirodriguezaffonso]

"Swift 6" on warning? The latest version is 5.7 🤥
https://www.swift.org/download/

Currently I'm using Xcode 14, pod --version (1.11.3) & SwiftyJSON (5.0.0) and it works fine.

[owenDecorus]

"Swift 6" on warning? The latest version is 5.7 🤥 https://www.swift.org/download/

No lying going on here. What this error means is that the issue will become a build error, and not just a warning, when we inevitably get Swift 6.

To add to the issue, I am seeing this as well. It's not really an issue right now obviously as everything is still working fine but, for whatever reason, we won't be able to build in Swift 6 because of this.

[luoxiu]

Actually, it allows:

let n: Int? = nil
let x: Any = n as Any

Swift 5.7 does allow Any value to store a wildcard nil. Therefore the case nil is already useless in Swift 5.7. I think wrap the case in #if swift(<5.7) is a solution.

[gongzhang]

Indeed you can assign nil to Any, sorry I was wrong.

👍 After further research, even if nil is assigned explicitly, it is not equal to a naked nil. (Because the value is Optional<Int>.none)

let a: Int? = nil
let b: Any = a as Any

print(b) // nil
print(type(of: b)) // Optional<Int>

if case nil = b { // false
    // ❌ never
}

if case Optional<Int>.none = b { // true
    // ✅ matched
}

Actually, it allows:

let n: Int? = nil
let x: Any = n as Any

Swift 5.7 does allow `Any` value to store a wildcard `nil`. Therefore the `case nil` is already useless in Swift 5.7. I think wrap the case in `#if swift(<5.7)` is a solution.

[luoxiu]

Changing case nil: to case Optional<Any>.none: will fix this issue, already raised a pr here: #1124.

[gongzhang]

@luoxiu 🤔 Is it really possible to get into the branch of Optional<Any>.none? I still feel that case nil should be removed directly.

[luoxiu]

@luoxiu 🤔 Is it really possible to get into the branch of Optional<Any>.none? I still feel that case nil should be removed directly.

Yes, take a look at the test I added to pr.

[gongzhang]

@luoxiu hmmm... looks good to me

[HangyuYe]

We need to wait for a new swift package to fix the issue, or is there some workaround to skip this warning on the current swift pak.