Skip to content

Commit 3ad437f

Browse files
author
hornik
committed
s/sapply/vapply.
git-svn-id: https://svn.r-project.org/R/trunk@88302 00db46b3-68df-0310-9c12-caf00c1e9a41
1 parent 0a86797 commit 3ad437f

File tree

5 files changed

+45
-49
lines changed

5 files changed

+45
-49
lines changed

src/library/base/R/dynload.R

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -151,16 +151,15 @@ function(x, ...)
151151
x <- x[n > 0]
152152
n <- max(n)
153153
d <- list()
154-
sapply(names(x),
155-
function(id) {
156-
d[[id]] <<- rep.int("", n)
157-
names <- vapply(x[[id]], `[[`, "", "name")
158-
if(length(names)) d[[id]][seq_along(names)] <<- names
159-
d[[paste(id, "numParameters")]] <<- rep.int("", n)
160-
names <- vapply(x[[id]], `[[`, 0L, "numParameters")
161-
if(length(names))
162-
d[[paste(id, "numParameters")]][seq_along(names)] <<- names
163-
})
154+
for(id in names(x)) {
155+
d[[id]] <- rep.int("", n)
156+
names <- vapply(x[[id]], `[[`, "", "name")
157+
if(length(names)) d[[id]][seq_along(names)] <- names
158+
d[[paste(id, "numParameters")]] <- rep.int("", n)
159+
names <- vapply(x[[id]], `[[`, 0L, "numParameters")
160+
if(length(names))
161+
d[[paste(id, "numParameters")]][seq_along(names)] <- names
162+
}
164163
print(as.data.frame(d), ...)
165164
invisible(x)
166165
}

src/library/tools/R/QC.R

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3190,7 +3190,7 @@ function(dir, force_suggests = TRUE, check_incoming = FALSE,
31903190
installed_in <- c(installed_in, rep.int(lib, length(pkgs)))
31913191
}
31923192
if (length(lreqs)) {
3193-
reqs <- unique(sapply(lreqs, `[[`, 1L))
3193+
reqs <- unique(vapply(lreqs, `[[`, "", 1L))
31943194
bad <- setdiff(reqs, installed)
31953195
if(length(bad)) {
31963196
## EDanalysis has a package in all of Depends, Imports, Suggests.
@@ -3226,12 +3226,12 @@ function(dir, force_suggests = TRUE, check_incoming = FALSE,
32263226
if (length(lenhances) &&
32273227
!config_val_to_logical(Sys.getenv("_R_CHECK_PACKAGE_DEPENDS_IGNORE_MISSING_ENHANCES_",
32283228
"FALSE"))) {
3229-
m <- setdiff(sapply(lenhances, `[[`, 1L), installed)
3229+
m <- setdiff(vapply(lenhances, `[[`, "", 1L), installed)
32303230
if(length(m))
32313231
bad_depends$enhances_but_not_installed <- m
32323232
}
32333233
if (!force_suggests && length(lsuggests)) {
3234-
m <- setdiff(sapply(lsuggests, `[[`, 1L), installed)
3234+
m <- setdiff(vapply(lsuggests, `[[`, "", 1L), installed)
32353235
if(length(m))
32363236
bad_depends$suggests_but_not_installed <- m
32373237
}
@@ -3345,12 +3345,12 @@ function(dir, force_suggests = TRUE, check_incoming = FALSE,
33453345
## First use dependencies which are installed: strict dependencies
33463346
## need to be for a full check.
33473347
## Suggests might not even exist, so we suppress warnings.
3348-
mt <- utils::maintainer
3349-
strict2 <- sapply(strict, function(x) suppressWarnings(mt(x)))
3348+
mt <- function(x) suppressWarnings(utils::maintainer(x))
3349+
strict2 <- vapply(strict, mt, "")
33503350
miss1 <- is.na(strict2)
33513351
weak <- setdiff(as.character(suggests),
33523352
bad_depends$suggested_but_not_installed)
3353-
weak2 <- sapply(weak, function(x) suppressWarnings(mt(x)))
3353+
weak2 <- vapply(weak, mt, "")
33543354
miss2 <- is.na(weak2)
33553355
if((any(miss1) || any(miss2)) &&
33563356
!inherits(tryCatch(db <- CRAN_package_db()[, c("Package",

src/library/tools/R/install.R

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1222,9 +1222,12 @@ if(FALSE) {
12221222
if (!is.na(use_C))
12231223
ev1 <- c(sprintf(c("CC%s", "C%sFLAGS"), use_C),
12241224
ev[-(1:2)])
1225-
ev2 <- sapply(ev1, function(x)
1226-
system2(file.path(R.home("bin"), "R"), c("CMD", "config", x),
1227-
stdout = TRUE))
1225+
ev2 <- vapply(ev1,
1226+
function(x)
1227+
system2(file.path(R.home("bin"), "R"),
1228+
c("CMD", "config", x),
1229+
stdout = TRUE),
1230+
"")
12281231
ev3 <- paste0(ev, "=", shQuote(ev2))
12291232
## skip any which are empty, possible for CXX)
12301233
ev3 <- ev3[nzchar(ev2)]

src/library/utils/R/package.skeleton.R

Lines changed: 21 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -217,33 +217,27 @@ package.skeleton <-
217217
file.path(docs_dir,
218218
sprintf("%s-package.Rd", name)),
219219
lib.loc = path)
220-
vapply(list,
221-
function(item) {
222-
prompt(get(item, envir = environment),
223-
name = item,
224-
filename =
225-
file.path(docs_dir,
226-
sprintf("%s.Rd", list0[item])))
227-
},
228-
"")
229-
vapply(classesList,
230-
function(item) {
231-
methods::promptClass(item,
232-
filename =
233-
file.path(docs_dir,
234-
sprintf("%s-class.Rd", classes0[item])),
235-
where = environment)
236-
},
237-
"")
238-
vapply(methodsList,
239-
function(item) {
240-
methods::promptMethods(item,
241-
filename =
242-
file.path(docs_dir,
243-
sprintf("%s-methods.Rd", methods0[item])),
244-
methods::findMethods(item, where = environment))
245-
},
246-
"")
220+
for(item in list) {
221+
prompt(get(item, envir = environment),
222+
name = item,
223+
filename =
224+
file.path(docs_dir,
225+
sprintf("%s.Rd", list0[item])))
226+
}
227+
for(item in classesList) {
228+
methods::promptClass(item,
229+
filename =
230+
file.path(docs_dir,
231+
sprintf("%s-class.Rd", classes0[item])),
232+
where = environment)
233+
}
234+
for(item in methodsList) {
235+
methods::promptMethods(item,
236+
filename =
237+
file.path(docs_dir,
238+
sprintf("%s-methods.Rd", methods0[item])),
239+
methods::findMethods(item, where = environment))
240+
}
247241
}))
248242
## don't document generic functions from other packages
249243
for(item in methodsList) {

src/library/utils/R/sessionInfo.R

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -157,9 +157,9 @@ print.sessionInfo <- function(x, locale = TRUE, tzone = locale,
157157
...)
158158
{
159159
mkLabel <- function(L, n) {
160-
vers <- sapply(L[[n]], function(x) x[["Version"]])
161-
pkg <- sapply(L[[n]], function(x) x[["Package"]])
162-
paste(pkg, vers, sep = "_")
160+
paste(vapply(L[[n]], `[[`, "", "Package"),
161+
vapply(L[[n]], `[[`, "", "Version"),
162+
sep = "_")
163163
}
164164

165165
cat(x$R.version$version.string, "\n", sep = "")

0 commit comments

Comments
 (0)