Skip to content

Commit e4e9a29

Browse files
authored
Remove not needed new_expanded_quosures() (#7090)
1 parent 1d17672 commit e4e9a29

File tree

1 file changed

+7
-14
lines changed

1 file changed

+7
-14
lines changed

R/across.R

Lines changed: 7 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -758,7 +758,7 @@ expand_across <- function(quo) {
758758

759759
# Empty expansion
760760
if (length(vars) == 0L) {
761-
return(new_expanded_quosures(list()))
761+
return(list())
762762
}
763763

764764
fns <- setup$fns
@@ -767,7 +767,7 @@ expand_across <- function(quo) {
767767
# No functions, so just return a list of symbols
768768
if (is.null(fns)) {
769769
# TODO: Deprecate and remove the `.fns = NULL` path in favor of `pick()`
770-
expressions <- pmap(list(vars, names, seq_along(vars)), function(var, name, k) {
770+
exprs <- pmap(list(vars, names, seq_along(vars)), function(var, name, k) {
771771
quo <- new_quosure(sym(var), empty_env())
772772
quo <- new_dplyr_quosure(
773773
quo,
@@ -777,9 +777,8 @@ expand_across <- function(quo) {
777777
column = var
778778
)
779779
})
780-
names(expressions) <- names
781-
expressions <- new_expanded_quosures(expressions)
782-
return(expressions)
780+
names(exprs) <- names
781+
return(exprs)
783782
}
784783

785784
n_vars <- length(vars)
@@ -788,8 +787,7 @@ expand_across <- function(quo) {
788787
seq_vars <- seq_len(n_vars)
789788
seq_fns <- seq_len(n_fns)
790789

791-
expressions <- vector(mode = "list", n_vars * n_fns)
792-
columns <- character(n_vars * n_fns)
790+
exprs <- new_list(n_vars * n_fns, names = names)
793791

794792
k <- 1L
795793
for (i in seq_vars) {
@@ -799,7 +797,7 @@ expand_across <- function(quo) {
799797
fn_call <- as_across_fn_call(fns[[j]], var, env, mask)
800798

801799
name <- names[[k]]
802-
expressions[[k]] <- new_dplyr_quosure(
800+
exprs[[k]] <- new_dplyr_quosure(
803801
fn_call,
804802
name = name,
805803
is_named = TRUE,
@@ -811,12 +809,7 @@ expand_across <- function(quo) {
811809
}
812810
}
813811

814-
names(expressions) <- names
815-
new_expanded_quosures(expressions)
816-
}
817-
818-
new_expanded_quosures <- function(x) {
819-
structure(x, class = "dplyr_expanded_quosures")
812+
exprs
820813
}
821814

822815
as_across_fn_call <- function(fn, var, env, mask) {

0 commit comments

Comments
 (0)