-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Class X is implemented in both A and B. One of the two will be used. Which one is undefined. #4782
Closed
2 tasks done
Comments
+1, just discovered it with |
+1 - Seems to be Xcode 14.3 specific |
Closed
Seeing this with 14.3 as well |
This was referenced Apr 11, 2023
keith
added a commit
that referenced
this issue
Apr 17, 2023
This uses a recent but unannounced (also read as private) feature of dyld where it ignores duplicate Objective-C classes when they're in a special format in the binary. https://github.com/apple-oss-distributions/dyld/blob/c8a445f88f9fc1713db34674e79b00e30723e79d/dyld/PrebuiltLoader.cpp#L1660-L1662 I think this is generally safe because hopefully people aren't actually using the SwiftSyntax classes through the Objective-C runtime, but if they are we'd still probably prefer to silence the noise and accept the UB. Fixes #4782
🙈 #4901 |
keith
added a commit
that referenced
this issue
Apr 17, 2023
This uses a recent but unannounced (also read as private) feature of dyld where it ignores duplicate Objective-C classes when they're in a special format in the binary. https://github.com/apple-oss-distributions/dyld/blob/c8a445f88f9fc1713db34674e79b00e30723e79d/dyld/PrebuiltLoader.cpp#L1660-L1662 I think this is generally safe because hopefully people aren't actually using the SwiftSyntax classes through the Objective-C runtime, but if they are we'd still probably prefer to silence the noise and accept the UB. Fixes #4782
keith
added a commit
that referenced
this issue
Apr 17, 2023
This uses a recent but unannounced (also read as private) feature of dyld where it ignores duplicate Objective-C classes when they're in a special format in the binary. https://github.com/apple-oss-distributions/dyld/blob/c8a445f88f9fc1713db34674e79b00e30723e79d/dyld/PrebuiltLoader.cpp#L1660-L1662 I think this is generally safe because hopefully people aren't actually using the SwiftSyntax classes through the Objective-C runtime, but if they are we'd still probably prefer to silence the noise and accept the UB. Fixes #4782
jpsim
pushed a commit
that referenced
this issue
Apr 17, 2023
This uses a recent but unannounced (also read as private) feature of dyld where it ignores duplicate Objective-C classes when they're in a special format in the binary. https://github.com/apple-oss-distributions/dyld/blob/c8a445f88f9fc1713db34674e79b00e30723e79d/dyld/PrebuiltLoader.cpp#L1660-L1662 I think this is generally safe because hopefully people aren't actually using the SwiftSyntax classes through the Objective-C runtime, but if they are we'd still probably prefer to silence the noise and accept the UB. Fixes #4782
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
New Issue Checklist
Describe the bug
This will happen when you analyze the code with "swiftlint" or "swiftlint analyze". It will not break the build or give any crashes.
Complete output when running SwiftLint, including the stack trace and command used
Environment
SwiftLint version: 0.50.3
Installation method used: Homebrew
Paste your configuration file: none
Are you using nested configurations? -none
Which Xcode version are you using?
Xcode 14.3
Build version 14E5197f
Do you have a sample that shows the issue? Run
echo "[string here]" | swiftlint lint --no-cache --use-stdin --enable-all-rules
to quickly test if your example is really demonstrating the issue. If your example is more
complex, you can use
swiftlint lint --path [file here] --no-cache --enable-all-rules
.The text was updated successfully, but these errors were encountered: