CLEANUP: Remove check for defunct argument 'value' from resolve() met…

…hods; it's been defunct since Dec 2019
futureverse · Jan 1, 2025 · ad85fb7 · ad85fb7
1 parent 777da99
commit ad85fb7
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 21 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,5 +1,5 @@
 Package: future
-Version: 1.34.0-9004
+Version: 1.34.0-9005
 Title: Unified Parallel and Distributed Processing in R for Everyone
 Imports:
     digest,

diff --git a/R/resolve.R b/R/resolve.R
@@ -52,11 +52,6 @@ resolve.default <- function(x, ...) x
 
 #' @export
 resolve.Future <- function(x, idxs = NULL, recursive = 0, result = FALSE, stdout = FALSE, signal = FALSE, force = FALSE, sleep = getOption("future.wait.interval", 0.01), ...) {
-  ## BACKWARD COMPATIBILITY
-  if (missing(result) && "value" %in% names(list(...))) {
-    .Defunct(msg = "Argument 'value' of resolve() is defunct. It was deprecated in future (>= 1.15.0) [2019-11-07]. Use 'result' instead.", package = .packageName)
-  }
-
   ## Automatically update journal entries for Future object
   if (inherits(future, "Future") &&
       inherits(future$.journal, "FutureJournal")) {
@@ -134,11 +129,6 @@ resolve.Future <- function(x, idxs = NULL, recursive = 0, result = FALSE, stdout
 
 #' @export
 resolve.list <- function(x, idxs = NULL, recursive = 0, result = FALSE, stdout = FALSE, signal = FALSE, force = FALSE, sleep = getOption("future.wait.interval", 0.01), ...) {
-  ## BACKWARD COMPATIBILITY
-  if (missing(result) && "value" %in% names(list(...))) {
-    .Defunct(msg = "Argument 'value' of resolve() is defunct. It was deprecated in future (>= 1.15.0) [2019-11-07]. Use 'result' instead.", package = .packageName)
-  }
-
   if (is.logical(recursive)) {
     if (recursive) recursive <- getOption("future.resolve.recursive", 99)
   }
@@ -262,11 +252,6 @@ resolve.list <- function(x, idxs = NULL, recursive = 0, result = FALSE, stdout =
 
 #' @export
 resolve.environment <- function(x, idxs = NULL, recursive = 0, result = FALSE, stdout = FALSE, signal = FALSE, force = FALSE, sleep = getOption("future.wait.interval", 0.01), ...) {
-  ## BACKWARD COMPATIBILITY
-  if (missing(result) && "value" %in% names(list(...))) {
-    .Defunct(msg = "Argument 'value' of resolve() is defunct. It was deprecated in future (>= 1.15.0) [2019-11-07]. Use 'result' instead.", package = .packageName)
-  }
-
   if (is.logical(recursive)) {
     if (recursive) recursive <- getOption("future.resolve.recursive", 99)
   }
@@ -385,11 +370,6 @@ resolve.environment <- function(x, idxs = NULL, recursive = 0, result = FALSE, s
 
 #' @export
 resolve.listenv <- function(x, idxs = NULL, recursive = 0, result = FALSE, stdout = FALSE, signal = FALSE, force = FALSE, sleep = getOption("future.wait.interval", 0.01), ...) {
-  ## BACKWARD COMPATIBILITY
-  if (missing(result) && "value" %in% names(list(...))) {
-    .Defunct(msg = "Argument 'value' of resolve() is defunct. It was deprecated in future (>= 1.15.0) [2019-11-07]. Use 'result' instead.", package = .packageName)
-  }
-
   if (is.logical(recursive)) {
     if (recursive) recursive <- getOption("future.resolve.recursive", 99)
   }