-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Version 1.5.1 #2812
Merged
Merged
Version 1.5.1 #2812
Conversation
This file contains 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
#2710) * User identityHashCode instead of deprecated `Kotlin_Any_hashCode` for Native debugging capabilities
Co-authored-by: Pierfrancesco Soffritti <psoffritti@google.com>
… ExecutorService.asCoroutineDispatcher (#2727) * Get rid of ThreadPoolDispatcher and PoolThread classes * Reuse the same class for both asCoroutineDispatcher and newFixedThreadPoolContext * Replace 3-classes hierarchy by a single impl class * Copy the auto-closing logic to test source * Document and tweak the contract of Executor.asCoroutineDispatcher and ExecutorService.asCoroutineDispatcher * Document it properly * Make it more robust to signature changes and/or delegation (e.g. see the implementation of java.util.concurrent.Executors.newScheduledThreadPool) * Give a public way to reduce the memory pressure via ScheduledFuture.cancel Fixes #2601
#2731) Prevent StackOverflowError in CompletableFuture.asDeferred and properly report exceptions from completion handlers * It turned out that 'cancel' on completed future tries to help and invoke 'whenComplete' handlers that also invoke 'cancel' on the very same future * Use top-level exception handler as a last resort to deliver an exception Fixes #2730
That helps to pro-actively catch cases like #2717 and to report such exception in even more robust manner
…ect classpath (#2741) Otherwise, the dependency from Kotlin plugin to coroutines 1.4.3 (plugin -> compiler.jar -> coroutines) leaks into the runtime classpath
* Improve ':kotlinx-coroutines-core' substitution for all modules. Ensure "org.jetbrains.kotlinx:kotlinx-coroutines-core" transitive dependencies are substituted with project itself to avoid test errors. Co-authored-by: Yahor Berdnikau <egorr.berd@gmail.com>
* ICA is misused here as lint, for which we consistently use @deprecated in other places * The current state is incompatible with 1.5.30 where KT-45844 is implemented
* Update supervisorScope documentation to avoid confusion Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
…age identity in debugging.md Addresses #1931
This implementation note mentions that withContext(IO) does not lead to a context switch, which may be misunderstood as a general statement, while it in fact only applies to switches between Default and IO dispatchers. This is one example of misunderstanding: https://stackoverflow.com/questions/68069529/is-kotlinx-coroutines-withcontext-safe-to-use-with-spring-webflux
…uation. (#2773) Otherwise compilation fails on 1.5.30 due to KT-45844
* Make releaseInterceptedContinuation final * CoroutineDispatcher is internal for implementation and always has been * Overriding this method may lead to memory leaks in parent coroutines and is considered dangerous
#2772) * Properly detect non-released reusable continuations in non-reusable ones and await for reusability to have a consistent state * Ensure that the caller to DispatchedContinuation.isReusable is reusable itself * Using the previous invariant, simplify DispatchedContinuation.isReusable to a single null-check * It also restores the invariant that `cc.isReusable() == cc.resumeMode.isReusableMode` Fixes #2736 Fixes #2768
* Migrate to Dokka pre-1.5.0 Co-authored-by: Kamil Doległo <kamilok1965@interia.pl> Co-authored-by: dkhalanskyjb <52952525+dkhalanskyjb@users.noreply.github.com>
* Suppress all deprecated signatures from the documentation * Migrate inline classes to value * Get rid of annoying Gradle output when tasks are configured * Opt-in into delicate coroutines API * Get rid of legacy jvm_ir_enabled that was used for pre-release testing of JVM IR compiler * Update README references
dkhalanskyjb
reviewed
Jul 9, 2021
dkhalanskyjb
approved these changes
Jul 9, 2021
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.
No description provided.