LibJS: Math.round() should call round() instead of roundf()

awesomekling · awesomekling · commit beda751d33c5 · 2020-04-15T19:12:10.000+02:00
Neither LibM functions are very strong right now, but let's at least
call the right one.
diff --git a/Libraries/LibJS/Runtime/MathObject.cpp b/Libraries/LibJS/Runtime/MathObject.cpp
@@ -108,8 +108,7 @@ Value MathObject::round(Interpreter& interpreter)
     auto number = interpreter.argument(0).to_number();
     if (number.is_nan())
         return js_nan();
-    // FIXME: Use ::round() instead of ::roundf().
-    return Value(::roundf(number.as_double()));
+    return Value(::round(number.as_double()));
 }
 
 Value MathObject::max(Interpreter& interpreter)

Original file line number	Diff line number	Diff line change
`@@ -108,8 +108,7 @@ Value MathObject::round(Interpreter& interpreter)`
`108`	`108`	`auto number = interpreter.argument(0).to_number();`
`109`	`109`	`if (number.is_nan())`
`110`	`110`	`return js_nan();`
`111`		`- // FIXME: Use ::round() instead of ::roundf().`
`112`		`- return Value(::roundf(number.as_double()));`
	`111`	`+ return Value(::round(number.as_double()));`
`113`	`112`	`}`
`114`	`113`
`115`	`114`	`Value MathObject::max(Interpreter& interpreter)`