cancel() on an already canceled future did not return it invisibly

HenrikBengtsson · HenrikBengtsson · commit 7266db4e134d · 2025-07-23T23:38:05.000+02:00
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,5 +1,5 @@
 Package: future
-Version: 1.58.0-9024
+Version: 1.58.0-9025
 Title: Unified Parallel and Distributed Processing in R for Everyone
 Depends:
     R (>= 3.2.0)
diff --git a/R/core_api-cancel.R b/R/core_api-cancel.R
@@ -88,7 +88,7 @@ cancel.Future <- function(x, interrupt = TRUE, ...) {
   ## Only running futures can be canceled, ignore everything else
   if (future[["state"]] != "running") {
     if (debug) mdebug("Skipping, because a non-running future")
-    return(future)
+    return(invisible(future))
   }
 
   if (interrupt) {

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ cancel.Future <- function(x, interrupt = TRUE, ...) {`
`88`	`88`	`## Only running futures can be canceled, ignore everything else`
`89`	`89`	`if (future[["state"]] != "running") {`
`90`	`90`	`if (debug) mdebug("Skipping, because a non-running future")`
`91`		`- return(future)`
	`91`	`+ return(invisible(future))`
`92`	`92`	`}`
`93`	`93`
`94`	`94`	`if (interrupt) {`