ceil/floor/truncate/round return integers.

ceil/floor/truncate/round return integers.

This CL supercedes https://codereview.chromium.org/11748016/

Committed: https://code.google.com/p/dart/source/detail?r=19810

[Lasse Reichstein Nielsen, 2013-02-26 14:04:09]

LGTM.

[floitsch, 2013-02-26 15:26:10]

Adding srdjan who was opposing the first attempt. I hope this approach is more
acceptable.

[srdjan, 2013-02-26 16:35:26]

LGTM with comments and after checking that there are no benchmark regressions.

https://codereview.chromium.org/12317107/diff/2001/runtime/lib/double.dart
File runtime/lib/double.dart (right):

https://codereview.chromium.org/12317107/diff/2001/runtime/lib/double.dart#ne...
runtime/lib/double.dart:111: double truncateToDouble() native "Double_truncate";
Why not use roundToInt, and leave original semantic in? It is expected to that
every operation with double as receiver returns a double  (pow, abs, +, - ...)
unless explicitly said (toInt(), isNegative ..).

[floitsch, 2013-02-26 17:44:35]

https://codereview.chromium.org/12317107/diff/2001/runtime/lib/double.dart
File runtime/lib/double.dart (right):

https://codereview.chromium.org/12317107/diff/2001/runtime/lib/double.dart#ne...
runtime/lib/double.dart:111: double truncateToDouble() native "Double_truncate";
On 2013/02/26 16:35:26, srdjan wrote:
> Why not use roundToInt, and leave original semantic in? It is expected to that
> every operation with double as receiver returns a double  (pow, abs, +, - ...)
> unless explicitly said (toInt(), isNegative ..). 

That's already not the case: 5.2 ~/ 2.0 => 2

[floitsch, 2013-03-11 18:55:09]

Committed patchset #6 manually as r19810 (presubmit successful).