[Rd] Lack of 'seq_len' in 'head' in 'stopifnot'
Martin Maechler
maechler at stat.math.ethz.ch
Sat Feb 4 17:27:54 CET 2017
>>>>> Suharto Anggono Suharto Anggono via R-devel <r-devel at r-project.org>
>>>>> on Sat, 4 Feb 2017 10:18:33 +0000 writes:
> Function 'stopifnot' in R devel r72104 has this.
> head <- function(x, n = 6L) ## basically utils:::head.default()
> x[if(n < 0L) max(length(x) + n, 0L) else min(n, length(x))]
> If definition like utils:::head.default is intended, the index of 'x' should be wrapped in seq_len(...):
> x[seq_len(...)]
You are right... that was "lost in translation" .
As seq_len(1) is 1 and that seems to have been the only case
much exercised, nobody seems to have noticed the problem till
now ((this assumes people *would* report it if they noticed.
Yes, "hope dies last" ;-))
Thank you, this is amended now.
Martin
More information about the R-devel
mailing list