Skip to content

4.x cleanup: spinwait, grammar, minor syntax simplifications - #8185

Closed
akarnokd wants to merge 2 commits into
ReactiveX:4.xfrom
akarnokd:4xCleanups4
Closed

4.x cleanup: spinwait, grammar, minor syntax simplifications#8185
akarnokd wants to merge 2 commits into
ReactiveX:4.xfrom
akarnokd:4xCleanups4

Conversation

@akarnokd

Copy link
Copy Markdown
Member
  • Use Thread.onSpinWait()
  • Javadoc insert <p> to correct styling.
  • Some signalled -> signaled
  • Some cancelled -> canceled

@akarnokd akarnokd added this to the 4.0 milestone Jun 26, 2026
@github-actions

Copy link
Copy Markdown

🐷 TruffleHog + Entropy Beauty Scan

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

Changed files entropy:

build.gradle: 5.005
src/main/java/io/reactivex/rxjava4/disposables/Disposable.java: 4.677
src/main/java/io/reactivex/rxjava4/disposables/DisposableContainer.java: 4.421
src/main/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableRefCount.java: 4.338
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableAmb.java: 4.224
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableCombineLatest.java: 4.104
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableRefCount.java: 4.318
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableZip.java: 3.973
src/main/java/io/reactivex/rxjava4/internal/schedulers/DeferredExecutorScheduler.java: 3.992
src/main/java/io/reactivex/rxjava4/internal/schedulers/ExecutorScheduler.java: 4.073
src/main/java/io/reactivex/rxjava4/internal/util/AwaitCoordinatorStatic.java: 4.508
src/main/java/io/reactivex/rxjava4/internal/virtual/FlowableVirtualCreateExecutor.java: 4.454
src/main/java/io/reactivex/rxjava4/internal/virtual/FlowableVirtualTransformExecutor.java: 4.143
src/test/java/io/reactivex/rxjava4/completable/CompletableTest.java: 4.630
src/test/java/io/reactivex/rxjava4/core/DisposeTaskTest.java: 4.772
src/test/java/io/reactivex/rxjava4/core/PeriodicDirectTaskTest.java: 4.776
src/test/java/io/reactivex/rxjava4/disposables/CompositeDisposableTest.java: 4.490
src/test/java/io/reactivex/rxjava4/disposables/DisposableTest.java: 4.643
src/test/java/io/reactivex/rxjava4/disposables/FutureDisposableTest.java: 4.892
src/test/java/io/reactivex/rxjava4/disposables/SerialDisposableTests.java: 4.541

✅ 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

❌ Patch coverage is 89.47368% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.34%. Comparing base (78230e3) to head (b115d5d).

Files with missing lines Patch % Lines
...nal/schedulers/BlockingCurrentThreadScheduler.java 0.00% 0 Missing and 2 partials ⚠️

❌ Your patch check has failed because the patch coverage (89.47%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff              @@
##                4.x    #8185      +/-   ##
============================================
+ Coverage     98.32%   98.34%   +0.01%     
  Complexity     6936     6936              
============================================
  Files           785      785              
  Lines         48998    48987      -11     
  Branches       6599     6598       -1     
============================================
- Hits          48179    48174       -5     
+ Misses          622      620       -2     
+ Partials        197      193       -4     

☔ 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.636 bits/char
Verdict: ✅ Mid-4 beauty detected (thoughtful human code!)

Changed files entropy:

build.gradle: 5.005
src/main/java/io/reactivex/rxjava4/disposables/Disposable.java: 4.677
src/main/java/io/reactivex/rxjava4/disposables/DisposableContainer.java: 4.421
src/main/java/io/reactivex/rxjava4/internal/operators/flowable/FlowableRefCount.java: 4.338
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableAmb.java: 4.224
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableCombineLatest.java: 4.104
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableRefCount.java: 4.318
src/main/java/io/reactivex/rxjava4/internal/operators/observable/ObservableZip.java: 3.973
src/main/java/io/reactivex/rxjava4/internal/schedulers/DeferredExecutorScheduler.java: 3.992
src/main/java/io/reactivex/rxjava4/internal/schedulers/ExecutorScheduler.java: 4.073
src/main/java/io/reactivex/rxjava4/internal/util/AwaitCoordinatorStatic.java: 4.508
src/main/java/io/reactivex/rxjava4/internal/virtual/FlowableVirtualCreateExecutor.java: 4.454
src/main/java/io/reactivex/rxjava4/internal/virtual/FlowableVirtualTransformExecutor.java: 4.143
src/test/java/io/reactivex/rxjava4/completable/CompletableTest.java: 4.630
src/test/java/io/reactivex/rxjava4/core/DisposeTaskTest.java: 4.772
src/test/java/io/reactivex/rxjava4/core/PeriodicDirectTaskTest.java: 4.776
src/test/java/io/reactivex/rxjava4/disposables/CompositeDisposableTest.java: 4.490
src/test/java/io/reactivex/rxjava4/disposables/DisposableTest.java: 4.643
src/test/java/io/reactivex/rxjava4/disposables/FutureDisposableTest.java: 4.892
src/test/java/io/reactivex/rxjava4/disposables/SerialDisposableTests.java: 4.541

✅ No secrets or suspicious high-entropy strings found.

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

@akarnokd

Copy link
Copy Markdown
Member Author

Another weird crash on CI. Probably that spinwait introduced a runaway allocation somewhere?

@akarnokd akarnokd closed this Jun 26, 2026
@akarnokd
akarnokd deleted the 4xCleanups4 branch June 26, 2026 12:58
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