From eb9efb54d60c7fca75779435e566cfa2bd08132a Mon Sep 17 00:00:00 2001 From: Florian Angerer <florian.angerer@oracle.com> Date: Thu, 1 Jun 2017 15:09:54 +0200 Subject: [PATCH] Cleanup --- .../nodes/builtin/fastr/FastRSourceInfo.java | 12 +++--- .../r/nodes/builtin/tools/R/tools_overrides.R | 37 ------------------- 2 files changed, 5 insertions(+), 44 deletions(-) diff --git a/com.oracle.truffle.r.nodes.builtin/src/com/oracle/truffle/r/nodes/builtin/fastr/FastRSourceInfo.java b/com.oracle.truffle.r.nodes.builtin/src/com/oracle/truffle/r/nodes/builtin/fastr/FastRSourceInfo.java index 232cabd534..5f9863bb15 100644 --- a/com.oracle.truffle.r.nodes.builtin/src/com/oracle/truffle/r/nodes/builtin/fastr/FastRSourceInfo.java +++ b/com.oracle.truffle.r.nodes.builtin/src/com/oracle/truffle/r/nodes/builtin/fastr/FastRSourceInfo.java @@ -28,7 +28,6 @@ import static com.oracle.truffle.r.runtime.builtins.RBuiltinKind.PRIMITIVE; import com.oracle.truffle.api.dsl.Fallback; import com.oracle.truffle.api.dsl.Specialization; -import com.oracle.truffle.api.frame.VirtualFrame; import com.oracle.truffle.api.nodes.Node; import com.oracle.truffle.api.source.SourceSection; import com.oracle.truffle.r.nodes.builtin.RBuiltinNode; @@ -51,17 +50,16 @@ public abstract class FastRSourceInfo extends RBuiltinNode.Arg1 { } @Specialization - public Object srcInfo(VirtualFrame frame, RFunction fun) { - return execute(frame, fun.getRootNode()); + public Object srcInfo(RFunction fun) { + return srcInfo(fun.getRootNode()); } @Specialization - public Object srcInfo(VirtualFrame frame, RLanguage fun) { - return execute(frame, fun.getRep()); + public Object srcInfo(RLanguage fun) { + return srcInfo(fun.getRep()); } - @Specialization - protected Object srcInfo(Node fun) { + private Object srcInfo(Node fun) { SourceSection ss = fun.getSourceSection(); if (ss != null) { String path = ss.getSource().getPath(); diff --git a/com.oracle.truffle.r.nodes.builtin/src/com/oracle/truffle/r/nodes/builtin/tools/R/tools_overrides.R b/com.oracle.truffle.r.nodes.builtin/src/com/oracle/truffle/r/nodes/builtin/tools/R/tools_overrides.R index 9eee831d8b..6b33560d53 100644 --- a/com.oracle.truffle.r.nodes.builtin/src/com/oracle/truffle/r/nodes/builtin/tools/R/tools_overrides.R +++ b/com.oracle.truffle.r.nodes.builtin/src/com/oracle/truffle/r/nodes/builtin/tools/R/tools_overrides.R @@ -20,43 +20,6 @@ # questions. eval(expression({ -makeLazyLoading <- - function(package, lib.loc = NULL, compress = TRUE, - keep.source = getOption("keep.source.pkgs")) -{ - if(!is.logical(compress) && ! compress %in% c(2,3)) - stop("invalid value for 'compress': should be FALSE, TRUE, 2 or 3") - options(warn = 1L) - findpack <- function(package, lib.loc) { - pkgpath <- find.package(package, lib.loc, quiet = TRUE) - if(!length(pkgpath)) - stop(gettextf("there is no package called '%s'", package), - domain = NA) - pkgpath - } - - if (package == "base") - stop("this cannot be used for package 'base'") - - loaderFile <- file.path(R.home("share"), "R", "nspackloader.R") - pkgpath <- findpack(package, lib.loc) - codeFile <- file.path(pkgpath, "R", package) - - if (!file.exists(codeFile)) { - warning("package contains no R code") - return(invisible()) - } - if (file.size(codeFile) == file.size(loaderFile)) - warning("package seems to be using lazy loading already") - else { - code2LazyLoadDB(package, lib.loc = lib.loc, - keep.source = keep.source, compress = compress) - file.copy(loaderFile, codeFile, TRUE) - } - - invisible() -} - .install_package_code_files <- function(dir, outDir) { -- GitLab