Re: Review Request: KJS: Extend strictEqual check for numbers by NaN and signbit check
This looks wrong to me; strictEqual is used for ===, which is defined in 11.9.6, and doesn't do any freaky deviations from IEEE FP. You'll likely need a separate version for SameValue proper. On 3/21/12, Bernd Buschinski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104358/ > --- > > (Updated March 22, 2012, 12:41 a.m.) > > > Review request for kdelibs. > > > Changes > --- > > Whoops sorry, wrong/old version uploaded > - now really check for NaN using isNan > > > Description > --- > > In c++ NAN == NAN return false, true in javascript 9.12 Step 4.a > also +0 == -0 is true in c++, false in javascript 9.12 Step 4.b (same for -0 > == +0 , Step 4c) > > > Diffs (updated) > - > > kjs/operations.cpp d4c0066 > > Diff: http://git.reviewboard.kde.org/r/104358/diff/ > > > Testing > --- > > Tesed via ecmascript, fixes some tests that rely on +0 not beeing the same > as -0 > > > Thanks, > > Bernd Buschinski > >
Re: Review Request: KJS: Extend strictEqual check for numbers by NaN and signbit check
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104358/ --- (Updated March 22, 2012, 12:41 a.m.) Review request for kdelibs. Changes --- Whoops sorry, wrong/old version uploaded - now really check for NaN using isNan Description --- In c++ NAN == NAN return false, true in javascript 9.12 Step 4.a also +0 == -0 is true in c++, false in javascript 9.12 Step 4.b (same for -0 == +0 , Step 4c) Diffs (updated) - kjs/operations.cpp d4c0066 Diff: http://git.reviewboard.kde.org/r/104358/diff/ Testing --- Tesed via ecmascript, fixes some tests that rely on +0 not beeing the same as -0 Thanks, Bernd Buschinski
Review Request: KJS: Extend strictEqual check for numbers by NaN and signbit check
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104358/ --- Review request for kdelibs. Description --- In c++ NAN == NAN return false, true in javascript 9.12 Step 4.a also +0 == -0 is true in c++, false in javascript 9.12 Step 4.b (same for -0 == +0 , Step 4c) Diffs - kjs/operations.cpp d4c0066 Diff: http://git.reviewboard.kde.org/r/104358/diff/ Testing --- Tesed via ecmascript, fixes some tests that rely on +0 not beeing the same as -0 Thanks, Bernd Buschinski