implicit returns

MichaelChirico · web-flow · commit 0bb079913b8d · 2025-07-01T11:45:58.000-07:00
diff --git a/R/mergelist.R b/R/mergelist.R
@@ -41,7 +41,7 @@ hasindex = function(x, by, retGrp=FALSE) {
   idx = attr(index, idx_name, TRUE)
   if (is.null(idx)) return(FALSE)
   if (!retGrp) return(TRUE)
-  return(!is.null(attr(idx, "starts", TRUE)))
+  !is.null(attr(idx, "starts", TRUE))
 }
 
 # fdistinct applies mult='first|last'
@@ -140,7 +140,7 @@ dtmerge = function(x, i, on, how, mult, join.many, void=FALSE, verbose) {
   if (len.i != len.x)
     internal_error("dtmerge out len.i != len.x") # nocov
 
-  return(list(ans=ans, irows=irows, xrows=xrows))
+  list(ans=ans, irows=irows, xrows=xrows)
 }
 
 # Previously, we had a custom C implementation here, which is ~2x faster,

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ hasindex = function(x, by, retGrp=FALSE) {`
`41`	`41`	`idx = attr(index, idx_name, TRUE)`
`42`	`42`	`if (is.null(idx)) return(FALSE)`
`43`	`43`	`if (!retGrp) return(TRUE)`
`44`		`- return(!is.null(attr(idx, "starts", TRUE)))`
	`44`	`+ !is.null(attr(idx, "starts", TRUE))`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`# fdistinct applies mult='first\|last'`
`@@ -140,7 +140,7 @@ dtmerge = function(x, i, on, how, mult, join.many, void=FALSE, verbose) {`
`140`	`140`	`if (len.i != len.x)`
`141`	`141`	`internal_error("dtmerge out len.i != len.x") # nocov`
`142`	`142`
`143`		`- return(list(ans=ans, irows=irows, xrows=xrows))`
	`143`	`+ list(ans=ans, irows=irows, xrows=xrows)`
`144`	`144`	`}`
`145`	`145`
`146`	`146`	`# Previously, we had a custom C implementation here, which is ~2x faster,`