From @cspencer

The floor/ceiling/round math builtins were returning Num values, causing
statements such as​:

  my Int $x = round(5.4);

to fail with a type mismatch. This patch fixes the above builtins so that
Int values are returned as per S29.

The patch is against src/builtins/math.pir

From @cspencer

28,30c28,31
<     .param num a
<     floor a
<     .return (a)
---
>     .param num n
>     .local int i
>     floor i, n
>     .return (i)
44,46c45,48
<     .param num a
<     ceil a
<     .return (a)
---
>     .param num n
>     .local int i
>     ceil i, n
>     .return (i)
61,64c63,67
<     .param num a
<     a += 0.5
<     $N0 = floor a
<     .return ($N0)
---
>     .param num n
>     .local int i
>     n += 0.5
>     floor i, n
>     .return (i)

From @moritz

On Fri Dec 19 08​:35​:04 2008, cspencer wrote​:

The floor/ceiling/round math builtins were returning Num values, causing
statements such as​:

my Int $x = round(5.4);

to fail with a type mismatch. This patch fixes the above builtins so
that
Int values are returned as per S29.

The patch is against src/builtins/math.pir

Thanks, applied as r34116, and added tests to t/spec/S29-num/rounders.t

In future it would be nice if you could generate your patches with 'svn
diff path/to/file', that way they will be easier to apply for me (or if
you do want to use the normal diff, pass the -u option to it).

Cheers,
Moritz

The RT System itself - Status changed from 'new' to 'open'

@moritz - Status changed from 'open' to 'resolved'