From maechler at stat.math.ethz.ch  Mon Feb 27 10:20:36 2017
From: maechler at stat.math.ethz.ch (Martin Maechler)
Date: Mon, 27 Feb 2017 10:20:36 +0100
Subject: [Rd] rep/rep.int: in NEWS, but not yet ported from trunk
In-Reply-To: <1490161585.1255328.1488110564692@mail.yahoo.com>
References: <1490161585.1255328.1488110564692.ref@mail.yahoo.com>
	<1490161585.1255328.1488110564692@mail.yahoo.com>
Message-ID: <22707.61284.80736.472517@stat.math.ethz.ch>

>>>>> Suharto Anggono Suharto Anggono via R-devel <r-devel at r-project.org>
>>>>>     on Sun, 26 Feb 2017 12:02:44 +0000 writes:

    > According to "CHANGES IN R 3.3.2 patched" in NEWS, rep(x,
    > times) and rep.int(x, times) also work when 'times' has
    > length greater than one and has element larger than the
    > maximal integer. In fact, it is still not the case in R
    > 3.3.3 beta r72259. In seq.c
    > (https://svn.r-project.org/R/branches/R-3-3-branch/src/main/seq.c),
    > 'times' that is a vector with storage mode "double" and
    > length greater than one is still changed first to storage
    > mode "integer". Number in 'times' that represents an
    > integer that is larger than the maximal integer becomes NA
    > and error is issued for such 'times'.  
    > I have put a comment,
    > https://bugs.r-project.org/bugzilla3/show_bug.cgi?id=16932#c30
    > .

Thank you for noticing!

- I've changed the NEWS entry for R-patched (and moved the more
general statement to a new entry for R-devel). 
- The changes were quite substantial so I did not port them to
R-patched at the time..  We could have ported them later, but
not now, immediately before code freeze (of R 3.3.3).

- I would say   rep(5, list(6))  was never "meant to" work and had worked
  incidentally only.
  OTOH, you are correct with your comments 11 & 29 in the about
  bug report, and your proposal to make the simple case   rep(s, list(7))
  work as previously seems ok to me.

However, for all this, we will concentrate on R-devel (to become
R 3.4.0).

Best regards,
Martin Maechler



[Rd] rep/rep.int: in NEWS, but not yet ported from trunk
Martin Maechler