Re: [PATCH 2/3] jscript: Improve Number_toString implementation

[Paul Vriens]

Piotr Caban wrote:
> dlls/jscript/number.c     |  113
> ++++++++++++++++++++++++++++++++++++++++++--
>  dlls/jscript/tests/api.js |   50 +++++++++++++++++--
>  2 files changed, 151 insertions(+), 12 deletions(-)

Hi Piotr,

These new tests fail on several platforms, see:

http://test.winehq.org/data/tests/jscript:run.html

Could you have a look?

[PATCH 2/3] jscript: Improve Number_toString implementation

[Piotr Caban]

Hi Paul,

Could you rerun the tests with following patch? (error message will be
more verbose).

Thanks,
Piotr

diff --git a/dlls/jscript/tests/api.js b/dlls/jscript/tests/api.js
index 6af9bb8..7b1afe6 100644
--- a/dlls/jscript/tests/api.js
+++ b/dlls/jscript/tests/api.js
@@ -607,7 +607,7 @@ function equals(val, base) {
         num *= Math.pow(base, exp);
     }

-    ok(num>val-val/1000 && num<val+val/1000, "equals: num = " + num);
+    ok(num>val-val/1000 && num<val+val/1000, "equals: num = " + num + " val = " + val + " base = " + base + " str = " + str);
 }

ok((10).toString(11) === "a", "(10).toString(11) = " + (10).toString(11));

[PATCH 2/3] jscript: Improve Number_toString implementation

[Paul Vriens]

Piotr Caban wrote:
> Hi Paul,
>
> Could you rerun the tests with following patch? (error message will be
> more verbose).
>
> Thanks,
> Piotr

Hi Piotr,

Win98 results attached.