Skip to content

Commit 89fa37a

Browse files
committed
Fix lintr warnings
1 parent db2ce74 commit 89fa37a

File tree

1 file changed

+28
-14
lines changed

1 file changed

+28
-14
lines changed

R/logger.R

Lines changed: 28 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -393,44 +393,58 @@ validate_log_level <- function(level) {
393393
#' @export
394394
#' @rdname log_level
395395
log_fatal <- function(..., namespace = NA_character_,
396-
.logcall = sys.call(), .topcall = sys.call(-1), .topenv = parent.frame(), .timestamp = Sys.time()) {
397-
log_level(FATAL, ..., namespace = namespace, .logcall = .logcall, .topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
396+
.logcall = sys.call(), .topcall = sys.call(-1),
397+
.topenv = parent.frame(), .timestamp = Sys.time()) {
398+
log_level(FATAL, ..., namespace = namespace, .logcall = .logcall,
399+
.topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
398400
}
399401
#' @export
400402
#' @rdname log_level
401403
log_error <- function(..., namespace = NA_character_,
402-
.logcall = sys.call(), .topcall = sys.call(-1), .topenv = parent.frame(), .timestamp = Sys.time()) {
403-
log_level(ERROR, ..., namespace = namespace, .logcall = .logcall, .topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
404+
.logcall = sys.call(), .topcall = sys.call(-1),
405+
.topenv = parent.frame(), .timestamp = Sys.time()) {
406+
log_level(ERROR, ..., namespace = namespace, .logcall = .logcall,
407+
.topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
404408
}
405409
#' @export
406410
#' @rdname log_level
407411
log_warn <- function(..., namespace = NA_character_,
408-
.logcall = sys.call(), .topcall = sys.call(-1), .topenv = parent.frame(), .timestamp = Sys.time()) {
409-
log_level(WARN, ..., namespace = namespace, .logcall = .logcall, .topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
412+
.logcall = sys.call(), .topcall = sys.call(-1),
413+
.topenv = parent.frame(), .timestamp = Sys.time()) {
414+
log_level(WARN, ..., namespace = namespace, .logcall = .logcall,
415+
.topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
410416
}
411417
#' @export
412418
#' @rdname log_level
413419
log_success <- function(..., namespace = NA_character_,
414-
.logcall = sys.call(), .topcall = sys.call(-1), .topenv = parent.frame(), .timestamp = Sys.time()) {
415-
log_level(SUCCESS, ..., namespace = namespace, .logcall = .logcall, .topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
420+
.logcall = sys.call(), .topcall = sys.call(-1),
421+
.topenv = parent.frame(), .timestamp = Sys.time()) {
422+
log_level(SUCCESS, ..., namespace = namespace, .logcall = .logcall,
423+
.topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
416424
}
417425
#' @export
418426
#' @rdname log_level
419427
log_info <- function(..., namespace = NA_character_,
420-
.logcall = sys.call(), .topcall = sys.call(-1), .topenv = parent.frame(), .timestamp = Sys.time()) {
421-
log_level(INFO, ..., namespace = namespace, .logcall = .logcall, .topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
428+
.logcall = sys.call(), .topcall = sys.call(-1),
429+
.topenv = parent.frame(), .timestamp = Sys.time()) {
430+
log_level(INFO, ..., namespace = namespace, .logcall = .logcall,
431+
.topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
422432
}
423433
#' @export
424434
#' @rdname log_level
425435
log_debug <- function(..., namespace = NA_character_,
426-
.logcall = sys.call(), .topcall = sys.call(-1), .topenv = parent.frame(), .timestamp = Sys.time()) {
427-
log_level(DEBUG, ..., namespace = namespace, .logcall = .logcall, .topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
436+
.logcall = sys.call(), .topcall = sys.call(-1),
437+
.topenv = parent.frame(), .timestamp = Sys.time()) {
438+
log_level(DEBUG, ..., namespace = namespace, .logcall = .logcall,
439+
.topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
428440
}
429441
#' @export
430442
#' @rdname log_level
431443
log_trace <- function(..., namespace = NA_character_,
432-
.logcall = sys.call(), .topcall = sys.call(-1), .topenv = parent.frame(), .timestamp = Sys.time()) {
433-
log_level(TRACE, ..., namespace = namespace, .logcall = .logcall, .topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
444+
.logcall = sys.call(), .topcall = sys.call(-1),
445+
.topenv = parent.frame(), .timestamp = Sys.time()) {
446+
log_level(TRACE, ..., namespace = namespace, .logcall = .logcall,
447+
.topcall = .topcall, .topenv = .topenv, .timestamp = .timestamp)
434448
}
435449

436450

0 commit comments

Comments
 (0)