🍒 [5.10] Avoid issuing warning when user lib implements only ExecutorJob enqueue #68618
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The latter example NIODefaultSerialEventLoopExecutor when the type is more available than the enqueue implementation provided via an extrension would issue a warning that that this enqueue(ExecutorJob) would not be used which is not true. We need to filter out all "default impls" from the stdlib as we issue this warning. We also put a note on the offending declaration that one can remove now, to ease getting rid of warnings when possible Resolves rdar://115166475
ktoso
changed the title
|
@swift-ci please test |
hborla
approved these changes
Sep 20, 2023
ktoso
deleted the
pick-improve-will-not-be-called-warning-nio-sample
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description: The latter example NIODefaultSerialEventLoopExecutor when the type is more available than the enqueue implementation provided via an extrension would issue a warning that that this enqueue(ExecutorJob) would not be used which is not true.
We need to filter out all "default impls" from the stdlib as we issue this warning.
We also put a note on the offending declaration that one can remove now, to ease getting rid of warnings when possible.
Risk: Low, harmless warning diagnostic removal only
Impact: Only on macOS where availability and extensions are involved AFAICS and according to @FranzBusch testing and NIO ci.
Review by: @hborla
Testing: CI testing, NIO to verify with toolchain
Original PR: #68617
Radar: rdar://115166475