warningf

venom1204 · venom1204 · commit b150b0a5ffc6 · 2025-08-20T20:45:32.000Z
diff --git a/R/data.table.R b/R/data.table.R
@@ -740,8 +740,10 @@ replace_dot_alias = function(e) {
       if (is.character(j) || (is.numeric(j) && !is.logical(j))) {
           if (!missingby) {
               j_type = if (is.character(j)) "a character" else "a numeric"
-              fmt = "`by` or `keyby` is ignored when `j` is %s vector used for column selection. Perhaps you intended to use `.SD`? For example: DT[, .SD[, %s], by = ...]"
-              warningf(sprintf(fmt, j_type, gsub("%", "%%", deparse(jsub))), call. = FALSE)
+              warningf(
+                  "`by` or `keyby` is ignored when `j` is %s vector used for column selection. Perhaps you intended to use `.SD`? For example: DT[, .SD[, %s], by = ...]",j_type,
+                  gsub("%", "%%", deparse(jsub))
+              )
           }
       }
 

Original file line number	Diff line number	Diff line change
`@@ -740,8 +740,10 @@ replace_dot_alias = function(e) {`
`740`	`740`	`if (is.character(j) \|\| (is.numeric(j) && !is.logical(j))) {`
`741`	`741`	`if (!missingby) {`
`742`	`742`	`j_type = if (is.character(j)) "a character" else "a numeric"`
`743`		- fmt = "`by` or `keyby` is ignored when `j` is %s vector used for column selection. Perhaps you intended to use `.SD`? For example: DT[, .SD[, %s], by = ...]"
`744`		`- warningf(sprintf(fmt, j_type, gsub("%", "%%", deparse(jsub))), call. = FALSE)`
	`743`	`+ warningf(`
	`744`	+ "`by` or `keyby` is ignored when `j` is %s vector used for column selection. Perhaps you intended to use `.SD`? For example: DT[, .SD[, %s], by = ...]",j_type,
	`745`	`+ gsub("%", "%%", deparse(jsub))`
	`746`	`+ )`
`745`	`747`	`}`
`746`	`748`	`}`
`747`	`749`