CLEANUP: Drop 'collect = FALSE' to signalEarly() for already resolved uniprocess and sequential futures, because it makes no difference from the default 'collect = TRUE'

HenrikBengtsson · HenrikBengtsson · commit e3df9d4a05c5 · 2025-12-06T13:33:24.000-08:00
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,5 +1,5 @@
 Package: future
-Version: 1.68.0-9024
+Version: 1.68.0-9025
 Title: Unified Parallel and Distributed Processing in R for Everyone
 Depends:
     R (>= 3.2.0)
diff --git a/R/backend_api-11.SequentialFutureBackend-class.R b/R/backend_api-11.SequentialFutureBackend-class.R
@@ -72,7 +72,7 @@ launchFuture.SequentialFutureBackend <- function(backend, future, ...) {
   signalImmediateConditions(future)
 
   ## Signal conditions early, iff specified for the given future
-  signalEarly(future, collect = FALSE)
+  signalEarly(future)
 
   hooks <- backend[["hooks"]]
   if (hooks) {
diff --git a/R/backend_api-UniprocessFuture-class.R b/R/backend_api-UniprocessFuture-class.R
@@ -45,7 +45,7 @@ run.UniprocessFuture <- function(future, ...) {
   signalImmediateConditions(future)
 
   ## Signal conditions early, iff specified for the given future
-  signalEarly(future, collect = FALSE)
+  signalEarly(future)
 
   future
 }

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ run.UniprocessFuture <- function(future, ...) {`
`45`	`45`	`signalImmediateConditions(future)`
`46`	`46`
`47`	`47`	`## Signal conditions early, iff specified for the given future`
`48`		`- signalEarly(future, collect = FALSE)`
	`48`	`+ signalEarly(future)`
`49`	`49`
`50`	`50`	`future`
`51`	`51`	`}`