averello/swift-mirror
1
0
Fork 0
mirror of https://github.com/apple/swift.git synced 2025-12-21 12:14:44 +01:00
Code Issues Packages Projects Releases Wiki Activity

[Reflection] Handle -disable-reflection-names in _forEachField.

Browse Source 
Invoke our callback with an empty name string when reflection names are disabled, rather than throwing a fatal error.

rdar://108709009
This commit is contained in:
Mike Ash
2023-05-01 17:03:54 -04:00
parent 3368e710a2
commit a6bab7332d
2 changed files with 38 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
test/stdlib/ForEachField.swift
View File

@@ -11,6 +11,7 @@
//===----------------------------------------------------------------------===//
// RUN: %target-run-simple-swift
// RUN: %target-run-simple-swift(-Xfrontend -disable-reflection-names -DNO_FIELD_NAMES)
// REQUIRES: executable_test
// REQUIRES: reflection
@@ -150,6 +151,14 @@ func checkFields<T>(
count += 1
let fieldName = String(cString: charPtr)
if fieldName == "" {
#if NO_FIELD_NAMES
expectTrue(fields.values.contains{ $0 == offset && $1 == type })
#else
expectTrue(false, "Empty field name")
#endif
return true
}
guard let (checkOffset, checkType) = fields[fieldName] else {
expectTrue(false, "Unexpected field '\(fieldName)'")
return true
@@ -176,6 +185,14 @@ func checkFieldsWithKeyPath<T>(
count += 1
let fieldName = String(cString: charPtr)
if fieldName == "" {
#if NO_FIELD_NAMES
expectTrue(fields.values.contains{ $0 == keyPath })
#else
expectTrue(false, "Empty field name")
#endif
return true
}
guard let checkKeyPath = fields[fieldName] else {
expectTrue(false, "Unexpected field '\(fieldName)'")
return true
@@ -450,8 +467,13 @@ if #available(SwiftStdlib 5.2, *) {
charPtr, _, type, _ in
let fieldName = String(cString: charPtr)
#if NO_FIELD_NAMES
return type == (Double, Double).self
&& fieldName == ""
#else
return type == (Double, Double).self
&& fieldName == "point"
#endif
})
expectTrue(_forEachField(of: EmptyNSObject.self, options: .classType) {