🌐 US-Proxy
class="logged-out env-production page-responsive" style="word-wrap: break-word;" >
Skip to content

4.x: Disposable no longer impls AutoCloseable - #8184

Merged
akarnokd merged 2 commits into
ReactiveX:4.xfrom
akarnokd:4xDisposableAutoCloseableChange
Jun 26, 2026
Merged

4.x: Disposable no longer impls AutoCloseable#8184
akarnokd merged 2 commits into
ReactiveX:4.xfrom
akarnokd:4xDisposableAutoCloseableChange

Conversation

@akarnokd

Copy link
Copy Markdown
Member

Making DIsposable directly AutoCloseable adds unnecessary resource warnings to most spaces. Disposable is mainly there to signal dispose/cancel, not specifically resource lifecycles.

The interface no longer extends AutoCloseable, all places where SuppressWarnings("resource") have been removed.

A new method Disposable.asAutoCloseable was added for the case when a try-with-resources is needed over it.

@github-actions

Copy link
Copy Markdown

🐷 TruffleHog + Entropy Beauty Scan

Average entropy of changed code: 4.558 bits/char
Verdict: ✅ Mid-4 beauty detected (thoughtful human code!)

Changed files entropy:

build.gradle: 5.004
docs/Creating-Observables.md: 5.159
docs/What's-different-in-2.0.md: 4.905
src/main/java/io/reactivex/rxjava4/core/Completable.java: 4.835
src/main/java/io/reactivex/rxjava4/core/CompletionStageDisposable.java: 4.609
src/main/java/io/reactivex/rxjava4/core/Streamable.java: 4.515
src/main/java/io/reactivex/rxjava4/exceptions/CompositeException.java: 4.442
src/main/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableWindowBoundarySelector.java: 4.038
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableWindowBoundarySelector.java: 4.090
src/main/java/io/reactivex/rxjava4/internal/operators/streamable/StreamableJust.java: 4.494
src/main/java/io/reactivex/rxjava4/internal/schedulers/SchedulerToExecutorService.java: 4.454
src/test/java/io/reactivex/rxjava4/flowable/FlowableCollectTest.java: 4.600
src/test/java/io/reactivex/rxjava4/internal/observers/QueueDrainObserverTest.java: 4.575
src/test/java/io/reactivex/rxjava4/internal/operators/completable/CompletableDetachTest.java: 4.650
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableCombineLatestTest.java: 4.693
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableConcatMapSchedulerTest.java: 4.751
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableConcatTest.java: 4.665
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableFromSourceTest.java: 4.455
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableGroupByTest.java: 4.630
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableMergeTest.java: 4.648

✅ No secrets or suspicious high-entropy strings found.

Mid-4 beauty heuristic in action — powered by our entropy chats! 😊

@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.32%. Comparing base (5091ed2) to head (a797027).

Additional details and impacted files
@@             Coverage Diff              @@
##                4.x    #8184      +/-   ##
============================================
- Coverage     98.33%   98.32%   -0.01%     
- Complexity     6933     6935       +2     
============================================
  Files           785      785              
  Lines         48999    48998       -1     
  Branches       6599     6599              
============================================
- Hits          48181    48176       -5     
+ Misses          623      622       -1     
- Partials        195      200       +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions

Copy link
Copy Markdown

🐷 TruffleHog + Entropy Beauty Scan

Average entropy of changed code: 4.558 bits/char
Verdict: ✅ Mid-4 beauty detected (thoughtful human code!)

Changed files entropy:

build.gradle: 5.004
docs/Creating-Observables.md: 5.159
docs/What's-different-in-2.0.md: 4.905
src/main/java/io/reactivex/rxjava4/core/Completable.java: 4.835
src/main/java/io/reactivex/rxjava4/core/CompletionStageDisposable.java: 4.609
src/main/java/io/reactivex/rxjava4/core/Streamable.java: 4.515
src/main/java/io/reactivex/rxjava4/exceptions/CompositeException.java: 4.442
src/main/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableWindowBoundarySelector.java: 4.038
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableWindowBoundarySelector.java: 4.090
src/main/java/io/reactivex/rxjava4/internal/operators/streamable/StreamableJust.java: 4.494
src/main/java/io/reactivex/rxjava4/internal/schedulers/SchedulerToExecutorService.java: 4.454
src/test/java/io/reactivex/rxjava4/flowable/FlowableCollectTest.java: 4.600
src/test/java/io/reactivex/rxjava4/internal/observers/QueueDrainObserverTest.java: 4.575
src/test/java/io/reactivex/rxjava4/internal/operators/completable/CompletableDetachTest.java: 4.650
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableCombineLatestTest.java: 4.693
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableConcatMapSchedulerTest.java: 4.751
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableConcatTest.java: 4.665
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableFromSourceTest.java: 4.455
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableGroupByTest.java: 4.630
src/test/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableMergeTest.java: 4.648

✅ No secrets or suspicious high-entropy strings found.

Mid-4 beauty heuristic in action — powered by our entropy chats! 😊

@akarnokd
akarnokd merged commit 78230e3 into ReactiveX:4.x Jun 26, 2026
6 checks passed
@akarnokd
akarnokd deleted the 4xDisposableAutoCloseableChange branch June 26, 2026 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant