as.numeric(levels(factor(x))) may be a decreasing sequence
On Wed, May 27, 2009 at 10:51:38PM +0200, Martin Maechler wrote:
I have very slightly modified the changes (to get rid of -Wall warnings) and also exported the function as Rf_dropTrailing0(), and tested the result with 'make check-all' .
Thank you very much for considering the patch. -Wall indeed requires to add
parentheses
warning: suggest parentheses around comparison in operand of &
warning: suggest parentheses around assignment used as truth value
If there are also other changes, i would like to ask you to make your modification
available, mainly due to a possible further discussion.
Let me also suggest a modification of my original proposal. It contains a cycle
while (*(replace++) = *(p++)) {
;
}
If the number has no trailing zeros, but contains an exponent, this cycle
shifts the exponent by 0 positions, which means that it copies each of its
characters to itself. This may be eliminated as follows
if (replace != p) {
while (*(replace++) = *(p++)) {
;
}
}
Petr.